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

Improve updating image deps workflow #4222

Merged
merged 10 commits into from
Dec 14, 2023
Merged

Conversation

sgalsaleh
Copy link
Member

@sgalsaleh sgalsaleh commented Dec 14, 2023

What this PR does / why we need it:

This changes the workflow for updating image tags for 3rd party images like minio, rqlite, and dex from a bottom-top approach (PR -> main -> Dockerhub -> regression tests -> release) to a top-down approach (Dockerhub -> PR -> main -> regression tests -> release), which helps avoid making unnecessary decisions in our CI for when the images need to be pushed or overwritten.

This PR also creates a daily job that:

  • Retrieves the latest versions and epochs from the Wolfi OSS packages for the minio, rqlite, and dex images.
  • If the version or epoch is different, a new image will be built and pushed to our Dockerhub account.
  • Runs the imagedeps script to retrieve the latest versions for all dependencies.
  • Creates a PR if there are new versions available.
  • Sends a Slack notification with the link to the PR.

The imagedeps scripts has been updated to retrieve the latest tags for the minio, rqlite, and dex images from our Dockerhub account.

Which issue(s) this PR fixes:

Fixes SC-94750

Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

NONE

Does this PR require documentation?

NONE

cbodonnell
cbodonnell previously approved these changes Dec 14, 2023
@sgalsaleh sgalsaleh merged commit 4bd2b94 into main Dec 14, 2023
163 of 166 checks passed
@sgalsaleh sgalsaleh deleted the improve-image-deps-workflow branch December 14, 2023 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants