Skip to content

Pin peribolos image by digest, tracked by Dependabot#16

Merged
banesullivan merged 2 commits into
mainfrom
chore/pin-peribolos-image
May 23, 2026
Merged

Pin peribolos image by digest, tracked by Dependabot#16
banesullivan merged 2 commits into
mainfrom
chore/pin-peribolos-image

Conversation

@banesullivan

@banesullivan banesullivan commented May 23, 2026

Copy link
Copy Markdown
Member

scripts/run-peribolos.sh pulled us-docker.pkg.dev/k8s-infra-prow/images/peribolos:latest. The tag is mutable: runs are not reproducible, and a compromised upstream image would execute in CI with the org-admin installation token mounted at /etc/github/token.

Pin the image to a digest and route updates through the repo's existing Dependabot hookup so the digest does not drift silently:

  • docker/peribolos/Dockerfile holds the pinned FROM line. The Dockerfile is never built; it exists as a single source of truth that Dependabot can watch.
  • .github/dependabot.yml gains a docker ecosystem entry pointed at that directory. Dependabot opens a PR when a new digest is available, on the same monthly cadence as the existing github-actions updates.
  • scripts/run-peribolos.sh parses the FROM line so the digest lives in exactly one place.

Initial pin:

sha256:6978d5adbb75487cbdb9088eef1437acd8a93a6e75f01abe76c5d0fca853bba8

Digest taken from the image pulled on run 26330237447.

Also enabled Dependabot security updates on the repo (settings toggle, no diff in this PR).

@banesullivan banesullivan requested a review from a team as a code owner May 23, 2026 18:46
@banesullivan banesullivan changed the title Pin peribolos image by digest Pin peribolos image by digest, tracked by Dependabot May 23, 2026
@banesullivan banesullivan enabled auto-merge May 23, 2026 18:50
@banesullivan banesullivan added this pull request to the merge queue May 23, 2026
Merged via the queue into main with commit 8809f79 May 23, 2026
6 checks passed
@banesullivan banesullivan deleted the chore/pin-peribolos-image branch May 23, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants