Giuseppe Lavagetto has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/377436 )

Change subject: swift: use !~ instead of ! $title =~ /.../
......................................................................


swift: use !~ instead of ! $title =~ /.../

Current form breaks with the future parser, probably due to operator
precedence.

Change-Id: I14e7f0db85c0f447b74301928d440795963f920c
---
M modules/swift/manifests/init_device.pp
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Giuseppe Lavagetto: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/swift/manifests/init_device.pp 
b/modules/swift/manifests/init_device.pp
index 5d74f1f..2f1d72c 100644
--- a/modules/swift/manifests/init_device.pp
+++ b/modules/swift/manifests/init_device.pp
@@ -1,5 +1,5 @@
 define swift::init_device($partition_nr='1') {
-    if (! $title =~ /^\/dev\/([hvs]d[a-z]+|md[0-9]+)$/) {
+    if ($title !~ /^\/dev\/([hvs]d[a-z]+|md[0-9]+)$/) {
         fail("unable to init ${title} for swift")
     }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I14e7f0db85c0f447b74301928d440795963f920c
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon Liambotis <fai...@wikimedia.org>
Gerrit-Reviewer: Giuseppe Lavagetto <glavage...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to