Closed lleyton closed 4 months ago
Ouch, I totally missed this in my SMTP PR 🤦🏻♀️ Thanks for fixing it @lleyton!
hope it gets merged soon so I don't have to rely on my fork :3
Thanks for your fixing! When is this PR likely to be merged?
PR ready, approved, but no one merging this fix leaving the Helm-Chart broken for months.... please merge this.
@timetinytim @renchap Would y'all be able to give this a quick look/merge? Currently the only thing preventing me (and others) from running this chart directly when using Flux CD.
@wrmilling No problem, doing so now! Sorry for the overall delay in getting all these PRs merged, I'm finally working through them all :)
@timetinytim All good, thanks for jumping on it!
Use an unique annotation key for the smtp secret checksum, fixing the conflict mentioned in #48.