Can you fix this or should I open a PR. I'm blocked by this.

I can open a PR

You guys are the maintainers of this repo

https://github.com/allegroai/clearml-helm-charts/pull/107

Thanks! PR merged 🚀

Yes please 🙂
BTW: I originally thought the double quotes (in your PR) were also a bug, this is why I was asking, wdyt?

I dont know how to do that

Test it on your local setup (I would hate to push a broken fix)
Is that possible?

Merged, is it working for you now?

(once you verify PR the fix, I'll make sure it is merged)

Thanks everyone!

Hi everyone, I just fixed releases so new charts containing this fix are published. ty!

I learned helm a few days ago

Shouldn't this be a real value and not a template

you mean value being pulled to the pod that failed ?

AgitatedDove14 note the missing brackets https://github.com/allegroai/clearml-helm-charts/blob/main/charts/clearml-agent/templates/agentk8sglue-deployment.yaml#L22

Yea that is a similar bug, needs the same fix

I'm not familiar with helm that well to clone this, fix it, and then test

I cannot test it at the moment, hence my question.
JuicyFox94 any chance you can blindly approve ?

Its clearly a bug

I think the quotes don't effect the yaml

Seems like its just missing the brackets

Do you want me to PR that fix?

Yea this is a bug

Yes! Thanks so much for the quick turnaround

https://github.com/allegroai/clearml-helm-charts/blob/0871e7383130411694482468c228c987b0f47753/charts/clearml-agent/templates/agentk8sglue-deployment.yaml#L47

Yes! Thanks so much for the quick turnaround

My pleasure 🙂

BTW: did you see this (it seems like the same bug?!)
https://github.com/allegroai/clearml-helm-charts/blob/0871e7383130411694482468c228c987b0f47753/charts/clearml-agent/templates/agentk8sglue-configmap.yaml#L14

ouch, I think you are correct, can you test a fix?

For instance, quotes are used

I made the PR here JuicyFox94 AgitatedDove14 https://github.com/allegroai/clearml-helm-charts/pull/106

Can you fix locally, just to verify ?