[07:50:53] Hi y'all! FYI, I've found a bug in modules/app/job_1.0.0.tpl when investigated a weird CI linting error in a WIP chart. The fix is provided here https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/1021843 and all details are laid out in T362954 [08:04:59] brouberol: theres a conflicting change that looks similar, mind to double check and maybe abandon/extend? https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/969802/2 [08:05:20] sure, I missed that [08:09:37] done [08:37:27] thanks. I've asked effie to take a look - IIRC she did some work in that area recently [08:38:41] on a general note: It usually helps review to add a CR that just creates a copy of the module to be changed (with the new version number) and then make changes in a subsequent CR [08:40:08] oh right, that's a good idea