Skip to content

ci(security): pin third-party actions to digests in build-images.yml - #2094

Merged
Gregory91G merged 1 commit into
security-hardeningfrom
ci/pin-actions-build-images
Aug 1, 2026
Merged

ci(security): pin third-party actions to digests in build-images.yml#2094
Gregory91G merged 1 commit into
security-hardeningfrom
ci/pin-actions-build-images

Conversation

@Gregory91G

Copy link
Copy Markdown
Contributor

Refs #2020. Second half of #2093 — same change, isolated on purpose.

This file builds and publishes every image and the umbrella chart. A mistake here stops delivery rather than a test, so it is kept reviewable and revertable on its own rather than folded into the 62-line change across the other six workflows.

64 references rewritten, 10 distinct tags resolved through the GitHub API. Nothing else in the file is touched.

Test plan

  • All 10 distinct tags resolved; after the rewrite build-images.yml has 0 references left on a tag
  • Confirm the path filters on this workflow before merge — if a workflow-only change does not trigger the image builds, run it via workflow_dispatch so the pins are exercised before they reach main
  • One full image build and chart publish completes after merge

@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c332169a-f970-4067-8648-9e02c1ae6063

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Gregory91G
Gregory91G changed the base branch from main to security-hardening July 31, 2026 08:51
Same change as the rest of the workflows, kept apart because this file publishes
every image and the umbrella chart: a problem here stops delivery rather than a
test, so it needs to be reviewable and revertable on its own.

Signed-off-by: Gregory Gogin <grigoriy.gogin@constructor.tech>
@Gregory91G
Gregory91G force-pushed the ci/pin-actions-build-images branch from fedab99 to 57fc697 Compare July 31, 2026 18:05
@Gregory91G
Gregory91G merged commit 9c2199f into security-hardening Aug 1, 2026
2 checks passed
@Gregory91G
Gregory91G deleted the ci/pin-actions-build-images branch August 1, 2026 13:45
ktursunov pushed a commit that referenced this pull request Aug 4, 2026
The rebase landed on top of #2093/#2094, which pinned every third-party
action in these two files. The lines this branch added were pinned
independently and to different revisions of the same actions, which left
the repo naming one action at two digests — the state pinning exists to
prevent.

Aligned on what the rest of the tree already carries: docker/login-action
and docker/setup-buildx-action (four call sites that were still on a
floating tag), attest-build-provenance, and setup-uv.

Signed-off-by: Konstantin Tursunov <Konstantin.Tursunov@constructor.tech>
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