[08:44:23] hi folks :) [08:44:46] I think that https://gerrit.wikimedia.org/r/c/operations/puppet/+/909687 is really great, we should push to review/deploy it before it gets stale [08:45:51] if we leave aside the puppet implementation, is the above approach ok for everybody? [08:46:10] once we have an agreement then a smaller group can definitely complete the review [08:48:44] <_joe_> I'm gonna try to take a look on friday [08:49:29] it's a big change, I was planning to have some time to take a look at this [08:51:39] yes yes it definitely is :) [10:03:50] does running ci locally for deployment-charts works for you? [10:04:40] I am getting weird error at line 417 of Rake, related to git [10:05:09] but I suspect it is a PEBCAK [10:10:27] I get the same error [10:11:11] ok then I am not crazy :D [10:11:30] for some reason the ?include doesn't work, but I printed the array and "origin" is definitely there [10:53:31] create https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/915540 that seems to work, lemme know your thoughts [10:54:28] I didn't trace back what change triggered it though [12:09:37] <_joe_> elukey: sorry It's fixed already in a patch I made d'oh [12:10:36] <_joe_> sorry about that [13:24:17] _joe_ ah no problem nice! Going to abandon mine :) [13:24:58] <_joe_> no please let's merge it [13:25:37] ack applying your suggestion first [13:26:38] _joe_ merged :) [15:20:16] qq - I'd need to allow the mesh istio configs to handle POST requests too, since ores-legacy will have to handle them.. is there a way to do it now or should the POST support be added? [15:34:26] ah wait maybe it is uvicorn, will report bakc [15:40:54] yes silly me, all works of course :)