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

Add Helm GitHub Actions #2

Merged
merged 8 commits into from
Aug 15, 2020
Merged

Add Helm GitHub Actions #2

merged 8 commits into from
Aug 15, 2020

Conversation

scottrigby
Copy link
Member

@scottrigby scottrigby commented Aug 9, 2020

Adds Helm GitHub actions for CI (linting and testing any changed charts in a PR), and CD (packaging merged chart changes and uploading to GitHub releases, and updating the repo index in GitHub pages). For more info, see https://github.com/helm/charts-repo-actions-demo

Note about steps:

  1. Merging this PR should not publish any charts yet, since there was no previous version of them in a GitHub pages helm repo index file.
    Edit: this did add all the existing charts on PR merge. I seem to remember that not working in the past, but I could be wrong or we may have fixed it upstream in chart-releaser. Either way, it works now. I deleted the releases/tags and reverted gh-pages back to the initial empty commit so we can still proceed with step 2 below
  2. In subsequent PRs, we can bump the versions of individual charts that we want to list. When we do, we should update each chart's README to add the new helm repo to installation instructions. See Create a new git repo for prometheus-community helm charts community#28 for the overall plan to incrementally improve/simplify charts after moving from stable to a new helm repo in https://github.com/prometheus-community

Signed-off-by: Scott Rigby <scott@r6by.com>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…chart-releaser-action#26

Signed-off-by: Scott Rigby <scott@r6by.com>
Signed-off-by: Scott Rigby <scott@r6by.com>
Signed-off-by: Scott Rigby <scott@r6by.com>
Signed-off-by: Scott Rigby <scott@r6by.com>

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
This reverts commit 81c50e5.
@scottrigby scottrigby merged commit 594a870 into main Aug 15, 2020
@scottrigby scottrigby deleted the add-actions branch August 15, 2020 17:14
@scottrigby
Copy link
Member Author

Fixed #4

sathieu pushed a commit to sathieu/helm-charts-prometheus-community that referenced this pull request Nov 26, 2021
Vitali-dshbq added a commit to Vitali-dshbq/helm-charts that referenced this pull request Jan 26, 2024
jkroepke added a commit that referenced this pull request Feb 10, 2025
* Extract version instead

Signed-off-by: aidanleuck <aidan@leucks.com>

* Bump version

Signed-off-by: aidanleuck <aidan@leucks.com>

* job.yaml aktualisieren

Signed-off-by: Jan-Otto Kröpke <github@jkroepke.de>

---------

Signed-off-by: aidanleuck <aidan@leucks.com>
Signed-off-by: Jan-Otto Kröpke <github@jkroepke.de>
Co-authored-by: Jan-Otto Kröpke <github@jkroepke.de>
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.

None yet

1 participant