rimusz / charts

Helm Charts for Kubernetes
MIT License
90 stars 70 forks source link

Newest version doesn't work 0.25.3 #145

Closed rhummelmose closed 1 year ago

rhummelmose commented 1 year ago

Service name no longer corresponds to application name rendering the Helm chart non-functional. It caused 2 of my production setups a few hours of down time.

Please don't do something like this again.............

Caused by commit https://github.com/rimusz/charts/commit/867f85b9f5fa9d5dd13483e3308d21788c370d8f

rhummelmose commented 1 year ago

@danyworks @daibrahi

rhummelmose commented 1 year ago

This is how the problem looks when encountered:

➜ kubectl port-forward -n frihedsbrevet-system service/gcloud-sqlproxy :3306   
error: Pod 'gcloud-sqlproxy-6f8787bf8b-5nxjc' does not have a named port 'frihe-ems4pbefh'
rhummelmose commented 1 year ago

@rimusz

danyworks commented 1 year ago

@rhummelmose sorry for the inconvenience, i will fix it straight away!

danyworks commented 1 year ago

@rhummelmose PR is created, it passed all checks, now waiting for @rimusz to merge it

rhummelmose commented 1 year ago

Awesome. The reason you didn't encounter this problem is because you specify the short instance name explicitly in values?