Bug #7085

hammer hostroup create & update requires all options as ids.

Added by Tomáš Strachota over 3 years ago. Updated over 3 years ago.

Status:Closed
Priority:Normal
Assigned To:Tomáš Strachota
Category:-
Target version:Foreman - Sprint 27
Difficulty: Bugzilla link:1102331
Found in release: Pull request:https://github.com/theforeman/hammer-cli-foreman/pull/147
Story points-
Velocity based estimate-

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 over 3 years ago

Fixes #7085 - name options for puppet proxies in hostgroup

Revision b6ddaa68
Added by Tomas Strachota over 3 years ago

Merge pull request #147 from tstrachota/hostgroups

Fixes #7085 - name options for puppet proxies in hostgroup

History

#1 Updated by Tomáš Strachota over 3 years ago

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

#2 Updated by The Foreman Bot over 3 years ago

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

#3 Updated by Tomáš Strachota over 3 years ago

  • Target version deleted (Sprint 27)

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 over 3 years ago

  • Target version set to Sprint 27

#5 Updated by Anonymous over 3 years ago

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

Also available in: Atom PDF