Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #29867 - '--show-host-status' as a flag, not boolean argument #38

Closed

Conversation

stejskalleos
Copy link
Contributor

Following suggestion from https://bugzilla.redhat.com/show_bug.cgi?id=1825458#c7 comment

Before:

hammer job-invocation info --id 15 --host-status true/1

Now:

hammer job-invocation info --id 15 --show-host-status

@adamruzicka
Copy link
Contributor

Could you post output of hammer job-invocation info --help here?

@stejskalleos
Copy link
Contributor Author

Usage:
    hammer job-invocation <info|show> [OPTIONS]

Options:
 --fields FIELDS                   Show specified fields or predefined field sets only. (See below)
                                   Comma separated list of values. Values containing comma should be quoted or escaped with backslash.
                                   JSON is acceptable and preferred way for complex parameters
 --host-status HOST_STATUS         Show Job status for the hosts
                                   One of true/false, yes/no, 1/0.
 --id ID                            
 --location-id LOCATION_ID         Set the current location context for the request
 --organization-id ORGANIZATION_ID Set the current organization context for the request
 --show-host-status                Show the status of job result for the Hosts.
 --show-inputs                     Show the complete input of the job
 -h, --help                        Print help

@adamruzicka
Copy link
Contributor

This PR should also take care of removing --host-status HOST_STATUS Show Job status for the hosts

@stejskalleos
Copy link
Contributor Author

This PR should also take care of removing --host-status HOST_STATUS Show Job status for the hosts

@adamruzicka updated

@adamruzicka
Copy link
Contributor

Merged as 83cb30b and 6a2c661

@adamruzicka adamruzicka closed this Dec 8, 2020
@adamruzicka
Copy link
Contributor

Thank you @stejskalleos !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants