Alexandros Kosiaris has submitted this change and it was merged.
Change subject: Add a comment about the lvs_grain's information purposes
......................................................................
Add a comment about the lvs_grain's information purposes
The newly added lvs_grain, while great for operational purposes, is not
a knob but only an indicator. point that out in a comment
Change-Id: If11281d5eb7b6223ab2d8f9e5f0cabd29ee4732c
---
M modules/lvs/manifests/configuration.pp
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Alexandros Kosiaris: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/lvs/manifests/configuration.pp
b/modules/lvs/manifests/configuration.pp
index facbcbe..982c683 100644
--- a/modules/lvs/manifests/configuration.pp
+++ b/modules/lvs/manifests/configuration.pp
@@ -51,6 +51,8 @@
$v6_ip = '::'
}
+ # NOTE: This is for informational purposes only. The actual configuration
+ # that decides primary/secondary is done at the BGP level on the routers.
$lvs_grain = $::hostname ? {
/^lvs100[123789]$/ => 'primary',
/^lvs200[123]$/ => 'primary',
--
To view, visit https://gerrit.wikimedia.org/r/282104
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If11281d5eb7b6223ab2d8f9e5f0cabd29ee4732c
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: BBlack <[email protected]>
Gerrit-Reviewer: Ema <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
