[07:42:09] <brouberol>	 elukey: It seems that your solution works, even with somewhat convoluted black sabbathic helm invocations we have in the `airflow` chart!
[07:43:07] <elukey>	 brouberol: o/ I was about to send a patch :)
[07:43:19] <elukey>	 IIUC removing tpl() should be enough, what do you think?
[07:43:27] <elukey>	 if so I have base:helper 1.1.4 ready
[07:43:48] <elukey>	 or did you test tpl( etc...?)
[07:44:07] <elukey>	 I am not 100% sure what's best between include and tpl to be honest
[07:44:09] <brouberol>	 I tested rendering with `{{- $renderedResources := tpl (print .Root.Template.BasePath .resourceFilePath) .Root }}` instead of `{{- $renderedResources := tpl (include (print .Root.Template.BasePath .resourceFilePath) .Root) .Root }}`
[07:44:15] <brouberol>	 aka, drop the `include`
[07:44:58] <brouberol>	 let me experiment with `include` without `tpl`, but I have a sense that  it'll fail the airflow rendering
[07:46:21] <brouberol>	 well, I'll be damned
[07:46:25] <brouberol>	 it works perfectly
[07:47:08] <brouberol>	 heh, 🤷 that's good monday morning news
[07:50:36] <elukey>	 I tried to look for differences between tpl() and include() but I couldn't find anything substancial
[07:50:58] <elukey>	 only that include is part of helm and tpl is more golang-template-related
[07:51:06] <elukey>	 but I am very puzzled
[07:51:16] <elukey>	 anyway, I'd stick with include, ok for you?
[07:51:25] <elukey>	 I can file the patches in a bit
[07:51:31] <elukey>	 thanks a lot for the brainbounce :)
[08:01:17] <brouberol>	 for sure!
[08:01:51] <elukey>	 brouberol: https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/1101450 and next
[08:16:59] <elukey>	 thanks for the reviews <3
[09:42:59] <brouberol>	 np :)
[21:43:35] <inflatador>	 working on a net-new service in https://gerrit.wikimedia.org/r/c/operations/deployment-charts/+/1091827 ...can anyone advise what `admission webhook "validation.istio.io" denied the request: configuration is invalid: duplicate hosts in virtual service: blunderbuss.ddiscovery.wmnet` means? Guessing we screwed up ingress?
[21:52:17] <inflatador>	 nm, got it