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

Can you fix locally, just to verify ?

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

Thanks! PR merged 🚀

I think the quotes don't effect the yaml

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

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

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

You guys are the maintainers of this repo

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

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

Do you want me to PR that fix?

Thanks everyone!

Yes! Thanks so much for the quick turnaround

Yea this is a bug

Seems like its just missing the brackets

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

I dont know how to do that

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

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

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

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

Yea that is a similar bug, needs the same fix

For instance, quotes are used

I learned helm a few days ago

Its clearly a bug

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

I can open a PR

Merged, is it working for you now?