Skip to content

Conversation

carrodher
Copy link
Member

@carrodher carrodher commented Sep 19, 2025

This PR removes some actions that are not used anymore after changes in the offering:
.github/workflows/cd-pipeline.yml: Helm charts are maintained in terms of code, but no new releases are performed in the index.yaml nor OCI registry

  • .github/workflows/index-monitor.yml: Index is not updated anymore (although it is not going to be removed)
  • .github/workflows/index-update.yml: Index is not updated anymore (although it is not going to be removed)
  • .github/workflows/retry-failed-releases.yml: There are no more automated PRs to retry in case of failure
  • .github/workflows/ci-pipeline.yml: Was modified so now it only performs the following actions:
    • Look for hardcoded images
    • Automatically update README, CRDs and CHANGELOG
    • Get chart latest Kubescape score

Related to #36275

@github-actions github-actions bot added verify Execute verification workflow for these changes bitnami labels Sep 19, 2025
@github-actions github-actions bot requested a review from alvneiayu September 19, 2025 12:23
migruiz4
migruiz4 previously approved these changes Sep 19, 2025
Copy link
Collaborator

@fmulero fmulero left a comment

Choose a reason for hiding this comment

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

I think the ci-pipeline will require a higher refactor:

  • kubescape is used to get score in main branch and this value is used in VIB as threshold. If we remove VIB, this step doesn't make sense.
  • CRDs step has conditions that we will never meet again.
  • We could remove all the secrets and use pull_request events instead of pull_request_target

carrodher and others added 2 commits September 30, 2025 16:27
Signed-off-by: Carlos Rodríguez Hernández <[email protected]>
Copy link
Member Author

@carrodher carrodher left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@dgomezleon dgomezleon left a comment

Choose a reason for hiding this comment

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

LGTM

@fmulero fmulero merged commit 2027c67 into main Oct 1, 2025
15 checks passed
@fmulero fmulero deleted the TNZ-60424-1 branch October 1, 2025 05:39
@github-actions github-actions bot added the solved label Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bitnami solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants