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

renovate: Update builder and runtime images once a week #24846

Merged
merged 1 commit into from
Apr 12, 2023
Merged

Conversation

michi-covalent
Copy link
Contributor

The golang image SHAs keep changing even for the same patch version, and renovate keeps opening pull requests like this one:

#24831

Let's try limiting these updates to once a week. It should be enough since we only release patch versions once a month.

@michi-covalent michi-covalent requested a review from a team as a code owner April 12, 2023 17:33
@michi-covalent michi-covalent requested a review from bimmlerd April 12, 2023 17:33
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 12, 2023
@michi-covalent michi-covalent added the release-note/ci This PR makes changes to the CI. label Apr 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 12, 2023
The golang image SHAs keep changing even for the same patch version,
and renovate keeps opening pull requests like this one:

#24831

Let's try limiting these updates to once a week. It should be enough
since we only release patch versions once a month.

Signed-off-by: Michi Mutsuzaki <[email protected]>
@aanm aanm merged commit d827951 into master Apr 12, 2023
@aanm aanm deleted the pr/michi/renovate branch April 12, 2023 18:13
@gandro
Copy link
Member

gandro commented Apr 13, 2023

I don't think this change was effective. Renovate just opened a PR updating some digests on a Thursday: #24861

date Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

@michi-covalent
Copy link
Contributor Author

ah i think i know what's going on. https://github.com/cilium/cilium/pull/24861/files is not updating runtime/builder images, but it's updating:

  • images/cilium-docker-plugin/Dockerfile
  • images/clustermesh-apiserver/Dockerfile
  • images/hubble-relay/Dockerfile
  • images/operator/Dockerfile

let's rate-limit these as well ✅

michi-covalent added a commit that referenced this pull request Apr 13, 2023
We added weekly schedule for the builder and runtime images in #24846.
There are other Dockerfiles that directly use docker.io/library/golang
for building (see #24861). Let's do the same weekly update for these
images.

Signed-off-by: Michi Mutsuzaki <[email protected]>
michi-covalent added a commit that referenced this pull request Apr 14, 2023
We added weekly schedule for the builder and runtime images in #24846.
There are other Dockerfiles that directly use docker.io/library/golang
for building (see #24861). Let's do the same weekly update for these
images.

Signed-off-by: Michi Mutsuzaki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants