Muehlenhoff has submitted this change and it was merged.

Change subject: Fix quoting for br_netfilter kmod configuration
......................................................................


Fix quoting for br_netfilter kmod configuration

\n is only expanded for double-quoted strings, with single-quoted
strings we end up with a literal "\n", which makes kmod not find
the module

Change-Id: I5cb266183588c1fe8e1ed0753b5057f290b52520
---
M modules/role/manifests/labs/openstack/nova.pp
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Rush: Looks good to me, but someone else must approve
  Muehlenhoff: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/role/manifests/labs/openstack/nova.pp 
b/modules/role/manifests/labs/openstack/nova.pp
index 5772c4e..808bd73 100644
--- a/modules/role/manifests/labs/openstack/nova.pp
+++ b/modules/role/manifests/labs/openstack/nova.pp
@@ -410,7 +410,7 @@
             group   => 'root',
             mode    => '0444',
             require => File['/etc/modules-load.d/'],
-            content => 'br_netfilter\n',
+            content => "br_netfilter\n",
         }
     }
 

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5cb266183588c1fe8e1ed0753b5057f290b52520
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: Chasemp <[email protected]>
Gerrit-Reviewer: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Rush <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to