Bug 732538
Summary: | adding a "person" pool to an activation key should be blocked | ||
---|---|---|---|
Product: | [Retired] Subscription Asset Manager | Reporter: | John Sefler <jsefler> |
Component: | candlepin | Assignee: | William Poteat <wpoteat> |
Status: | CLOSED WONTFIX | QA Contact: | SAM QE List <sam-qe-list> |
Severity: | low | Docs Contact: | |
Priority: | unspecified | ||
Version: | 1.3 | CC: | bkearney, dgoodwin, gkhachik, jesusr, jomara, skallesh, spandey, tkolhar, tomckay, wpoteat |
Target Milestone: | rc | Keywords: | Triaged |
Target Release: | 1.3 | ||
Hardware: | Unspecified | ||
OS: | Solaris | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-08-14 20:31:30 UTC | Type: | --- |
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: | 833466 |
Description
John Sefler
2011-08-22 20:21:13 UTC
I'm setting this back to NEW as I don't see where a fix or proposed fix was discussed or committed. In my testing it appears that some work has been done here as type=person is now blocked; however, the error message returned is not very useful. # subscription-manager register --username=admin --password=admin --org=ACME_Corporation --environment=DEV --type=person Validation failed: Cp type is not included in the list katello-headpin-all-0.1.135-2.el6.noarch This is not our most efficient bug of all time. :) This was indeed fixed with the requested message in comment 1 by wpoteat. Will: Please note commit hash and branch the fix will appear in. Eric: In your comment you are reporting the error message client side when using the key, not the error message blocking the person pool from being added to the activation key to begin with. (which is what this bug is about) I think the message you are reporting is also coming from Katello, so if this is still an issue we should file a separate bug to have that text fix, it looks like rails validation code to me. I am changing the error message from comment #1 however to something more concise: "Cannot add pools restricted to consumer type 'person' to activation keys." fixed in master by c387aeb0ef9500de1921b6034cdeedfb599f26be Will be in candlepin-tomcat6 0.7.5 or later. FAILED: Tried testing with the packages : # rpm -qa |grep katello-headpin-all katello-headpin-all-1.1.15-2h.el6_3.noarch # rpm -qa | grep candlepin-tomcat candlepin-tomcat6-0.7.13-1.el6_3.noarch On client machine registered to sam server (1.2 puddle) : # subscription-manager register --username=testuser1 --password=password --org=admin --type person Validation failed: Cp type is not included in the list had followed the steps given in the previous comments to verify but it failed in step 1. moving it to NEW . Switching product from Subscription Asset Manager to Candlepin for visibility Tested in current source on 12/19/12, commit be639376f54aa75ce683bf17a002151eceb3c146 [wpoteat@wpoteat-desktop candlepin (master u=)]$ curl -k -u admin:admin -X POST https://localhost:8443/candlepin/activation_keys/8a8d09953bb8ad52013bb8b1e247003f/pools/8a8d09953bb4a6c9013bb4a7bad504fa { "displayMessage" : "Cannot add pools restricted to consumer type 'person' to activation keys." } Moving all POST / MODIFIED bugs to ON_QA due to the new builds. FailedQA : # rpm -qa | grep katello katello-selinux-1.4.4-2.el6sat.noarch katello-candlepin-cert-key-pair-1.0-1.noarch katello-certs-tools-1.4.2-2.el6sat.noarch katello-cli-common-1.4.3-5.el6sat.noarch katello-cli-1.4.3-5.el6sat.noarch katello-common-1.4.3-6.el6sam_splice.noarch katello-configure-1.4.4-2.el6sat.noarch katello-glue-elasticsearch-1.4.3-6.el6sam_splice.noarch katello-headpin-all-1.4.3-6.el6sam_splice.noarch katello-glue-candlepin-1.4.3-6.el6sam_splice.noarch signo-katello-0.0.10-2.el6sat.noarch katello-headpin-1.4.3-6.el6sam_splice.noarch # subscription-manager register --username=admin --password=*** --org ACME_Corporation --type person --force Validation failed: Cp type is not included in the list We dot use person subscriptions. Closing. |