Bug #7085

hammer hostroup create & update requires all options as ids.

Added by Tomáš Strachota almost 4 years ago. Updated 6 days ago.

Status:Closed
Priority:Normal
Assignee:Tomáš Strachota
Category:-
Target version:1.7.5
Difficulty: Team Backlog:
Triaged: Fixed in Releases:
Bugzilla link:1102331 Found in Releases:
Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/147

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1102331
Description of problem:

Looking at hammer hostgroup create --help:

[root@box ~]# hammer hostgroup create --help
Usage:
hammer hostgroup create [OPTIONS]

Options:
--architecture-id ARCHITECTURE_ID
--domain-id DOMAIN_ID
--environment-id ENVIRONMENT_ID
--medium-id MEDIUM_ID
--name NAME
--operatingsystem-id OPERATINGSYSTEM_ID
--parent-id PARENT_ID
--ptable-id PTABLE_ID
--puppet-ca-proxy-id PUPPET_CA_PROXY_ID
--puppet-proxy-id PUPPET_PROXY_ID
--puppetclass-ids PUPPETCLASS_IDS Comma separated list of values.
--realm-id REALM_ID
--subnet-id SUBNET_ID
-h, --help print help

The following options require you to specify an id:

--architecture-id ARCHITECTURE_ID  
--domain-id DOMAIN_ID
--environment-id ENVIRONMENT_ID
--medium-id MEDIUM_ID
--operatingsystem-id OPERATINGSYSTEM_ID
--parent-id PARENT_ID
--ptable-id PTABLE_ID
--puppet-ca-proxy-id PUPPET_CA_PROXY_ID
--puppet-proxy-id PUPPET_PROXY_ID
--puppetclass-ids PUPPETCLASS_IDS Comma separated list of values.
--realm-id REALM_ID
--subnet-id SUBNET_ID

There should be someway to do all this without referencing the ID.

Associated revisions

Revision 1eca7638
Added by Tomas Strachota almost 4 years ago

Fixes #7085 - name options for puppet proxies in hostgroup

Revision b6ddaa68
Added by Tomas Strachota almost 4 years ago

Merge pull request #147 from tstrachota/hostgroups

Fixes #7085 - name options for puppet proxies in hostgroup

History

#1 Updated by Tomáš Strachota almost 4 years ago

  • Project changed from Foreman to Hammer CLI
  • Status changed from New to Assigned

#2 Updated by The Foreman Bot almost 4 years ago

  • Status changed from Assigned to Ready For Testing
  • Target version set to 1.7.5
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/147 added

#3 Updated by Tomáš Strachota almost 4 years ago

  • Target version deleted (1.7.5)

The associated PR fixes only --puppet-ca-proxy and --puppet-proxy. The other options are already fixed in the master.

#4 Updated by The Foreman Bot almost 4 years ago

  • Target version set to 1.7.5

#5 Updated by Anonymous almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF