From 13088d262b01cc7a2331599875d1e0a622220a20 Mon Sep 17 00:00:00 2001 From: Al Bailey Date: Thu, 8 Nov 2018 09:40:32 -0600 Subject: [PATCH] Fix pylint issue exposed by yamllint 1.12.1 When yamllint 1.12.1 was released Oct 17, 2018 it was pickier about the str.strip syntax. This change fixes pylint. Verified that strip('/') and strip('//') work the same Change-Id: I82864134289cdabb642876bc2c867d0a35428b96 Signed-off-by: Al Bailey Closes-Bug: 1802329 --- .../controllerconfig/controllerconfig/upgrades/controller.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/controllerconfig/controllerconfig/controllerconfig/upgrades/controller.py b/controllerconfig/controllerconfig/controllerconfig/upgrades/controller.py index bc10ee6329..20343511ed 100644 --- a/controllerconfig/controllerconfig/controllerconfig/upgrades/controller.py +++ b/controllerconfig/controllerconfig/controllerconfig/upgrades/controller.py @@ -1402,7 +1402,7 @@ def upgrade_controller_simplex(backup_file): 'controller-0') hieradata_tmpdir = os.path.join(staging_dir, - constants.HIERADATA_PERMDIR.strip('//')) + constants.HIERADATA_PERMDIR.strip('/')) print_log_info("Applying Bootstrap manifest...") cutils.apply_manifest(controller_0_address, sysinv_constants.CONTROLLER,