Skip to content

Conversation

@tariq1890
Copy link
Contributor

@tariq1890 tariq1890 commented Nov 26, 2025

The GitHub runners have now started using a newer version of docker-ce where --provenance is enabled by default. This results in the native-arch image build creating a docker manifest list instead of a single image. This causes issues in our multiarch builds as we need to create a docker manifest out of the two images built for the amd64 and arm64 architectures.

With this fix, we should no longer see errors like the one below:

$ docker manifest create \
    ${OPERATOR_MULTIARCH_IMAGE} \
    ${OPERATOR_IMAGE_AMD} \
    ${OPERATOR_IMAGE_ARM}
ghcr.io/nvidia/gpu-operator:d8d2842a-amd64 is a manifest list

Signed-off-by: Tariq Ibrahim <tibrahim@nvidia.com>
@coveralls
Copy link

Coverage Status

coverage: 22.93%. remained the same
when pulling 635b9ec on disable-provenance
into aa17fc8 on main.

@tariq1890 tariq1890 enabled auto-merge November 26, 2025 06:36
@tariq1890 tariq1890 merged commit 526cc24 into main Nov 26, 2025
30 of 31 checks passed
@tariq1890 tariq1890 deleted the disable-provenance branch November 26, 2025 08:11
@tariq1890
Copy link
Contributor Author

/cherry-pick release-25.10

@github-actions
Copy link

🤖 Backport PR created for release-25.10: #1941

rapids-bot bot pushed a commit to rapidsai/docker that referenced this pull request Dec 2, 2025
Closes #820

*(description copied from #820 (comment)

`build-*-manifest` CI jobs are failing like this:

```text
docker.io/rapidsai/staging:docker-cuvs-bench-820-26.02a-cuda13-py3.10-amd64 is a manifest list
Error: Process completed with exit code 1.
```

([build link](https://github.com/rapidsai/docker/actions/runs/19864919436/job/56926457521?pr=820#step:4:23))

Looks like in newer versions of `docker`, builds create manifest lists by default instead of single image builds:

* NVIDIA/gpu-operator#1940
* NVIDIA/k8s-driver-manager#132
* docker/build-push-action#1339 (comment)

This opts out of that using the configuration option added in docker/build-push-action#1343

It also updates all `pre-commit` hooks, to avoid needing another CI run for #820

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Bradley Dice (https://github.com/bdice)

URL: #821
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.

4 participants