Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Base: Update to Debian 12.5 #82

Merged
merged 5 commits into from
Jun 2, 2024
Merged

Conversation

mbargull
Copy link
Member

@mbargull mbargull commented Jun 1, 2024

mbargull added 4 commits June 1, 2024 22:13
Signed-off-by: Marcel Bargull <[email protected]>
Signed-off-by: Marcel Bargull <[email protected]>
Signed-off-by: Marcel Bargull <[email protected]>
@mbargull mbargull force-pushed the base-debian-12.5 branch 3 times, most recently from b256652 to 7bb1608 Compare June 2, 2024 11:03
Comment on lines +19 to +21
# travier/podman-action contains newer podman/buildah versions.
image: quay.io/travier/podman-action
options: --privileged
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

@mbargull mbargull Jun 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should fix bioconda/bioconda-utils#958 since this uses buildah >=1.35.0 which includes containers/storage#1847 .
On the Docker side, the issue should also be resolved for older images when docker >=25.0.4 onward is used.

Comment on lines +36 to +37
podman run --rm --privileged \
docker.io/tonistiigi/binfmt --install arm64
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is essentially the same as what docker/setup-qemu-action does (it even uses the same container image, IIRC).

Comment on lines 42 to 44
# jq is not installed in travier/podman-action
dnf install -y \
jq
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We didn't need this beforehand since jq is installed on the GitHub Actions runner image, but this now runs inside a container.

@mbargull mbargull force-pushed the base-debian-12.5 branch from 7bb1608 to dfb738f Compare June 2, 2024 11:30
@mbargull mbargull merged commit 55781f6 into bioconda:main Jun 2, 2024
3 checks passed
mbargull added a commit to bioconda/bioconda-utils that referenced this pull request Jun 2, 2024
New images use Debian 12.5 base and are built with Buildah >=1.35.0 for
better compatibility w.r.t. gh-958 .

refs:
- bioconda/bioconda-containers#82

fixes gh-958

Signed-off-by: Marcel Bargull <[email protected]>
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.

1 participant