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: build helm repo index #391

Merged
merged 1 commit into from
Mar 14, 2023
Merged

ci: build helm repo index #391

merged 1 commit into from
Mar 14, 2023

Conversation

apricote
Copy link
Member

We previously relied on the helm-charts repo to re-build the index. But this has problems:

  • The gh-pages pipeline is not properly started because events created by the default GITHUB_TOKEN do not trigger pipelines (in the same repo).
  • Adding the chart and re-indexing happens in two commits, slightly obfuscating the chain of events
  • When running helm repo index it updates the created timestamp of all existing entries. While I have not seen any issues yet, there might be some clients that rely on this field.

We previously relied on the helm-charts repo to re-build the
index. But this has problems:

- The gh-pages pipeline is not properly started because events created by the
  default GITHUB_TOKEN do not trigger pipelines (in the same repo).
- Adding the chart and re-indexing happens in two commits, slightly obfuscating
  the chain of events
- When running `helm repo index` it updates the created timestamp of all existing
  entries. While I have not seen any issues yet, there might be some clients that
  rely on this field.
@apricote apricote added the enhancement New feature or request label Mar 14, 2023
@apricote apricote requested a review from a team as a code owner March 14, 2023 09:16
@apricote apricote self-assigned this Mar 14, 2023
@apricote apricote merged commit 17e534d into main Mar 14, 2023
@apricote apricote deleted the build-helm-repo-index branch March 14, 2023 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants