Project

General

Profile

Bug #11137

Hammer sends values for numeric params as strings

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

Status:
Closed
Priority:
Low
Assignee:
Martin Bacovsky
Category:
Foreman commands (obsolete)
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/hammer-cli-foreman/pull/187
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:
Red Hat JIRA:

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

Has duplicate Hammer CLI - Bug #11084: Send API params of type number as numbersDuplicate2015-07-10

Associated revisions

Revision a90b7e79 (diff)
Added by Martin Bacovsky over 7 years ago

Refs #11137 - Add normalizer converting id parameter values to numbers

Revision 553be54e (diff)
Added by Martin Bacovsky over 7 years ago

Fixes #11137 - Add normalizer converting id parameter values to numbers

Revision b491ce7d
Added by Tomas Strachota over 7 years ago

Merge pull request #187 from mbacovsky/11137_id_normalizer

Fixes #11137 - Add normalizer converting id parameter values to numbers

History

#1 Updated by Martin Bacovsky over 7 years ago

  • Legacy Backlogs Release (now unused) deleted (35)

#2 Updated by Martin Bacovsky over 7 years ago

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

#3 Updated by The Foreman Bot over 7 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 ()

#4 Updated by Martin Bacovsky over 7 years ago

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

Also available in: Atom PDF