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

Chore: Automate helm test/doc update for renovate PR #6270

Merged
merged 5 commits into from
Oct 6, 2023

Conversation

ying-jeanne
Copy link
Contributor

@ying-jeanne ying-jeanne commented Oct 6, 2023

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

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@ying-jeanne ying-jeanne marked this pull request as ready for review October 6, 2023 15:43
@ying-jeanne ying-jeanne requested a review from a team as a code owner October 6, 2023 15:43
@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

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.

@ying-jeanne
Copy link
Contributor Author

ying-jeanne commented Oct 6, 2023

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.

Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov left a 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!

.github/workflows/lint-renovate-pr.yml Outdated Show resolved Hide resolved
.github/workflows/lint-renovate-pr.yml Outdated Show resolved Hide resolved
.github/workflows/lint-renovate-pr.yml Outdated Show resolved Hide resolved
.github/workflows/lint-renovate-pr.yml Outdated Show resolved Hide resolved
.github/workflows/lint-renovate-pr.yml Outdated Show resolved Hide resolved
@ying-jeanne ying-jeanne changed the title Chore: Fix the linter for renovate PR Chore: Automate helm test/doc update for renovate PR Oct 6, 2023
Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov left a comment

Choose a reason for hiding this comment

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

LGTM

@dimitarvdimitrov dimitarvdimitrov enabled auto-merge (squash) October 6, 2023 16:57
@dimitarvdimitrov dimitarvdimitrov merged commit 80ee3c1 into main Oct 6, 2023
28 checks passed
@dimitarvdimitrov dimitarvdimitrov deleted the renovatefixlint branch October 6, 2023 17:33
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.

3 participants