[08:32:04] Hi y'all! Could someone from serviceops give https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/1084031 a review please? Appreciated! [09:42:26] brouberol: o/. Janis in on PTO, I just got back, I am not sure of the availability of other members of the team yet, it might take a while [09:57:11] no rush. c.danis seemed to indicate he was on board, modulo a typo I fixed, but I don't want to assume [11:33:49] given the new feature is opt-in and doesn't impact anything else, if you get another +1 from Chris I'd say you are good to go [11:33:53] enough quorum in my opinion [14:11:20] oops [14:13:11] brouberol: +1 one optional nit [14:19:10] thanks! [14:20:03] oh I think I was unclear -- `with` works like `if`, except its arg evaluates to true, that arg becomes `.` inside the inner block [14:20:10] so it's equivalent but a bit more DRY [14:20:20] s/except its/except when its/ [14:20:47] ah, then It was my understanding of how it worked that was wrong [14:21:00] I can submit a fix patch if you want [14:21:12] I don't mind either way, I just think it's a nice idiom (which also surprised me when I first learned about it) [16:41:29] hey folks, getting back to the registry subject: https://phabricator.wikimedia.org/T375645 [16:41:51] not really urgent, and I get there is a lot of things to read, but if anybody is interested in reviewing lemme know