From 9ea519cf58bd631b4445f19ead2b4c0586cc38bb Mon Sep 17 00:00:00 2001 From: Lucas Alvares Gomes Date: Wed, 11 Mar 2015 16:07:12 +0000 Subject: [PATCH] Add node UUID to deprecated log message That makes easier to operators to find out which node is actually using the deprecated parameter. Change-Id: Ie8fcc9339747717a906affb48611482538b09eb7 --- ironic/drivers/modules/pxe.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/ironic/drivers/modules/pxe.py b/ironic/drivers/modules/pxe.py index 1b5787a28c..12bb347699 100644 --- a/ironic/drivers/modules/pxe.py +++ b/ironic/drivers/modules/pxe.py @@ -131,7 +131,8 @@ def _parse_driver_info(node): # NOTE(lucasagomes): For backwards compatibility let's keep accepting # pxe_deploy_{kernel, ramdisk}, should be removed in Liberty. deprecated_msg = _LW('The "%(old_param)s" parameter is deprecated. ' - 'Please use "%(new_param)s" instead.') + 'Please update the node %(node)s to use ' + '"%(new_param)s" instead.') for parameter in ('deploy_kernel', 'deploy_ramdisk'): value = info.get(parameter) @@ -140,7 +141,8 @@ def _parse_driver_info(node): value = info.get(old_parameter) if value: LOG.warning(deprecated_msg, {'old_param': old_parameter, - 'new_param': parameter}) + 'new_param': parameter, + 'node': node.uuid}) d_info[parameter] = value error_msg = _("Cannot validate PXE bootloader. Some parameters were"