Bug 1244821
| Summary: | [RFE] Hammer does not report error if configuration file is not found | ||
|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Peter Vreman <peter.vreman> |
| Component: | Hammer | Assignee: | Tomas Strachota <tstrachota> |
| Status: | CLOSED ERRATA | QA Contact: | Sachin Ghai <sghai> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 6.1.0 | CC: | bbuckingham, mbacovsk, peter.vreman, sghai, sthirugn, xdmoon |
| Target Milestone: | Unspecified | Keywords: | FutureFeature, Triaged |
| Target Release: | Unused | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://projects.theforeman.org/issues/11158 | ||
| Whiteboard: | Verified in Upstream | ||
| Fixed In Version: | Doc Type: | Enhancement | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-07-27 10:59:51 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: | 260381, 1122832 | ||
|
Description
Peter Vreman
2015-07-20 14:33:04 UTC
Since this issue was entered in Red Hat Bugzilla, the release flag has been set to ? to ensure that it is properly evaluated for this release. Created redmine issue http://projects.theforeman.org/issues/11158 from this bug Config file is optional and interactive mode is turned on by default in hammer so the behaviour you describe is intentional. Would explicitly turning interactive mode off solve the problem for you? However I agree that the message hammer prints is not extra helpful and could be improved: $ hammer --interactive no os list Invalid username or password I expect that when i pass a value for a parameter that the value is still validated, even if the parameter is optional. Therefor i expected a Warning that the supplied configuration file is not found, and then it switches to interactive mode. Upstream bug assigned to tstrachota Moving to POST since upstream bug http://projects.theforeman.org/issues/11158 has been closed ------------- Anonymous Applied in changeset commit:169bffe4b7098027279385dafd98e75110c4dfe9. Verified with upstream: foreman-release-1.10.0-0.develop.201509041547git84f7fc6.el6.noarch foreman-1.10.0-0.develop.201509041547git84f7fc6.el6.noarch katello-2.4.0-6.nightly.el6.noarch katello-installer-base-2.4.0-1.nightly.201509040043gitb37b92b.el6.noarch katello-installer-2.4.0-1.nightly.201509040043gitb37b92b.el6.noarch tfm-rubygem-hammer_cli_foreman-0.3.0-2.201508261233git7334971.el6.noarch tfm-rubygem-hammer_cli-0.3.0-2.201508280738git8a70aa0.el6.noarch tfm-rubygem-hammer_cli_katello-0.0.17-4.el6.noarch Now we can see proper error on passing non-existing config file. [root@cloud-qe hammerify]# hammer -u admin -p changeme -c /notexistingfile.conf user list Error: Custom configuration file /notexistingfile.conf does not exist. [root@cloud-qe hammerify]# 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/RHBA-2016:1501 |