Hi, can someone review this PR and reopen this iss...
# contribute-code
s
a
Hi @shy-keyboard-55519, I’ll see if someone can take a peak, thanks! Just to confirm, you are still experiencing this issue in the latest version?
s
TY 🙏 Yeah, I'm still facing it, the part that my PR fixes was already reported in the issue, but only environment variables mentioned at title of issue (
EBEAN_*
) was fixed and issue was closed. I don't understand at all how it can work for you (your team), the manifests generated by Helm are not valid for Kubernetes and Kubernetes will deny to persist them. So I haven't been able to try even this upgrade (https://datahubspace.slack.com/archives/C029A3M079U/p1675851378010729) yet because the helm chart since then still has this bug.
So if my PR gets merged, you don't need to reopen the issue. Everything will be fixed.
a
Hey @brainy-tent-14503, is there any chance we could get this merged soon? We have lots of duplicate keys in our system upgrade job too.
b
LGTM! Thank you! Merged