[09:00:55] 06Traffic, 06DC-Ops, 10ops-eqiad, 06SRE, 13Patch-For-Review: Q3:test NIC for lvs1017 - https://phabricator.wikimedia.org/T387145#10886841 (10cmooney) > Check with @cmooney for changes required to hieradata/common/lvs/interfaces.yaml (to add lvs1016 there) and also to hieradata/role/eqiad/lvs/balancer.yam... [09:20:39] o/ is this morning quiet enough for a trafficserver change? [09:21:17] AFAIK yes [09:21:23] unless you plan to set it on fire [09:21:30] then it's not quiet or cold enough [09:23:56] I suppose that depends on the actual definition of "on fire", "slightly smoking" or "some burn marks" seems tolerable [09:24:16] just for reference it's this https://gerrit.wikimedia.org/r/c/operations/puppet/+/1149624 - I'll disable puppet and roll out to a single host as usual [09:24:39] hnowlan: sounds pretty safe from ATS PoV [09:27:20] cool [09:36:07] change looks okay, reenabling puppet [09:41:39] 👍 [11:52:09] FIRING: LVSHighRX: Excessive RX traffic on lvs2013:9100 (eno12399np0) - https://bit.ly/wmf-lvsrx - https://grafana.wikimedia.org/d/000000377/host-overview?orgId=1&var-server=lvs2013 - https://alerts.wikimedia.org/?q=alertname%3DLVSHighRX [11:57:09] RESOLVED: LVSHighRX: Excessive RX traffic on lvs2013:9100 (eno12399np0) - https://bit.ly/wmf-lvsrx - https://grafana.wikimedia.org/d/000000377/host-overview?orgId=1&var-server=lvs2013 - https://alerts.wikimedia.org/?q=alertname%3DLVSHighRX [12:04:47] Hii, is it okay if I merge this patch in ATS that's beta cluster only? https://gerrit.wikimedia.org/r/c/operations/puppet/+/1153632 [13:03:50] Amir1: why are you adding that kind of logic in ATS? [13:03:58] that usually belongs to varnish land [13:04:54] vgutierrez: I honestly don't know but the same redirect is applied in ATS in production: https://gerrit.wikimedia.org/r/c/operations/puppet/+/1153632/1/modules/profile/files/trafficserver/rb-mw-mangling.lua [13:05:06] I wanted to mirror production [13:13:47] Amir1: yeah.. let's get it merged [13:14:23] Thanks! [13:14:28] I apply your change now [13:23:14] 06Traffic, 10Ganeti, 06SRE: Decommission doh7001 and durum7001 - https://phabricator.wikimedia.org/T396015#10887649 (10MoritzMuehlenhoff) >>! In T396015#10884209, @ssingh wrote: > @Muehlenhoff: Both of these are decommissioned. Let me know if any other action is required from my end, thanks! Thanks! When th... [13:37:44] modules/profile/manifests/trafficserver/backend.pp:161 WARNING selector inside resource block (selector_inside_resource) [13:37:47] le sigh [13:37:57] https://stackoverflow.com/questions/38729228/inline-if-statement-for-puppet-parameters [13:39:06] Amir1: try `ensure => stdlib::ensure($::realm == 'labs')` [13:39:19] ah thanks [13:40:02] TIL [13:55:05] https://puppet-compiler.wmflabs.org/output/1153632/6647/cp1114.eqiad.wmnet/index.html [13:55:27] Resources only in the new catalog: [13:55:27] > Trafficserver::Lua_script[rb-mw-mangling-beta] [13:55:42] I wonder whether it's just absent or because pcc is labs [13:56:27] check the catalog [13:56:33] you'll see that's ensure => absent [13:57:12] wow [13:57:15] thanks [14:19:33] 06Traffic, 10Liberica, 13Patch-For-Review: Test katran forwarding plane on lvs1013 - https://phabricator.wikimedia.org/T395228#10887820 (10ops-monitoring-bot) Icinga downtime and Alertmanager silence (ID=8a04aff9-1dff-495f-85e9-18f315f11e24) set by vgutierrez@cumin1002 for 1 day, 0:00:00 on 1 host(s) and the...