Bug #34873
closedmissing information about puppet attributes in API/CLI
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2049595
Description of problem:
Consist from 2 parts. Missing Apidoc and hammer documentation.
First part missing apidoc is regression from 6.10 and second part with hammer is not.
Version-Release number of selected component (if applicable):
Sat 7.0 snap 7 rhel 7
How reproducible:
Always
Steps to Reproduce:
1. enable puppet on the satellite [0]
1.5 hammer -r (another bug, #BZ2039696)
2. hammer host create/update shows: (look below in actual results)
second part:
3. open apidoc an look at create/edit host, (described in actual results)
- hammer host create --help | grep puppet_attributes
--config-group[s|-ids] LIST Deprecated in favor of host/puppet_attributes/config_group_names/ids
--environment[-id] VALUE Deprecated in favor of host/puppet_attributes/environment_name/id
--puppetclass[es|-ids] LIST Deprecated in favor of host/puppet_attributes/puppetclass_names/ids
- hammer host update --help | grep puppet_attributes
--config-group[s|-ids] LIST Deprecated in favor of host/puppet_attributes/config_group_names/ids
--environment[-id] VALUE Deprecated in favor of host/puppet_attributes/environment_name/id
--puppetclass[es|-ids] LIST Deprecated in favor of host/puppet_attributes/puppetclass_names/ids
second part:
create/update host apidoc -
i) missing depreceted one - host[puppetclass_ids], host[config_group_ids], host[config_group_ids] - mentioned regression from 6.10
ii) missing new one - puppet_attributes
Expected results:
A) hammer shows the not deprecated options of puppet_attributes
B) apidoc shows the deprecated and non-deprecated puppet options
Additional info:
is there some other entity except host that can use puppet (config groups, environment or puppetclasses) as option inside subaction like (create/edit/...) if so I guess the documentation will be missing there as well.
[0] - satellite-installer --enable-foreman-plugin-puppet \
--enable-foreman-cli-puppet \
--foreman-proxy-puppet true \
--foreman-proxy-puppetca true \
--foreman-proxy-content-puppet true \
--enable-puppet \
--puppet-server true \
--puppet-server-foreman-ssl-ca /etc/pki/katello/puppet/puppet_client_ca.crt \
--puppet-server-foreman-ssl-cert /etc/pki/katello/puppet/puppet_client.crt \
--puppet-server-foreman-ssl-key /etc/pki/katello/puppet/puppet_client.key
Updated by Oleh Fedorenko almost 2 years ago
- Project changed from Hammer CLI to Puppet
- Status changed from New to Assigned
- Assignee set to Oleh Fedorenko
Updated by The Foreman Bot almost 2 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman-puppet/pull/44 added
Updated by Oleh Fedorenko almost 2 years ago
- Status changed from Assigned to Closed