Elukey has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/383351 )

Change subject: wmf_auto_reimage_host: fix check in mgmt interfaces
......................................................................

wmf_auto_reimage_host: fix check in mgmt interfaces

Change-Id: Ib57843148adb87a4ecd43abdcd11060017d52275
---
M modules/profile/files/cumin/wmf_auto_reimage_host.py
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/51/383351/1

diff --git a/modules/profile/files/cumin/wmf_auto_reimage_host.py 
b/modules/profile/files/cumin/wmf_auto_reimage_host.py
index 3335bd7..1678f3c 100644
--- a/modules/profile/files/cumin/wmf_auto_reimage_host.py
+++ b/modules/profile/files/cumin/wmf_auto_reimage_host.py
@@ -62,8 +62,9 @@
                 raise argparse.ArgumentTypeError(
                     "Unable to resolve host '{name}'".format(name=name))
 
-    if '.mgmt.' not in mgmts.values():
-        raise argparse.ArgumentTypeError('The MGMT parameter must be in the 
*.mgmt.* format')
+    for mgmt in mgmts.values():
+        if '.mgmt.' not in mgmt:
+            raise argparse.ArgumentTypeError('The MGMT parameter must be in 
the *.mgmt.* format')
 
     return args
 

-- 
To view, visit https://gerrit.wikimedia.org/r/383351
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib57843148adb87a4ecd43abdcd11060017d52275
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Elukey <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to