search

mattermost / mattermost-helm

Mattermost Helm charts for Kubernetes
Apache License 2.0
162 stars 145 forks source link

Simplify release script and skip existing releases #368

Closed spirosoik closed 1 year ago

spirosoik commented 1 year ago

Summary

We simplified the script and utilising better the chart-releaser CLI. Now we iterate through all charts and we package them but we will upload only the packages are new. Added a new flag for chart-releaser which will help us to skip the existing packages.

Ticket Link

https://mattermost.atlassian.net/browse/CLD-3970

phoinixgrr commented 1 year ago

Great work here.

I'm always in font of simplifying things 👍 Shellcheck comes green, no objections merging this

spirosoik commented 1 year ago

@pfltdv I would like your feedback first before I merge. I removed the complexity and I just followed what everyone does out there actually for the public helm charts (deliveryhero, grafana, prometheus etc.)

ghost commented 1 year ago

@pfltdv I would like your feedback first before I merge. I removed the complexity and I just followed what everyone does out there actually for the public helm charts (deliveryhero, grafana, prometheus etc.)

Amazin work @spirosoik . It is so simple now with only adding one flag.