Bug 1320086

Summary: katello-service man page - supported options are not working as expected
Product: Red Hat Satellite Reporter: Tazim Kolhar <tkolhar>
Component: OtherAssignee: Chris Roberts <chrobert>
Status: CLOSED UPSTREAM QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.2.0CC: bbuckingham, chrobert, cwelton, sauchter
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-01-18 16:58:37 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:
Attachments:
Description Flags
Attaching foreman-debug none

Description Tazim Kolhar 2016-03-22 09:56:30 UTC
Created attachment 1138968 [details]
Attaching foreman-debug

Description of problem:
 katello-service man page - supported options are not working as expected

Version-Release number of selected component (if applicable):
# rpm -qa | grep foreman
tfm-rubygem-foreman_gutterball-0.0.1-6.el7sat.noarch
ibm-x3550m3-09.lab.eng.brq.redhat.com-foreman-proxy-1.0-1.noarch
foreman-selinux-1.11.0-1.el7sat.noarch
tfm-rubygem-foreman_theme_satellite-0.1.2-1.el7sat.noarch
tfm-rubygem-hammer_cli_foreman-0.5.1.2-1.el7sat.noarch
tfm-rubygem-foreman_openscap-0.5.3.0-1.el7sat.noarch
ibm-x3550m3-09.lab.eng.brq.redhat.com-foreman-client-1.0-1.noarch
tfm-rubygem-hammer_cli_foreman_tasks-0.0.10-2.el7sat.noarch
foreman-compute-1.11.0.6-1.el7sat.noarch
tfm-rubygem-foreman_bootdisk-6.1.0-1.el7sat.noarch
foreman-gce-1.11.0.6-1.el7sat.noarch
foreman-postgresql-1.11.0.6-1.el7sat.noarch
foreman-proxy-1.11.0.1-1.el7sat.noarch
foreman-installer-1.11.0.0-1.el7sat.noarch
tfm-rubygem-foreman-tasks-0.7.14.1-1.el7sat.noarch
foreman-vmware-1.11.0.6-1.el7sat.noarch
foreman-debug-1.11.0.6-1.el7sat.noarch
tfm-rubygem-foreman_hooks-0.3.9-2.el7sat.noarch
foreman-discovery-image-3.0.5-3.el7sat.noarch
tfm-rubygem-hammer_cli_foreman_bootdisk-0.1.3-4.el7sat.noarch
foreman-1.11.0.6-1.el7sat.noarch
foreman-ovirt-1.11.0.6-1.el7sat.noarch
foreman-libvirt-1.11.0.6-1.el7sat.noarch
tfm-rubygem-foreman_discovery-5.0.0.1-1.el7sat.noarch
foreman-installer-katello-3.0.0.3-1.el7sat.noarch
tfm-rubygem-hammer_cli_foreman_docker-0.0.3-5.el7sat.noarch
tfm-rubygem-foreman_docker-2.0.1-1.el7sat.noarch
tfm-rubygem-foreman-redhat_access-1.0.1-1.el7sat.noarch
ibm-x3550m3-09.lab.eng.brq.redhat.com-foreman-proxy-client-1.0-1.noarch
puppet-foreman_scap_client-0.3.3-10.el7sat.noarch
tfm-rubygem-foreman_remote_execution-0.3.0.1-1.el7sat.noarch


How reproducible:
everytime

Steps to Reproduce:
1.# katello-service --only tomcat.service restart
    Success!
    Success!
2.  # katello-service --exclude tomcat.service restart
    Redirecting to /bin/systemctl stop  foreman-tasks.service

    Redirecting to /bin/systemctl stop  httpd.service

    Redirecting to /bin/systemctl stop  pulp_workers.service

    Redirecting to /bin/systemctl stop  pulp_resource_manager.service

    Redirecting to /bin/systemctl stop  pulp_celerybeat.service

    Redirecting to /bin/systemctl stop  foreman-proxy.service

    Redirecting to /bin/systemctl stop  tomcat.service

    Redirecting to /bin/systemctl stop  qdrouterd.service

    Redirecting to /bin/systemctl stop  qpidd.service

    Redirecting to /bin/systemctl stop  postgresql.service

    Redirecting to /bin/systemctl stop  mongod.service

    Success!
    Redirecting to /bin/systemctl stop  foreman-tasks.service

    Redirecting to /bin/systemctl stop  httpd.service

    Redirecting to /bin/systemctl stop  pulp_workers.service

    Redirecting to /bin/systemctl stop  pulp_resource_manager.service

    Redirecting to /bin/systemctl stop  pulp_celerybeat.service

    Redirecting to /bin/systemctl stop  foreman-proxy.service

    Redirecting to /bin/systemctl stop  tomcat.service

    Redirecting to /bin/systemctl stop  qdrouterd.service

    Redirecting to /bin/systemctl stop  qpidd.service

    Redirecting to /bin/systemctl stop  postgresql.service

    Redirecting to /bin/systemctl stop  mongod.service

    Success!
3.

Actual results:
The messages displayed twice and services restarted twice
The excluded service does not get skipped

Expected results:
The excluded service should skip
The messages displayed once and services restarted

Additional info:
foreman-debug attached

Comment 3 Chris Roberts 2017-01-18 16:58:37 UTC
Tested on Katello 3.3

[root@centos7 bin]# katello-service --exclude tomcat restart
Redirecting to /bin/systemctl stop  foreman-tasks.service

Redirecting to /bin/systemctl stop  httpd.service

Redirecting to /bin/systemctl stop  pulp_celerybeat.service

Redirecting to /bin/systemctl stop  foreman-proxy.service

Redirecting to /bin/systemctl stop  pulp_streamer.service

Redirecting to /bin/systemctl stop  pulp_resource_manager.service

Redirecting to /bin/systemctl stop  pulp_workers.service

Redirecting to /bin/systemctl stop  squid.service

Redirecting to /bin/systemctl stop  qdrouterd.service

Redirecting to /bin/systemctl stop  qpidd.service

Redirecting to /bin/systemctl stop  postgresql.service

Redirecting to /bin/systemctl stop  mongod.service

Success!
Redirecting to /bin/systemctl start  mongod.service

Redirecting to /bin/systemctl start  postgresql.service

Redirecting to /bin/systemctl start  qpidd.service

Redirecting to /bin/systemctl start  qdrouterd.service

Redirecting to /bin/systemctl start  squid.service

Redirecting to /bin/systemctl start  pulp_workers.service

Redirecting to /bin/systemctl start  pulp_resource_manager.service

Redirecting to /bin/systemctl start  pulp_streamer.service

Redirecting to /bin/systemctl start  foreman-proxy.service

Redirecting to /bin/systemctl start  pulp_celerybeat.service

Redirecting to /bin/systemctl start  httpd.service

Redirecting to /bin/systemctl start  foreman-tasks.service

Success!