Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

ci(release): auto release #1213

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

TartanLeGrand
Copy link
Contributor

CI/CD Workflow Enhancement for Helm Charts 🚀

We've recently updated our CI/CD workflow to optimize how we build and publish our Helm charts for sentry, clickhouse, and sentry-kubernetes. Here are the key changes and their benefits:

  1. Selective Triggering: The workflow now initiates only upon the creation of tags by Release Please, ensuring the build and publish process runs only when necessary, avoiding unnecessary executions and redundant deployments. 🛠️

  2. Separate Chart Builds: Each chart is built independently, allowing for more granular version management and changes specific to each chart. This helps isolate the build processes, making debugging and testing easier and more focused. 🔍

  3. Efficient Indexing: Post-build, an index file is generated for the charts, improving discoverability and installation by users. Automatic indexing ensures the Helm repository is always up-to-date with the latest versions available. 📊

  4. No Manual Version Editing: With Release Please handling versioning, there's no longer a need to manually edit chart versions. This reduces the risk of human error and streamlines the release process. 📝

Benefits:

  • Optimized Performance: By limiting builds to when a release tag is created, we save resources and reduce wait time for deployments. 🚀
  • Increased Reliability: Each step is designed to be robust and less prone to errors, ensuring stable and reliable deployments. 🛡️
  • Ease of Maintenance: The clear separation of steps and improved documentation make the process more understandable and easier to manage. 📚

Next Steps:

We will closely monitor the workflow's behavior to ensure it doesn't trigger unnecessary builds, particularly avoiding re-building and re-pushing charts that haven't changed. Additionally, we'll likely need to write a contributor guide, as commits must adhere to the Conventional Commits specification to ensure the Release Please can correctly version the charts. 📝🔧

In summary, this update marks a significant step forward in automating and enhancing our CI/CD process for Helm charts, aligning our operations with best practices in release management and deployment.

@Mokto
Copy link
Contributor

Mokto commented Apr 12, 2024

Let's try this, I'm curious how it works. Seems nice!

@Mokto Mokto merged commit 82e7c71 into sentry-kubernetes:develop Apr 12, 2024
2 checks passed
@Mokto
Copy link
Contributor

Mokto commented Apr 12, 2024

@TartanLeGrand TartanLeGrand deleted the ci/release-please branch April 13, 2024 09:24
@TartanLeGrand
Copy link
Contributor Author

TartanLeGrand commented Apr 13, 2024

@Mokto yes sorry for this missing ... But now it's work 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants