search

bokysan / docker-postfix

Multi architecture simple SMTP server (postfix null relay) host for your Docker and Kubernetes containers. Based on Debian/Ubuntu/Alpine.
MIT License
555 stars 152 forks source link

Helm upgrade from 4.1.0 to 4.2.0 failing: mapping key "name" already defined #196

Closed m-a-w-i closed 7 months ago

m-a-w-i commented 7 months ago

When upgrading from helm chart 4.1.0 to 4.2.0 ran into the issue:

reconciliation failed: Helm upgrade failed: error while running post render on files: map[string]interface (nil): yaml: unmarshal errors: line 150: mapping key "name" already defined at line 148

From what I see this is, because the key name is used twice for the new tmp volumeMount in the statefulset template

bokysan commented 7 months ago

You are correct. Weird that this was not caught by automated tests.

I'll fix that ASAP.

bokysan commented 7 months ago

This commit should have resolved the issue.

Can you please check the :edge version (once it compiles) if it's still giving you the problems?

If no, I will publish a fix in the next few hours.

m-a-w-i commented 7 months ago

Thanks for the quick fix! Unfortunately I don't have an environment available, where I can easily run such tests. A released fix I should be able to easily test.

m-a-w-i commented 7 months ago

Thanks for the release. I can confirm that it is working.