Project

General

Profile

Bug #27868

No way to provide type for hosts parameters in Foreman >= 1.23

Added by Martin Bacovsky over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
-
Found in Releases:
In Kanboard:
Red Hat JIRA:

Description

Attempt to create or update host with parameters fail with

"Could not create the host:
Missing arguments for 'host[host_parameters_attributes][0][parameter_type]'

The reason is in Foreman 1.23 the API changed and parameter_type was added and is required.

Possible fixes:
- consider making the type optional with reasonable default in API. It should unblock the error and save us from incompatible changes in API
- add the default type on Hammer side and deprecate the --parameters option. Add new --typed-parameters that support also the new attributes


Related issues

Related to Foreman - Bug #27897: Param type should not be required in host(group) apiClosed
Related to Foreman - Feature #27544: Hosts and Hostgroups API endpoints don't provide parameter_type and hidden_value supportClosed

Associated revisions

Revision 3288835a (diff)
Added by Oleh Fedorenko over 3 years ago

Fixes #27868 - Add option to support host's param type (#448)

Revision d7e3a798 (diff)
Added by Oleh Fedorenko over 3 years ago

Refs #27868 - Fix method typo (#450)

History

#1 Updated by Oleh Fedorenko over 3 years ago

  • Triaged changed from No to Yes
  • Assignee set to Oleh Fedorenko
  • Status changed from New to Assigned

#2 Updated by Ewoud Kohl van Wijngaarden over 3 years ago

  • Related to Bug #27897: Param type should not be required in host(group) api added

#3 Updated by Oleh Fedorenko over 3 years ago

  • Related to Feature #27544: Hosts and Hostgroups API endpoints don't provide parameter_type and hidden_value support added

#4 Updated by The Foreman Bot over 3 years ago

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

#5 Updated by Martin Bacovsky over 3 years ago

  • Team Backlog Hammer added

#6 Updated by The Foreman Bot over 3 years ago

  • Fixed in Releases hammer-cli-foreman-0.19.0 added

#7 Updated by Oleh Fedorenko over 3 years ago

  • Status changed from Ready For Testing to Closed

#8 Updated by Oleh Fedorenko over 3 years ago

  • Fixed in Releases hammer-cli-foreman-0.18.1 added

#9 Updated by The Foreman Bot over 3 years ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/450 added

#10 Updated by Martin Bacovsky over 3 years ago

  • Status changed from Closed to Ready For Testing

#11 Updated by Shira Maximov over 3 years ago

  • Fixed in Releases hammer-cli-foreman-0.19.1, hammer-cli-0.18.1, hammer-cli-0.19.0 added
  • Fixed in Releases deleted (hammer-cli-foreman-0.18.1, hammer-cli-foreman-0.19.0)

#12 Updated by Shira Maximov over 3 years ago

  • Fixed in Releases hammer-cli-foreman-0.18.2 added

#13 Updated by Oleh Fedorenko over 3 years ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF