From 1afaaa18c79124558a7ea5f1ba09c3401fdae62c Mon Sep 17 00:00:00 2001 From: Hironori Shiina Date: Tue, 24 Jan 2017 03:53:32 +0900 Subject: [PATCH] Fix node-inject-nmi to pass an empty body Fix a request of inject_nmi API to pass an empty body because PUT request must be a JSON dict. A request without a body causes a validation error in the server. Change-Id: Iaaa9e88b4a4b1e1086217900480fb528bfae56dc Partial-Bug: #1526226 --- ironicclient/tests/unit/v1/test_node.py | 2 +- ironicclient/v1/node.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ironicclient/tests/unit/v1/test_node.py b/ironicclient/tests/unit/v1/test_node.py index 7b8184d..352f11a 100644 --- a/ironicclient/tests/unit/v1/test_node.py +++ b/ironicclient/tests/unit/v1/test_node.py @@ -1221,7 +1221,7 @@ class NodeManagerTest(testtools.TestCase): self.mgr.inject_nmi(NODE1['uuid']) expect = [ ('PUT', '/v1/nodes/%s/management/inject_nmi' % NODE1['uuid'], - {}, None), + {}, {}), ] self.assertEqual(expect, self.api.calls) diff --git a/ironicclient/v1/node.py b/ironicclient/v1/node.py index 5cf283d..8213f18 100644 --- a/ironicclient/v1/node.py +++ b/ironicclient/v1/node.py @@ -425,7 +425,7 @@ class NodeManager(base.CreateManager): def inject_nmi(self, node_uuid): path = "%s/management/inject_nmi" % node_uuid - return self.update(path, None, http_method='PUT') + return self.update(path, {}, http_method='PUT') def get_supported_boot_devices(self, node_uuid): path = "%s/management/boot_device/supported" % node_uuid