Can't set non-string values via CLI

Bug #1403491 reported by Lucas Alvares Gomes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ironicclient
Fix Released
Medium
Lucas Alvares Gomes

Bug Description

Some fields in Ironic support multi-type values, for example the "extra" attibute of the resources supports strings and ints in the moment, but via the CLI it's impossible to set non-string values for those fields, see:

stack@vm:~/python-ironicclient$ ironic node-update 8eeb8e5a-936a-49c2-a0d6-11de550a3f5c add extra/int=10
+------------------------+------------------------------------------------------------------+
| Property | Value |
+------------------------+------------------------------------------------------------------+
...
| extra | {u'int': u'10'} |
...
+------------------------+------------------------------------------------------------------+

Changed in python-ironicclient:
assignee: nobody → Lucas Alvares Gomes (lucasagomes)
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-ironicclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/142492

Changed in python-ironicclient:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-ironicclient (master)

Reviewed: https://review.openstack.org/142492
Committed: https://git.openstack.org/cgit/openstack/python-ironicclient/commit/?id=8910d45955073dd45c59c972b44a78b18c225f33
Submitter: Jenkins
Branch: master

commit 8910d45955073dd45c59c972b44a78b18c225f33
Author: Lucas Alvares Gomes <email address hidden>
Date: Wed Dec 17 16:35:10 2014 +0000

    Support setting non-string fields

    Some attributes in Ironic support multiple types of data, before the
    CLI was treating everything as a string this patch is changing that by
    making the CLI to JSON deserialize the values before sending it to Ironic.

    Change-Id: I7d53b31ad401bc7c36841e0ece28e818cdc8976d
    Closes-Bug: #1403491

Changed in python-ironicclient:
status: In Progress → Fix Committed
Changed in python-ironicclient:
milestone: none → 0.7.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.