Giuseppe Lavagetto has submitted this change and it was merged.
Change subject: confd::file: correct the regsubst arguments
......................................................................
confd::file: correct the regsubst arguments
Change-Id: I1f0ddebaefabf0ec4268e7778b7aefac667104b5
---
M modules/confd/manifests/file.pp
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Giuseppe Lavagetto: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/confd/manifests/file.pp b/modules/confd/manifests/file.pp
index 94d8090..c4ff2d7 100644
--- a/modules/confd/manifests/file.pp
+++ b/modules/confd/manifests/file.pp
@@ -13,7 +13,7 @@
$source = undef,
$content = undef,
) {
- $safe_name = regsubst('\/', '_', $title)
+ $safe_name = regsubst($title, '\/', '_')
unless ($source or $content) {
fail('We either need a source file or a content for the config file')
--
To view, visit https://gerrit.wikimedia.org/r/218293
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1f0ddebaefabf0ec4268e7778b7aefac667104b5
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: Giuseppe Lavagetto <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
