diff options
author | Zuul <zuul@review.opendev.org> | 2023-02-20 15:27:30 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-02-20 15:27:30 +0000 |
commit | 61f841d20178992b4548480a71fbae47e006de31 (patch) | |
tree | 342c9ca0049dddf6dce98c83c343f9e00dac8c64 | |
parent | 3707422bf70af2926744f669b6447da8f2ec2d67 (diff) | |
parent | b1643368beeb8c53572339e6a72e45e951abc2d4 (diff) | |
download | ironic-61f841d20178992b4548480a71fbae47e006de31.tar.gz |
Merge "Make metrics names a little more consistent"
-rw-r--r-- | ironic/drivers/modules/agent_base.py | 6 | ||||
-rw-r--r-- | releasenotes/notes/change-c9c01700dcfd599b.yaml | 9 |
2 files changed, 12 insertions, 3 deletions
diff --git a/ironic/drivers/modules/agent_base.py b/ironic/drivers/modules/agent_base.py index ff2a454ea..7732840d5 100644 --- a/ironic/drivers/modules/agent_base.py +++ b/ironic/drivers/modules/agent_base.py @@ -100,7 +100,7 @@ _FASTTRACK_HEARTBEAT_ALLOWED = (states.DEPLOYWAIT, states.CLEANWAIT, FASTTRACK_HEARTBEAT_ALLOWED = frozenset(_FASTTRACK_HEARTBEAT_ALLOWED) -@METRICS.timer('post_clean_step_hook') +@METRICS.timer('AgentBase.post_clean_step_hook') def post_clean_step_hook(interface, step): """Decorator method for adding a post clean step hook. @@ -128,7 +128,7 @@ def post_clean_step_hook(interface, step): return decorator -@METRICS.timer('post_deploy_step_hook') +@METRICS.timer('AgentBase.post_deploy_step_hook') def post_deploy_step_hook(interface, step): """Decorator method for adding a post deploy step hook. @@ -279,7 +279,7 @@ def _get_completed_command(task, commands, step_type): return last_command -@METRICS.timer('log_and_raise_deployment_error') +@METRICS.timer('AgentBase.log_and_raise_deployment_error') def log_and_raise_deployment_error(task, msg, collect_logs=True, exc=None): """Helper method to log the error and raise exception. diff --git a/releasenotes/notes/change-c9c01700dcfd599b.yaml b/releasenotes/notes/change-c9c01700dcfd599b.yaml new file mode 100644 index 000000000..dbb3bb11e --- /dev/null +++ b/releasenotes/notes/change-c9c01700dcfd599b.yaml @@ -0,0 +1,9 @@ +--- +upgrade: + - | + Two statsd metrics names have been modified to provide structural clarity + and consistency for consumers of statistics metrics. Consumers of metrics + statistics may need to update their dashboards as the + ``post_clean_step_hook`` metric is now named + ``AgentBase.post_clean_step_hook``, and the ``post_deploy_step_hook`` is + now named ``AgentBase.post_deploy_step_hook``. |