[09:34:50] 10netops, 10Infrastructure-Foundations, 10SRE: Migrate row E/F network aggregation to dedicated Spine switches - https://phabricator.wikimedia.org/T322937 (10cmooney) Just to update in terms of the LVS connections. After discussing with Brandon I thought it best if the links from all 4 LVS terminate on diff... [11:57:18] 10netops, 10Infrastructure-Foundations, 10SRE, 10Patch-For-Review, 10cloud-services-team (Kanban): Move WMCS servers to 1 NIC - https://phabricator.wikimedia.org/T319184 (10aborrero) [12:26:03] 10netops, 10Infrastructure-Foundations, 10SRE, 10Patch-For-Review, 10cloud-services-team (Kanban): Move WMCS servers to 1 NIC - https://phabricator.wikimedia.org/T319184 (10aborrero) [15:59:04] 10Traffic, 10SRE, 10Patch-For-Review: Upgrade Traffic hosts to bullseye - https://phabricator.wikimedia.org/T321309 (10ssingh) [17:06:17] 10Traffic, 10SRE, 10Patch-For-Review: ATS should alert if the number of total or active connections reached maximum - https://phabricator.wikimedia.org/T292815 (10BCornwall) @Vgutierrez, while this doesn't have strict support for multiple ATS instances, bblack suggested that by simplifying all this it would... [17:20:03] Hey :) The grace period for apple-search decom is ending tomorrow, hence the CR batch v.guttierez b.black. It doesn´t necessarily need to be done tomorrow, even if I´d like to, but I got a bit behind on creating the patches, and will probably need some level of hand holding at the time of deployment [17:20:54] ^^ Relevant task https://phabricator.wikimedia.org/T316296 [17:21:06] that typo just hurts [17:21:25] Oh I'm so sorry [17:21:30] ;P [17:21:38] I didn't tab complete and add the . and went full on bad spanish [17:21:44] <3 [17:22:03] you can fix it with some Belgian beer [17:22:07] I'd say French.. but you know [17:22:34] ;P [17:22:35] * claime takes the deserved humiliation stoically [17:22:43] :P [17:23:12] On the plus side, I'm sure it won't ping you if I write your name like this. [17:23:21] that's right [17:23:36] but you flooded my inbox with gerrit mails and wrote on -traffic [17:23:41] so that did it ;P [17:23:52] I'll take a look [17:24:29] I really didn't want to cause a context switch, it was some explanatory context that probably should have been in the task [17:30:48] claime: BTW, did you miss hieradata/common/profile/kubernetes/deployment_server.yaml :? [17:31:04] or that's on a CR that I'm not seeing? [17:31:50] vgutierrez: in what key? if it's profile::kubernetes::deployment_server::services that's afterwards [17:32:42] https://wikitech.wikimedia.org/wiki/Kubernetes/Remove_a_service [17:32:50] (Wrote the doc when I rename mwdebug) [17:34:30] yup [17:34:36] vgutierrez: DNS should be merged after switching to lvs_setup? I followed https://wikitech.wikimedia.org/wiki/LVS#Remove_a_load_balanced_service [17:35:23] so.. IIRC if you remove the DNS record before downgrading from production to lvs_setup that would trigger a puppet failure on lvs boxes [17:37:48] culprit being wmflib/functions/service/get_services_for.pp [17:38:34] Ok, adding dependency :) [17:39:32] hmmm or not [17:44:24] it doesn't seem to do lookups [17:47:48] nah, the dependency is with the discovery stanza on service.yaml [17:47:48] my fault [17:48:14] np, I'll remove the dependency then