diff options
author | Zuul <zuul@review.opendev.org> | 2019-10-07 23:17:03 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2019-10-07 23:17:03 +0000 |
commit | 656695c51f8380a7a64cc756d90ef884973d72a9 (patch) | |
tree | 6ff86822e697dd03768014fe8e0540a15ae89ab4 | |
parent | ac7f92ad57768648f007330f0011698f76d9aea6 (diff) | |
parent | 04df2635ba5f6ef2dda4676bef7b5694cb1ff2f9 (diff) | |
download | ironic-656695c51f8380a7a64cc756d90ef884973d72a9.tar.gz |
Merge "Change log level based on node status" into stable/train
-rw-r--r-- | ironic/conductor/task_manager.py | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/ironic/conductor/task_manager.py b/ironic/conductor/task_manager.py index 804d8e4a2..b99cd1084 100644 --- a/ironic/conductor/task_manager.py +++ b/ironic/conductor/task_manager.py @@ -486,12 +486,20 @@ class TaskManager(object): # publish the state transition by saving the Node self.node.save() - LOG.info('Node %(node)s moved to provision state "%(state)s" from ' - 'state "%(previous)s"; target provision state is ' - '"%(target)s"', - {'node': self.node.uuid, 'state': self.node.provision_state, - 'target': self.node.target_provision_state, - 'previous': self._prev_provision_state}) + + log_message = ('Node %(node)s moved to provision state "%(state)s" ' + 'from state "%(previous)s"; target provision state is ' + '"%(target)s"' % + {'node': self.node.uuid, + 'state': self.node.provision_state, + 'target': self.node.target_provision_state, + 'previous': self._prev_provision_state}) + + if (self.node.provision_state.endswith('failed') or + self.node.provision_state == 'error'): + LOG.error(log_message) + else: + LOG.info(log_message) if callback is None: self._notify_provision_state_change() |