Bug 1096241

Summary: hammer organization delete do not accept --id argument
Product: Red Hat Satellite Reporter: Elyézer Rezende <erezende>
Component: HammerAssignee: Adam Price <adprice>
Status: CLOSED CURRENTRELEASE QA Contact: sthirugn <sthirugn>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: bbuckingham, cwelton, jmontleo, sthirugn
Target Milestone: UnspecifiedKeywords: Regression, Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/5983
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-07-02 14:05:34 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:

Description Elyézer Rezende 2014-05-09 14:05:59 UTC
Description of problem:
When trying to delete an organization by its ID the hammer organization delete command does not accept the --id argument.


Version-Release number of selected component (if applicable):
# hammer --version
hammer (0.1.0)
 * hammer_cli_foreman (0.1.0)
 * hammer_cli_katello (0.0.3)


How reproducible:
Aways


Steps to Reproduce:
1. hammer organization delete --id=<an_org_id>

Actual results:
The command fail with the following error: Error: Unrecognised option '--id'


Expected results:
The command should accept the --id argument and delete the organization.


Additional info:
The --id argument is not listed on the expected arguments.

# hammer organization delete -h
Usage:
    hammer organization delete [OPTIONS]

Options:
    --label LABEL                 Label to search by
    --name NAME                   Name to search by
    -h, --help                    print help

Comment 3 Adam Price 2014-05-29 14:01:09 UTC
Created redmine issue http://projects.theforeman.org/issues/5983 from this bug

Comment 4 Adam Price 2014-05-30 15:17:57 UTC
https://github.com/Katello/katello/pull/4177

Comment 6 sthirugn@redhat.com 2014-06-06 16:11:26 UTC
Verified.

# hammer organization delete --help
Usage:
    hammer organization delete [OPTIONS]

Options:
    --id ID                       Organization ID
    --label LABEL                 Label to search by
    --name NAME                   Name to search by
    -h, --help                    print help


# hammer organization delete --id=5
Organization deleted


Version Tested:
* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.7-1.el6_5.noarch
* candlepin-scl-1-5.el6_4.noarch
* candlepin-scl-quartz-2.1.5-5.el6_4.noarch
* candlepin-scl-rhino-1.7R3-1.el6_4.noarch
* candlepin-scl-runtime-1-5.el6_4.noarch
* candlepin-selinux-0.9.7-1.el6_5.noarch
* candlepin-tomcat6-0.9.7-1.el6_5.noarch
* elasticsearch-0.90.10-4.el6sat.noarch
* foreman-1.6.0.14-1.el6sat.noarch
* foreman-compute-1.6.0.14-1.el6sat.noarch
* foreman-gce-1.6.0.14-1.el6sat.noarch
* foreman-libvirt-1.6.0.14-1.el6sat.noarch
* foreman-ovirt-1.6.0.14-1.el6sat.noarch
* foreman-postgresql-1.6.0.14-1.el6sat.noarch
* foreman-proxy-1.6.0.6-1.el6sat.noarch
* foreman-selinux-1.6.0-4.el6sat.noarch
* foreman-vmware-1.6.0.14-1.el6sat.noarch
* katello-1.5.0-25.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.5-1.el6sat.noarch
* katello-installer-0.0.45-1.el6sat.noarch
* openldap-2.4.23-32.el6_4.1.x86_64
* pulp-katello-0.3-3.el6sat.noarch
* pulp-nodes-common-2.4.0-0.18.beta.el6sat.noarch
* pulp-nodes-parent-2.4.0-0.18.beta.el6sat.noarch
* pulp-puppet-plugins-2.4.0-0.18.beta.el6sat.noarch
* pulp-puppet-tools-2.4.0-0.18.beta.el6sat.noarch
* pulp-rpm-plugins-2.4.0-0.18.beta.el6sat.noarch
* pulp-selinux-2.4.0-0.18.beta.el6sat.noarch
* pulp-server-2.4.0-0.18.beta.el6sat.noarch
* python-ldap-2.3.10-1.el6.x86_64
* ruby193-rubygem-net-ldap-0.3.1-3.el6sat.noarch
* ruby193-rubygem-runcible-1.1.0-2.el6sat.noarch

Comment 7 Bryan Kearney 2014-07-02 14:05:34 UTC
This was delivered with 6.0.3, which is the Satellite 6 Beta.