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

Simplify release-related logic in GHA's kubeapps general pipeline #5718

Merged
merged 2 commits into from
Nov 25, 2022

Conversation

beni0888
Copy link
Collaborator

Signed-off-by: Jesús Benito Calzada bjesus@vmware.com

Description of the change

This commit aims to simplify the logic in the GHA's Kubeapps General workflow. Currently, we have some logic that checks if the pipeline has been triggered by a tag pushed to the repo, and if it is, we run some jobs: push_images, sync_charts, release, etc. As we already have a top-level workflow kubeapps-release.yml that internally uses the general workflow, and only runs when a tag following the release pattern is pushed to the repo, we can just introduce an input parameter in the general workflow that controls whether those jobs should be run or not, instead of relying on the logic residing in the general pipeline.

Benefits

GHA's workflows kubeapps-general.yaml and kubeapps-release.yml are simpler to understand and maintain.

Possible drawbacks

None.

Applicable issues

Signed-off-by: Jesús Benito Calzada <bjesus@vmware.com>
@netlify
Copy link

netlify bot commented Nov 25, 2022

Deploy Preview for kubeapps-dev canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit b827c81
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/6380b478e7f6e00008bf6000

@beni0888 beni0888 self-assigned this Nov 25, 2022
@beni0888 beni0888 added component/ci Issue related to kubeapps ci system github_actions Pull requests that update GitHub Actions code labels Nov 25, 2022
@beni0888 beni0888 added this to the Migrate CI to GitHub Actions milestone Nov 25, 2022
@beni0888 beni0888 marked this pull request as ready for review November 25, 2022 12:17
Copy link
Contributor

@antgamdia antgamdia left a comment

Choose a reason for hiding this comment

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

Ah great, so this way the condition is only written once and then just passed as a param to the workflow. Neat!

Signed-off-by: Jesús Benito Calzada <bjesus@vmware.com>
@beni0888 beni0888 merged commit d891478 into vmware-tanzu:main Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-not-required component/ci Issue related to kubeapps ci system github_actions Pull requests that update GitHub Actions code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants