diff options
author | Yuriy Zveryanskyy <yzveryanskyy@mirantis.com> | 2013-09-25 18:52:40 +0300 |
---|---|---|
committer | Yuriy Zveryanskyy <yzveryanskyy@mirantis.com> | 2013-09-26 14:47:08 +0300 |
commit | 0d740b26cd0acecef457e185704a316f773e0cee (patch) | |
tree | 6a2ca56aaec204584ac5dd0ec2ceb81f66c81433 /ironic/api/controllers/v1/node.py | |
parent | 98670162c74c245cb671ca53c934f0533b3d570a (diff) | |
download | ironic-0d740b26cd0acecef457e185704a316f773e0cee.tar.gz |
Integer types support in api
This patch adds validation 'driver_info' (for nodes)
and 'extra' (nodes, chassis, ports) for accept
integers as values.
Change-Id: I930bb408ab9f3fde49034978413651ea0e7e28e9
Diffstat (limited to 'ironic/api/controllers/v1/node.py')
-rw-r--r-- | ironic/api/controllers/v1/node.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/ironic/api/controllers/v1/node.py b/ironic/api/controllers/v1/node.py index 073d37a87..306697ba3 100644 --- a/ironic/api/controllers/v1/node.py +++ b/ironic/api/controllers/v1/node.py @@ -16,6 +16,7 @@ # under the License. import jsonpatch +import six import pecan from pecan import rest @@ -195,17 +196,15 @@ class Node(base.APIBase): # NOTE: translate 'driver_info' internally to 'management_configuration' driver = wtypes.text - # FIXME(lucasagomes): it should accept at least wtypes.text or wtypes.int - # as value - driver_info = {wtypes.text: wtypes.text} + driver_info = {wtypes.text: utils.ValidTypes(wtypes.text, + six.integer_types)} - # FIXME(lucasagomes): it should accept at least wtypes.text or wtypes.int - # as value - extra = {wtypes.text: wtypes.text} + extra = {wtypes.text: utils.ValidTypes(wtypes.text, six.integer_types)} # NOTE: properties should use a class to enforce required properties # current list: arch, cpus, disk, ram, image - properties = {wtypes.text: wtypes.text} + properties = {wtypes.text: utils.ValidTypes(wtypes.text, + six.integer_types)} # NOTE: translate 'chassis_id' to a link to the chassis resource # and accept a chassis uuid when creating a node. |