Skip to content

build: verify SOCI and Nydus artifacts - #5340

Open
iahsanGill wants to merge 2 commits into
containerd:mainfrom
iahsanGill:ci/verify-soci-nydus-artifacts
Open

iahsanGill wants to merge 2 commits into
containerd:mainfrom
iahsanGill:ci/verify-soci-nydus-artifacts

Conversation

@iahsanGill

Copy link
Copy Markdown

Summary

SOCI and Nydus integration-test artifacts were downloaded and extracted without integrity verification, unlike the other binary dependencies in the image.

  • add pinned SHA-256 sums for amd64 and arm64
  • verify each archive before extraction
  • make the canary updater track Nydus releases from dragonflyoss/nydus

Refs #4163

Verification

  • docker buildx build --check --platform linux/amd64,linux/arm64 --target build-test-integration-artifacts .
  • bash -n hack/build-integration-canary.sh
  • git diff --check
  • recorded checksums match the upstream release checksum files and GitHub release digests

Assistance

OpenAI Codex assisted with the implementation and verification of this change.

SOCI and Nydus test artifacts were downloaded and extracted without integrity verification, unlike the other binary dependencies in the integration image.

Add pinned SHA-256 sums for both supported architectures and verify each archive before extraction. Point the canary updater at the current Nydus release repository so future version bumps generate matching checksum files.

Signed-off-by: iahsanGill <mgill.bese22seecs@seecs.edu.pk>
Comment thread Dockerfile Resolved
@AkihiroSuda AkihiroSuda added the area/ci e.g., CI failure label Nov 10, 2026
Signed-off-by: iahsanGill <mgill.bese22seecs@seecs.edu.pk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci e.g., CI failure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants