Jcrespo has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/363360 )
Change subject: mariadb: Correct systemd unit path
......................................................................
mariadb: Correct systemd unit path
The right place is /lib, not /etc- although surprisingly, it still
works.
Bug: T169514
Change-Id: I4ea990b69a37552972e4a3cd535246e1311ea51b
---
M modules/mariadb/manifests/service.pp
1 file changed, 3 insertions(+), 2 deletions(-)
Approvals:
jenkins-bot: Verified
Jcrespo: Looks good to me, approved
diff --git a/modules/mariadb/manifests/service.pp
b/modules/mariadb/manifests/service.pp
index 2ad529c..fc2f4af 100644
--- a/modules/mariadb/manifests/service.pp
+++ b/modules/mariadb/manifests/service.pp
@@ -39,8 +39,9 @@
service { $vendor:
ensure => $ensure,
enable => $enable,
- # shouldn't systemd unit be on /lib/systemd/system ?
- require => File["/etc/systemd/system/${vendor}.service"],
+ # $manage assumes only the main instance is managed-
+ # multiple instances have to be managed manually
+ require => File["/lib/systemd/system/${vendor}.service"],
}
}
# handle per-host special configuration
--
To view, visit https://gerrit.wikimedia.org/r/363360
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4ea990b69a37552972e4a3cd535246e1311ea51b
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Jcrespo <[email protected]>
Gerrit-Reviewer: Jcrespo <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
