Project

General

Profile

Actions
Copy link

Bug #27868

closed

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

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

Status:
Closed
Priority:
Normal
Assignee:
Oleh Fedorenko
Category:
-
Target version:
-
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/theforeman/hammer-cli-foreman/pull/448, https://github.com/theforeman/hammer-cli-foreman/pull/450
Team Backlog:
Hammer
Fixed in Releases:
hammer-cli-foreman-0.19.1, hammer-cli-foreman-0.18.2, hammer-cli-0.18.1, hammer-cli-0.19.0
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 2 (0 open — 2 closed)

Related to Foreman - Bug #27897: Param type should not be required in host(group) apiClosedOleh FedorenkoActions
Related to Foreman - Feature #27544: Hosts and Hostgroups API endpoints don't provide parameter_type and hidden_value supportClosedBaptiste AgasseActions
Actions
Copy link
 #1

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from New to Assigned
  • Assignee set to Oleh Fedorenko
  • Triaged changed from No to Yes
Actions
Copy link
 #2

Updated by Ewoud Kohl van Wijngaarden over 5 years ago

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

Updated by Oleh Fedorenko over 5 years ago

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

Updated by The Foreman Bot over 5 years ago

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

Updated by Martin Bacovsky over 5 years ago

  • Team Backlog Hammer added
Actions
Copy link
 #6

Updated by The Foreman Bot over 5 years ago

  • Fixed in Releases hammer-cli-foreman-0.19.0 added
Actions
Copy link
 #7

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link
 #8

Updated by Oleh Fedorenko over 5 years ago

  • Fixed in Releases hammer-cli-foreman-0.18.1 added
Actions
Copy link
 #9

Updated by The Foreman Bot over 5 years ago

  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/450 added
Actions
Copy link
 #10

Updated by Martin Bacovsky over 5 years ago

  • Status changed from Closed to Ready For Testing
Actions
Copy link
 #11

Updated by Shira Maximov over 5 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)
Actions
Copy link
 #12

Updated by Shira Maximov over 5 years ago

  • Fixed in Releases hammer-cli-foreman-0.18.2 added
Actions
Copy link
 #13

Updated by Oleh Fedorenko over 5 years ago

  • Status changed from Ready For Testing to Closed
Actions
Copy link

Also available in: Atom PDF