diff options
| -rw-r--r-- | ironic/conductor/utils.py | 6 | ||||
| -rw-r--r-- | ironic/drivers/modules/agent.py | 4 | ||||
| -rw-r--r-- | ironic/drivers/modules/inspect_utils.py | 5 | ||||
| -rw-r--r-- | ironic/tests/unit/conductor/test_utils.py | 4 | ||||
| -rw-r--r-- | ironic/tests/unit/drivers/modules/test_inspect_utils.py | 3 |
5 files changed, 10 insertions, 12 deletions
diff --git a/ironic/conductor/utils.py b/ironic/conductor/utils.py index 59b60a5da..2a1e19f48 100644 --- a/ironic/conductor/utils.py +++ b/ironic/conductor/utils.py @@ -238,9 +238,9 @@ def _can_skip_state_change(task, new_state): notify_utils.emit_power_set_notification( task, fields.NotificationLevel.INFO, fields.NotificationStatus.END, new_state) - LOG.warning("Not going to change node %(node)s power state because " - "current state = requested state = '%(state)s'.", - {'node': node.uuid, 'state': curr_state}) + LOG.debug("Not going to change node %(node)s power state because " + "current state = requested state = '%(state)s'.", + {'node': node.uuid, 'state': curr_state}) try: curr_state = task.driver.power.get_power_state(task) diff --git a/ironic/drivers/modules/agent.py b/ironic/drivers/modules/agent.py index 776b0186d..25193a545 100644 --- a/ironic/drivers/modules/agent.py +++ b/ironic/drivers/modules/agent.py @@ -106,8 +106,8 @@ def check_image_size(task): image_disk_format = node.instance_info.get('image_disk_format') # skip check if 'memory_mb' is not defined if 'memory_mb' not in properties: - LOG.warning('Skip the image size check as memory_mb is not ' - 'defined in properties on node %s.', node.uuid) + LOG.debug('Skip the image size check as memory_mb is not ' + 'defined in properties on node %s.', node.uuid) return image_show = images.image_show(task.context, image_source) diff --git a/ironic/drivers/modules/inspect_utils.py b/ironic/drivers/modules/inspect_utils.py index 41390b63a..05aae921e 100644 --- a/ironic/drivers/modules/inspect_utils.py +++ b/ironic/drivers/modules/inspect_utils.py @@ -47,6 +47,5 @@ def create_ports_if_not_exist( LOG.info("Port created for MAC address %(address)s for node " "%(node)s", {'address': mac, 'node': node.uuid}) except exception.MACAlreadyExists: - LOG.warning("Port already exists for MAC address %(address)s " - "for node %(node)s", - {'address': mac, 'node': node.uuid}) + LOG.info("Port already exists for MAC address %(address)s " + "for node %(node)s", {'address': mac, 'node': node.uuid}) diff --git a/ironic/tests/unit/conductor/test_utils.py b/ironic/tests/unit/conductor/test_utils.py index 1e563ec71..1e567ed79 100644 --- a/ironic/tests/unit/conductor/test_utils.py +++ b/ironic/tests/unit/conductor/test_utils.py @@ -454,7 +454,7 @@ class NodePowerActionTestCase(db_base.DbTestCase): self.assertEqual(states.POWER_ON, node['power_state']) self.assertIsNone(node['target_power_state']) self.assertIsNone(node['last_error']) - log_mock.warning.assert_called_once_with( + log_mock.debug.assert_called_once_with( u"Not going to change node %(node)s power state because " u"current state = requested state = '%(state)s'.", {'state': states.POWER_ON, 'node': node.uuid}) @@ -753,7 +753,7 @@ class NodePowerActionTestCase(db_base.DbTestCase): self.assertEqual(states.POWER_ON, node['power_state']) self.assertEqual(states.NOSTATE, node['target_power_state']) self.assertIsNone(node['last_error']) - mock_log.warning.assert_called_once_with( + mock_log.debug.assert_called_once_with( u"Not going to change node %(node)s power state because " u"current state = requested state = '%(state)s'.", {'state': states.POWER_ON, 'node': node.uuid}) diff --git a/ironic/tests/unit/drivers/modules/test_inspect_utils.py b/ironic/tests/unit/drivers/modules/test_inspect_utils.py index 8b44dde5a..aa7bd79be 100644 --- a/ironic/tests/unit/drivers/modules/test_inspect_utils.py +++ b/ironic/tests/unit/drivers/modules/test_inspect_utils.py @@ -55,8 +55,7 @@ class InspectFunctionTestCase(db_base.DbTestCase): port_obj1.create.assert_called_once_with() port_obj2.create.assert_called_once_with() - @mock.patch.object(utils.LOG, 'warning', - spec_set=True, autospec=True) + @mock.patch.object(utils.LOG, 'info', spec_set=True, autospec=True) @mock.patch.object(objects.Port, 'create', spec_set=True, autospec=True) def test_create_ports_if_not_exist_mac_exception(self, create_mock, |