Bug 872724
Summary: | Requesting package profile on consumer without a package profile results in error | |||
---|---|---|---|---|
Product: | [Retired] Pulp | Reporter: | Justin Sherrill <jsherril> | |
Component: | consumers | Assignee: | Sayli Karmarkar <skarmark> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Preethi Thomas <pthomas> | |
Severity: | unspecified | Docs Contact: | ||
Priority: | unspecified | |||
Version: | Master | CC: | cperry, jason.dobies, skarmark | |
Target Milestone: | --- | Keywords: | Triaged | |
Target Release: | 2.1.0 | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 874243 (view as bug list) | Environment: | ||
Last Closed: | 2013-04-08 16:02:42 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 854726, 874243 |
Description
Justin Sherrill
2012-11-02 20:15:24 UTC
I think it would make sense to return an empty array or a 404 (if it was not created). A 500 doesn't seem correct though. This is already fixed in the current build. Moving to on_qa. $ curl -u admin:admin -k https://localhost/pulp/api/v2/consumers/consumer2/profiles/rpm/ {"http_request_method": "GET", "exception": null, "error_message": "Missing resource(s): profile_id={'content_type': u'rpm', 'consumer_id': u'consumer2'}", "_href": "/pulp/api/v2/consumers/consumer2/profiles/rpm/", "http_status": 404, "traceback": null, "resources": {"profile_id": {"content_type": "rpm", "consumer_id": "consumer2"}}} There is no current build for 2.1.0 (i.e. a QE testing build). The build today was for the 2.0.7 release, which this isn't aligned to. I don't remember seeing a commit for this in the pulp-2.0 branch, nor should it be there. Are you sure this is ON_QA? FYI i just tested this on pulp-server-2.0.6-1.noarch and it seems to return a 404 and not a 500. So i would consider this ON_QA. (In reply to comment #3) > There is no current build for 2.1.0 (i.e. a QE testing build). The build > today was for the 2.0.7 release, which this isn't aligned to. I don't > remember seeing a commit for this in the pulp-2.0 branch, nor should it be > there. Are you sure this is ON_QA? Ya, it was fixed quite a few days back as a part of other bugfix. I had forgotten to move it to modified at that time. I verified it yesterday with the latest build and moved it to on_qa. verified [root@pulp-v2-testing ~]# rpm -q pulp-server pulp-server-2.0.7-0.4.beta.noarch [root@pulp-v2-testing ~]# [root@pulp-v2-testing ~]# curl -u admin:admin -k https://localhost/pulp/api/v2/consumers/consumer2/profiles/rpm/ {"http_request_method": "GET", "exception": null, "error_message": "Missing resource(s): profile_id={'content_type': u'rpm', 'consumer_id': u'consumer2'}", "_href": "/pulp/api/v2/consumers/consumer2/profiles/rpm/", "http_status": 404, "traceback": null, "resources": {"profile_id": {"content_type": "rpm", "consumer_id": "consumer2"}}}[root@pulp-v2-testing ~]# Pulp 2.1 released http://www.pulpproject.org/2013/04/05/pulp-2-1-0-released/ |