Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#1879: fix hook-scripts missing in helm upgrade #1880

Merged

Conversation

WatcherWhale
Copy link
Contributor

@WatcherWhale WatcherWhale commented Jan 29, 2024

Fixes #1879
Signed-off-by: Mathias Maes mathias.maes@aloxy.io

Signed-off-by: Mathias Maes <mathias.maes@aloxy.io>
@WatcherWhale WatcherWhale force-pushed the fix-helm-pre-upgarde-hook branch from ec5be3a to 6c1f364 Compare January 29, 2024 09:30
Copy link
Member

@thjaeckle thjaeckle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ouch, that's on me.

Thanks a lot for the quick fix, appreciate it a lot 👍

@thjaeckle thjaeckle merged commit 714b328 into eclipse-ditto:master Jan 29, 2024
9 checks passed
@thjaeckle
Copy link
Member

Fixed in deployed Helm chart 3.5.1

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Helm: incorrect deploy order in 3.5.0 helm chart
2 participants