Skip to content

contrib: add a Docker images digest checker#1185

Merged
mtardy merged 2 commits into
mainfrom
pr/mtardy/digest-checker
Jul 5, 2023
Merged

contrib: add a Docker images digest checker#1185
mtardy merged 2 commits into
mainfrom
pr/mtardy/digest-checker

Conversation

@mtardy

@mtardy mtardy commented Jul 4, 2023

Copy link
Copy Markdown
Member

Add a shell script and its workflow to automatically detect when we use arch-restricted Docker images when we could use multi-arch ones.

@mtardy
mtardy force-pushed the pr/mtardy/digest-checker branch 10 times, most recently from 2d4531d to 399ec48 Compare July 5, 2023 09:29
mtardy added 2 commits July 5, 2023 11:31
Add a shell script and its workflow to automatically detect when we use
arch-restricted Docker images when we could use multi-arch ones.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Go image used to build gops was manually updated to an arch-specific
digest of that tag. This replaces it with the generic digest for the
multi-arch image.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy
mtardy force-pushed the pr/mtardy/digest-checker branch from 399ec48 to 2d21faf Compare July 5, 2023 09:32
@mtardy
mtardy marked this pull request as ready for review July 5, 2023 09:33
@mtardy
mtardy requested review from a team and willfindlay as code owners July 5, 2023 09:33
@mtardy

mtardy commented Jul 5, 2023

Copy link
Copy Markdown
Member Author

My only concern is that this thing gets rate limited by Dockerhub when trying to fetch the manifest from there (our images are on quay). Let's see how it behaves, it should run rarely given the triggering rules. Let's 🚢 !

@mtardy
mtardy merged commit 8878c17 into main Jul 5, 2023
@mtardy
mtardy deleted the pr/mtardy/digest-checker branch July 5, 2023 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants