diff --git a/ironic/api/controllers/v1/node.py b/ironic/api/controllers/v1/node.py index 3e13f723b1..78e7778446 100644 --- a/ironic/api/controllers/v1/node.py +++ b/ironic/api/controllers/v1/node.py @@ -340,7 +340,7 @@ class Node(base.APIBase): @classmethod def _convert_with_links(cls, node, url, expand=True): if not expand: - except_list = ['instance_uuid', 'power_state', + except_list = ['instance_uuid', 'maintenance', 'power_state', 'provision_state', 'uuid'] node.unset_fields_except(except_list) else: diff --git a/ironic/tests/api/v1/test_nodes.py b/ironic/tests/api/v1/test_nodes.py index 6aeafae3bd..33e6ad187f 100644 --- a/ironic/tests/api/v1/test_nodes.py +++ b/ironic/tests/api/v1/test_nodes.py @@ -82,6 +82,11 @@ class TestListNodes(base.FunctionalTest): ndict = dbutils.get_test_node() node = self.dbapi.create_node(ndict) data = self.get_json('/nodes') + self.assertIn('instance_uuid', data['nodes'][0]) + self.assertIn('maintenance', data['nodes'][0]) + self.assertIn('power_state', data['nodes'][0]) + self.assertIn('provision_state', data['nodes'][0]) + self.assertIn('uuid', data['nodes'][0]) self.assertEqual(node['uuid'], data['nodes'][0]["uuid"]) self.assertNotIn('driver', data['nodes'][0]) self.assertNotIn('driver_info', data['nodes'][0]) @@ -89,7 +94,6 @@ class TestListNodes(base.FunctionalTest): self.assertNotIn('properties', data['nodes'][0]) self.assertNotIn('chassis_uuid', data['nodes'][0]) self.assertNotIn('reservation', data['nodes'][0]) - self.assertNotIn('maintenance', data['nodes'][0]) self.assertNotIn('console_enabled', data['nodes'][0]) self.assertNotIn('target_power_state', data['nodes'][0]) self.assertNotIn('target_provision_state', data['nodes'][0])