Bug #20725
closed
'hammer host update' removes existing host parameters
Added by Anthony Chevalet over 7 years ago.
Updated about 6 years ago.
Description
# rpm -q foreman
foreman-1.15.3-1.el7.noarch
# hammer --version
hammer (0.10.2)
* hammer_cli_foreman (0.10.2)
* hammer_cli_foreman_bootdisk (0.1.3)
* hammer_cli_foreman_docker (unknown version)
* hammer_cli_foreman_remote_execution (unknown version)
* hammer_cli_foreman_tasks (unknown version)
* hammer_cli_import (0.11.3)
* hammer_cli_katello (0.10.0)
# hammer host info --name client1.local | grep param
# hammer host update --name client1.local --parameters param1=value1,param2=value2
Host updated
# hammer host info --name client1.local | grep param param1 => value1
param2 => value2
# hammer host update --name client1.local --parameters param3=value3
Host updated
# hammer host info --name client1.local | grep param
param3 => value3
# hammer host update --name client1.local
Host updated
# hammer host info --name client1.local | grep param
#
- Project changed from Foreman to Hammer CLI
The requested change would mean backward-incompatible API change, that I don't think we're going to make.
Doesn't hammer host set-parameter
and hammer host delete-parameter
address your use-case?
- Category set to Hosts
- Difficulty set to trivial
Ivan, I don't think {set,delete}-parameter would help here. The issue is hammer clears the parameters when option --parameters isn't used in the update command:
> hammer host update --id 7 --parameters param1=value1,param2=value2
Host updated
> hammer host info --id 7
...
Parameters:
param1 => value1
param2 => value2
All parameters:
param1 => value1
param2 => value2
testname => testvalue
diskinfo2 => test
...
> hammer host update --id 7
Host updated
> hammer host info --id 7
...
Parameters:
All parameters:
testname => testvalue
diskinfo2 => test
...
From the log:
[ INFO 2017-08-25T13:40:03 API] PUT /api/hosts/7
[DEBUG 2017-08-25T13:40:03 API] Params: {
"host" => {
"compute_attributes" => {
"volumes_attributes" => {}
},
"content_facet_attributes" => {},
"subscription_facet_attributes" => {},
"host_parameters_attributes" => [],
"interfaces_attributes" => []
}
}
This is a hammer issue. API works fine.
- Bugzilla link set to 1486696
- Assignee set to Rahul Bajaj
- Target version set to 220
- Target version changed from 220 to 230
- Bugzilla link changed from 1486696 to 1554774
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/368 added
- Status changed from Ready For Testing to Closed
- Has duplicate Bug #26926: hammer cli wipes host parameters when build is toggled (UI or API does not) added
Also available in: Atom
PDF