Bug 1132174

Summary: hammer product remove-sync-plan - Remove unused options
Product: Red Hat Satellite Reporter: sthirugn <sthirugn>
Component: HammerAssignee: Mike McCune <mmccune>
Status: CLOSED ERRATA QA Contact: sthirugn <sthirugn>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: bbuckingham, bkearney
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/7357
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-08-12 05:15:09 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 sthirugn@redhat.com 2014-08-20 20:37:42 UTC
Description of problem:
hammer product remove-sync-plan - Remove unused options

Version-Release number of selected component (if applicable):
Version Tested:
GA Snap 6 - Satellite-6.0.4-RHEL-6-20140820.1

* apr-util-ldap-1.3.9-3.el6_0.1.x86_64
* candlepin-0.9.23-1.el6_5.noarch
* candlepin-common-1.0.1-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.23-1.el6_5.noarch
* candlepin-tomcat6-0.9.23-1.el6_5.noarch
* elasticsearch-0.90.10-6.el6sat.noarch
* foreman-1.6.0.41-1.el6sat.noarch
* foreman-compute-1.6.0.41-1.el6sat.noarch
* foreman-gce-1.6.0.41-1.el6sat.noarch
* foreman-libvirt-1.6.0.41-1.el6sat.noarch
* foreman-ovirt-1.6.0.41-1.el6sat.noarch
* foreman-postgresql-1.6.0.41-1.el6sat.noarch
* foreman-proxy-1.6.0.29-1.el6sat.noarch
* foreman-selinux-1.6.0.7-1.el6sat.noarch
* foreman-vmware-1.6.0.41-1.el6sat.noarch
* katello-1.5.0-29.el6sat.noarch
* katello-ca-1.0-1.noarch
* katello-certs-tools-1.5.6-1.el6sat.noarch
* katello-installer-0.0.60-1.el6sat.noarch
* openldap-2.4.23-34.el6_5.1.x86_64
* openldap-devel-2.4.23-34.el6_5.1.x86_64
* pulp-katello-0.3-3.el6sat.noarch
* pulp-nodes-common-2.4.0-0.30.beta.el6sat.noarch
* pulp-nodes-parent-2.4.0-0.30.beta.el6sat.noarch
* pulp-puppet-plugins-2.4.0-0.30.beta.el6sat.noarch
* pulp-puppet-tools-2.4.0-0.30.beta.el6sat.noarch
* pulp-rpm-plugins-2.4.0-0.30.beta.el6sat.noarch
* pulp-selinux-2.4.0-0.30.beta.el6sat.noarch
* pulp-server-2.4.0-0.30.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

How reproducible:
Always

Steps to Reproduce:
hammer> product remove-sync-plan --help
Usage:
     product remove-sync-plan [OPTIONS]

Options:
    --gpg-key GPG_KEY_NAME        Name to search by
    --gpg-key-id GPG_KEY_ID       gpg key numeric identifier
    --id ID                       product numeric identifier
    --name NAME                   Product name to search by
    --new-name NEW_NAME           Product name
    --organization ORGANIZATION_NAME Organization name to search by
    --organization-id ORGANIZATION_ID  
    --organization-label ORGANIZATION_LABEL Organization label to search by
    --sync-plan SYNC_PLAN_NAME    Sync plan name to search by
    --sync-plan-id SYNC_PLAN_ID   sync plan numeric identifier
    -h, --help                    print help


Actual results:
Currently a Product can be associated to only one sync plan.  So the following options are irrelevant for now.
    --sync-plan SYNC_PLAN_NAME    Sync plan name to search by
    --sync-plan-id SYNC_PLAN_ID   sync plan numeric identifier


Expected results:
Please remove the following irrelevant options:
    --sync-plan SYNC_PLAN_NAME    Sync plan name to search by
    --sync-plan-id SYNC_PLAN_ID   sync plan numeric identifier

Additional info:
If product remove-sync-plan is called, remove any sync plan associated with my product.  If no sync plan is set, let me know that the sync plan is not set yet.

Comment 2 Dustin Tsang 2014-09-04 18:06:15 UTC
Created redmine issue http://projects.theforeman.org/issues/7357 from this bug

Comment 3 Bryan Kearney 2014-09-19 16:04:26 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/7357 has been closed
-------------
dustin tsang
Applied in changeset commit:hammer-cli-katello|9662fea8222ede0e26bd8182f10599d132f47e0f.

Comment 6 sthirugn@redhat.com 2015-03-17 18:00:01 UTC
Verified.

Version Tested:
Satellite-6.1.0-RHEL-6-20150311.1

hammer> product remove-sync-plan --help
Usage:
     product remove-sync-plan [OPTIONS]

Options:
 --description DESCRIPTION               Product description
 --gpg-key GPG_KEY_NAME                  Name to search by
 --gpg-key-id GPG_KEY_ID                 gpg key numeric identifier
 --id ID                                 product numeric identifier
 --name NAME                             Product name to search by
 --new-name NEW_NAME                     Product name
 --organization ORGANIZATION_NAME        Organization name to search by
 --organization-id ORGANIZATION_ID       organization ID
 --organization-label ORGANIZATION_LABEL Organization label to search by
 -h, --help                              print help


hammer> product set-sync-plan --name=fakeproduct1 --sync-plan-id=4 --organization-id=1
Synchronization plan assigned.
hammer> product remove-sync-plan --name=fakeproduct1 --organization-id=1
Synchronization plan removed.

Comment 8 Bryan Kearney 2015-08-11 13:33:50 UTC
This bug is slated to be released with Satellite 6.1.

Comment 9 errata-xmlrpc 2015-08-12 05:15:09 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:1592