Bug #23204
closedinconsitent puppet environment naming in hammer
Description
host create/update commands use environment
# hammer host create --help | grep environment --environment ENVIRONMENT_NAME Environment name --environment-id ENVIRONMENT_ID
but host info reports puppet environment
# hammer host info --id 1 | head -n 5 Id: 1 Name: hostname.. Puppet Environment: production Puppet CA Proxy: hostname.. Puppet Master Proxy: hostname..
environment was changed to puppet environment in 6e83c9d82183f39dd4fd611698c0437409e1ee7a
http://projects.theforeman.org/issues/18750
tfm-rubygem-hammer_cli_katello-0.11.0-1.201804051800git94089a6.el7.noarch
tfm-rubygem-hammer_cli-0.13.pre.develop-1.201803232224gite493035.el7.noarch
tfm-rubygem-hammer_cli_foreman-0.13.pre.develop-1.201804041834git768148d.el7.noarch
Updated by Tomáš Strachota over 6 years ago
- Category set to Hosts
I agree that it would be better to name the parameters --puppet-environment instead of --environment. We should also consistently change the hostgroup command and possibly others that use --environment.
One implementation note:
When --puppet-environment is introduced, --environment should be hidden with a deprecation warning to keep backward compatibility. We can then remove it in the next version.
Updated by Oleh Fedorenko almost 6 years ago
- Status changed from New to Assigned
- Assignee set to Oleh Fedorenko
Updated by The Foreman Bot almost 6 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/401 added
Updated by Oleh Fedorenko over 5 years ago
- Related to Bug #26057: Cannot create hostgroup with --environment added
Updated by John Mitsch over 5 years ago
- Pull request https://github.com/Katello/hammer-cli-katello/pull/629 added
Updated by Oleh Fedorenko over 5 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset hammer-cli-foreman|1862075e302144527bcd10845624b881fc438bc1.
Updated by The Foreman Bot about 5 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/437 added