BBlack has submitted this change and it was merged. Change subject: LVS: document subnets in balancer assignment ......................................................................
LVS: document subnets in balancer assignment There's no "good" place to do this with how things are split up in hieradata, but putting it here (as well as in revdns comments) should hopefully make people think twice when they commit. Bug: T143915 Change-Id: I0fcbbcfa8b2547e44b10b55c7d657202104dd567 --- M modules/role/manifests/lvs/balancer.pp 1 file changed, 9 insertions(+), 9 deletions(-) Approvals: BBlack: Looks good to me, approved jenkins-bot: Verified diff --git a/modules/role/manifests/lvs/balancer.pp b/modules/role/manifests/lvs/balancer.pp index 37666a0..f36647e 100644 --- a/modules/role/manifests/lvs/balancer.pp +++ b/modules/role/manifests/lvs/balancer.pp @@ -12,17 +12,17 @@ $lvs_balancer_ips = $::hostname ? { # eqiad - /^(lvs10(0[14]|07|10))$/ => [ + /^(lvs10(0[14]|07|10))$/ => [ # IPs must be high-traffic1 subnet $sip['text'][$::site], ], - /^(lvs10(0[25]|08|11))$/ => [ + /^(lvs10(0[25]|08|11))$/ => [ # IPs must be high-traffic2 subnet $sip['upload'][$::site], $sip['maps'][$::site], $sip['dns_rec'][$::site], $sip['misc_web'][$::site], $sip['git-ssh'][$::site], ], - /^(lvs10(0[36]|09|12))$/ => [ + /^(lvs10(0[36]|09|12))$/ => [ # IPs must be low-traffic subnet $sip['apaches'][$::site], $sip['api'][$::site], $sip['rendering'][$::site], @@ -49,16 +49,16 @@ ], # codfw (should mirror eqiad above, eventually, and become merged with it via regex - /^(lvs200[14])$/ => [ + /^(lvs200[14])$/ => [ # IPs must be high-traffic1 subnet $sip['text'][$::site], ], - /^(lvs200[25])$/ => [ + /^(lvs200[25])$/ => [ # IPs must be high-traffic2 subnet $sip['upload'][$::site], $sip['maps'][$::site], $sip['misc_web'][$::site], $sip['dns_rec'][$::site], ], - /^(lvs200[36])$/ => [ + /^(lvs200[36])$/ => [ # IPs must be low-traffic subnet $sip['apaches'][$::site], $sip['api'][$::site], $sip['rendering'][$::site], @@ -82,16 +82,16 @@ ], # esams + ulsfo - /^(lvs[34]00[13])$/ => [ + /^(lvs[34]00[13])$/ => [ # IPs must be high-traffic1 subnet $sip['text'][$::site], ], - /^(lvs300[24])$/ => [ + /^(lvs300[24])$/ => [ # IPs must be high-traffic2 subnet $sip['upload'][$::site], $sip['maps'][$::site], $sip['misc_web'][$::site], $sip['dns_rec'][$::site], ], - /^(lvs400[24])$/ => [ + /^(lvs400[24])$/ => [ # IPs must be high-traffic2 subnet $sip['upload'][$::site], $sip['maps'][$::site], $sip['misc_web'][$::site], -- To view, visit https://gerrit.wikimedia.org/r/318083 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0fcbbcfa8b2547e44b10b55c7d657202104dd567 Gerrit-PatchSet: 1 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: BBlack <bbl...@wikimedia.org> Gerrit-Reviewer: BBlack <bbl...@wikimedia.org> Gerrit-Reviewer: Ema <e...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits