Project

General

Profile

Actions

Bug #11137

closed

Hammer sends values for numeric params as strings

Added by Martin Bacovsky almost 9 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Low
Category:
Foreman commands (obsolete)
Target version:
-
Difficulty:
Triaged:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

As seen in the hammer log
[ INFO 2015-07-16 17:27:30 HammerCLIForeman::Host::UpdateCommand] Called with options: {"option_volume_list"=>[], "option_interface_list"=>[], "option_architecture_id"=>"1", "option_id"=>"1"}
[ INFO 2015-07-16 17:27:30 API] PUT /api/hosts/1
[DEBUG 2015-07-16 17:27:30 API] Params: {
"host" => {
"architecture_id" => "1",
"compute_attributes" => {
"volumes_attributes" => {}
},
"host_parameters_attributes" => {},
"interfaces_attributes" => {}
}
}

architecture_id is described as numeric param and hammer should send it as such.
Severity is low as API handles the conversion well


Related issues 1 (0 open1 closed)

Has duplicate Hammer CLI - Bug #11084: Send API params of type number as numbersDuplicateMartin Bacovsky07/10/2015Actions
Actions #1

Updated by Martin Bacovsky almost 9 years ago

  • translation missing: en.field_release deleted (35)
Actions #2

Updated by Martin Bacovsky almost 9 years ago

  • Has duplicate Bug #11084: Send API params of type number as numbers added
Actions #3

Updated by The Foreman Bot almost 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/hammer-cli-foreman/pull/187 added
  • Pull request deleted ()
Actions #4

Updated by Martin Bacovsky almost 9 years ago

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

Also available in: Atom PDF