Faidon Liambotis has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/338364 )
Change subject: Reorder check for timesyncd or ntpd
......................................................................
Reorder check for timesyncd or ntpd
More compact/readable, as suggested by Faidon in 337009 (PS5)
Change-Id: Iab17bf50c364d82d9e344b079bd9e2dd77e166fd
---
M modules/standard/manifests/init.pp
1 file changed, 3 insertions(+), 7 deletions(-)
Approvals:
Faidon Liambotis: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/standard/manifests/init.pp
b/modules/standard/manifests/init.pp
index cc06107..d6c3a2d 100644
--- a/modules/standard/manifests/init.pp
+++ b/modules/standard/manifests/init.pp
@@ -13,14 +13,10 @@
include ::profile::cumin::target
}
- if hiera('use_timesyncd', false) or (os_version('debian >= stretch')) {
- unless $::fqdn in $::standard::ntp::wmf_peers[$::site] {
+ unless $::fqdn in $::standard::ntp::wmf_peers[$::site] {
+ if hiera('use_timesyncd', false) or (os_version('debian >= stretch')) {
include ::standard::ntp::timesyncd
- }
- }
- else
- {
- unless $::fqdn in $::standard::ntp::wmf_peers[$::site] {
+ } else {
include ::standard::ntp::client
}
}
--
To view, visit https://gerrit.wikimedia.org/r/338364
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iab17bf50c364d82d9e344b079bd9e2dd77e166fd
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Muehlenhoff <[email protected]>
Gerrit-Reviewer: Faidon Liambotis <[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
