Bug 1101657

Summary: 'hammer subscription' commands missing --organization & --organization-label options
Product: Red Hat Satellite Reporter: Justin Sherrill <jsherril>
Component: HammerAssignee: Adam Price <adprice>
Status: CLOSED CURRENTRELEASE QA Contact: sthirugn <sthirugn>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: adprice, bkearney, jmontleo, kbidarka, omaciel, sthirugn
Target Milestone: UnspecifiedKeywords: Reopened, Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/5747
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-02 14:08:22 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: 1097647, 1099975, 1101666, 1102331    

Description Justin Sherrill 2014-05-27 17:13:46 UTC
Currently the --organization-id is the only way to pass in an organization

Comment 2 Bryan Kearney 2014-05-27 19:52:49 UTC
Connecting redmine issue http://projects.theforeman.org/issues/5747 from this bug

Comment 3 Bryan Kearney 2014-05-27 20:47:31 UTC

*** This bug has been marked as a duplicate of bug 1095410 ***

Comment 4 Bryan Kearney 2014-06-04 12:57:43 UTC
I opened this back up and moved it to post since it has the upstream bugs attached to it.

Comment 5 Bryan Kearney 2014-06-04 12:57:50 UTC
I opened this back up and moved it to post since it has the upstream bugs attached to it.

Comment 8 sthirugn@redhat.com 2014-06-06 14:50:21 UTC
Failed

# hammer subscription --organization-id=1
Error: Unrecognised option '--organization-id'

See: 'hammer subscription --help'

# hammer subscription --organization='ACME_Corporation'
Error: Unrecognised option '--organization'

See: 'hammer subscription --help'

# hammer subscription --organization-label='ACME_Corporation'
Error: Unrecognised option '--organization-label'

See: 'hammer subscription --help'

Comment 9 sthirugn@redhat.com 2014-06-06 15:35:55 UTC
Ignore my previous comment.  This is the complete list of test scenarios and test results.

It looks like organization-label is not added for upload, manifest-history, delete-manifest and refresh-manifest options. It is added just for list option.  We need to fix this and make it consistent across all options.

# hammer subscription list --organization-id=3 - PASS
# hammer subscription list --organization=sthirugncorp - PASS
# hammer subscription list --organization-label=sthirugncorp - PASS

# hammer subscription upload --file /tmp/manifest.zip --organization-id=4 - PASS
# hammer subscription upload --file /tmp/manifest.zip --organization=testorg - PASS
# hammer subscription upload --file /tmp/manifest.zip --organization-label=testorg - FAIL

# hammer subscription manifest-history --organization-id=4 - PASS
# hammer subscription manifest-history --organization=testorg - PASS
# hammer subscription manifest-history --organization-label=testorg - FAIL

# hammer subscription delete-manifest --organization-id=4 - PASS
# hammer subscription delete-manifest --organization=testorg  - PASS
# hammer subscription delete-manifest --organization-label=testorg - FAIL

# hammer subscription refresh-manifest --organization-id=4 - PASS
# hammer subscription refresh-manifest --organization=testorg - PASS
# hammer subscription refresh-manifest --organization-label=testorg - FAIL

Version Tested:
katello-ca-1.0-1.noarch
pulp-katello-0.3-3.el6sat.noarch
katello-1.5.0-25.el6sat.noarch
rubygem-hammer_cli_katello-0.0.4-5.el6sat.noarch
ruby193-rubygem-katello-1.5.0-54.el6sat.noarch
katello-certs-tools-1.5.5-1.el6sat.noarch
katello-installer-0.0.45-1.el6sat.noarch
foreman-libvirt-1.6.0.14-1.el6sat.noarch
rubygem-hammer_cli_foreman-0.1.1-8.el6sat.noarch
foreman-proxy-1.6.0.6-1.el6sat.noarch
ruby193-rubygem-foreman-redhat_access-0.0.4-1.el6sat.noarch
foreman-selinux-1.6.0-4.el6sat.noarch
ibm-x3250m4-05.lab.eng.rdu2.redhat.com-foreman-client-1.0-1.noarch
foreman-ovirt-1.6.0.14-1.el6sat.noarch
ruby193-rubygem-foreman_bootdisk-2.0.6-1.1.el6sat.noarch
ruby193-rubygem-foreman-tasks-0.5.7-2.el6sat.noarch
foreman-postgresql-1.6.0.14-1.el6sat.noarch
foreman-compute-1.6.0.14-1.el6sat.noarch
foreman-vmware-1.6.0.14-1.el6sat.noarch
ruby193-rubygem-foreman_hooks-0.3.5-2.el6sat.noarch
foreman-1.6.0.14-1.el6sat.noarch
ruby193-rubygem-foreman_discovery-1.3.0-0.1.rc2.el6sat.noarch
ibm-x3250m4-05.lab.eng.rdu2.redhat.com-foreman-proxy-1.0-1.noarch
foreman-gce-1.6.0.14-1.el6sat.noarch
rubygem-hammer_cli_foreman_tasks-0.0.3-2.el6sat.noarch

Comment 10 sthirugn@redhat.com 2014-06-06 16:05:28 UTC
A new bug https://bugzilla.redhat.com/show_bug.cgi?id=1105653 is written up to fix the failures in Comment 9.  So marking this bug VERIFIED.

Comment 11 Bryan Kearney 2014-07-02 14:08:22 UTC
This was delivered with 6.0.3, which is the Satellite 6 Beta.

Comment 12 Bryan Kearney 2014-07-02 14:09:43 UTC
This was delivered in 6.0.3, the Beta version of Satellite 6.0