-
Notifications
You must be signed in to change notification settings - Fork 541
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
Chore: Automate helm test/doc update for renovate PR #6270
Conversation
b0722cf
to
8ab651f
Compare
Building new version of mimir-build-image. After image is built and pushed to Docker Hub, a new commit will automatically be added to this PR with new image version `grafana/mimir-build-image:pr6270-409ec28ca6`. This can take up to 1 hour. |
The installation is not straight forward, and I am getting the same error as here cli/cli#1797, plan to fix the renovate build for now. getting this improvement of using gh from build image in a separate PR to keep the changes small. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this, hopefully we can use this on human-submitted PRs too. I left a couple of minor comments, but overall - let's try it!
29ee863
to
5f5cb96
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What this PR does
When we update dependency like minio ying-jeanne#43, we need to update helm test equally in order to make ci passes, this PR is to add a script run after each renovate PR, update helm tests when necessary.
See test on my personal fork https://github.com/ying-jeanne/mimir/pull/43/commits
If you see some part seems "wierd", it is mainly because of using container in github action is tricky.
Which issue(s) this PR fixes or relates to
Fixes #
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]