Description of problem: Before Update: hammer -v -u admin -p changeme smart-variable info --id="100" Id: 100 Name: YJIQOcdDFQ1 Default Value: Type: string Puppet class: ntp Description: Hidden Value?: false Validator: Type: Rule: Override values: Merge overrides: no Merge default value: no Avoid duplicates: no Order: fqdn, hostgroup, os, domain Values: Created at: 2016/08/15 10:36:30 Updated at: 2016/08/15 10:42:18 Updating entity: hammer -v -u admin -p changeme --output=csv smart-variable update --id="100" --new-name="XO4txkfDEUD" --puppet-class="ntp" Message,Id Smart variable [YJIQOcdDFQ1] updated,100 Nothing happened and entity has old name: hammer -v -u admin -p changeme smart-variable info --id="100" Id: 100 Name: YJIQOcdDFQ1 Default Value: Type: string Puppet class: ntp Description: Hidden Value?: false Validator: Type: Rule: Override values: Merge overrides: no Merge default value: no Avoid duplicates: no Order: fqdn, hostgroup, os, domain Values: Created at: 2016/08/15 10:36:30 Updated at: 2016/08/15 10:42:36 Version-Release number of selected component (if applicable): Satellite 6.2.0-6.2.1 SNAP1.2 How reproducible: Always Steps to Reproduce: 1. Create smart variable for any puppet class 2. Try to update it with new value for its name 3. Actual results: Entity is not updated Expected results: Entity updated properly Additional info:
Created redmine issue http://projects.theforeman.org/issues/16119 from this bug
Upstream bug component is Hammer
As a temporary workaround, it's possible to use --variable parameter in hammer instead of --new-name. We are investigating further how can we fix it.
--variable is a more reference to create procedure and anyway such option does not present in help, so no chances that someone find out that
Upstream bug assigned to sshtein
Moving to POST since upstream bug http://projects.theforeman.org/issues/16119 has been closed
Verified on latest SNAP of 6.3. Gonna create separate issue for 6.2.z For 6.3: hammer -v -u admin -p changeme --output=csv smart-variable create --variable="czE8Mg5Ym8" --puppet-class="cli_test_variables" Message,Id Smart variable [czE8Mg5Ym8] created,21 hammer -v -u admin -p changeme --output=json smart-variable info --id="21" { "Id": 21, "Variable": "czE8Mg5Ym8", "Default Value": null, "Type": "string", "Puppet class": "cli_test_variables", "Description": null, "Hidden Value?": false, "Validator": { "Type": null, "Rule": null }, ... } hammer -v -u admin -p changeme --output=csv smart-variable update --puppet-class="cli_test_variables" --id="21" --new-variable="c3BisQxlnJViCX4" Message,Id Smart variable [c3BisQxlnJViCX4] updated,21 hammer -v -u admin -p changeme --output=json smart-variable info --id="21" { "Id": 21, "Variable": "c3BisQxlnJViCX4", "Default Value": null, "Type": "string", "Puppet class": "cli_test_variables", "Description": null, "Hidden Value?": false, "Validator": { "Type": null, "Rule": null }, ... "Created at": "2017-01-11 09:55:24 UTC", "Updated at": "2017-01-11 09:55:31 UTC" }
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. > > For information on the advisory, and where to find the updated files, follow the link below. > > If the solution does not work for you, open a new bug report. > > https://access.redhat.com/errata/RHSA-2018:0336