Skip to content

fix(renovate): skip devcontainer base digest pinning - #828

Merged
robarnold merged 1 commit into
mainfrom
robarnold/codex-ignore-devcontainer-base-digest
Aug 6, 2026
Merged

fix(renovate): skip devcontainer base digest pinning#828
robarnold merged 1 commit into
mainfrom
robarnold/codex-ignore-devcontainer-base-digest

Conversation

@robarnold

@robarnold robarnold commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • exclude the repo-owned devcontainer base image from Renovate's digest-pinning policy
  • preserve digest pinning for all other Docker dependencies

Why

The inherited config:best-practices preset enables docker:pinDigests. Renovate therefore treats the latest default used by .devcontainer/Containerfile as an unpinned Docker tag and creates PRs such as #813.

This image is built, validated, and published by this repository. Keeping its developer fallback tag floating avoids a circular self-update PR after each publication, while regular CI continues to override it with the commit-specific locally built candidate.

Validation

  • jq empty renovate.json
  • git diff --check
  • renovate-config-validator --no-global renovate.json
  • verified exactly one package rule matches ghcr.io/marinatedconcrete/devcontainer-base with pinDigests: false
  • all nine regular checks passed

Keep the repo-owned devcontainer base on its intentionally floating tag while preserving digest pinning for third-party images.
@robarnold
robarnold marked this pull request as ready for review August 4, 2026 04:45
@robarnold
robarnold requested a review from sdwilsh August 4, 2026 04:45
@sdwilsh

sdwilsh commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What if we just update it weekly (Mondays)?

@robarnold

Copy link
Copy Markdown
Contributor Author

I worry it is another two points of failure: automation deps and fedora's retention policy & behavior not change from under us

@robarnold
robarnold merged commit 910d6c7 into main Aug 6, 2026
11 checks passed
@robarnold
robarnold deleted the robarnold/codex-ignore-devcontainer-base-digest branch August 6, 2026 03:23
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.

2 participants