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

Builders: Ensure /etc/mtab is available on Docker mingw64 image build #10

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

eht16
Copy link
Member

@eht16 eht16 commented Sep 26, 2023

Since some weeks, building the image on Github Runners fails because during image build the file /etc/mtab is not present in the build container.

After some research, I noticed this file is created automatically when building with Docker without BuildKit. Once BuildKit is enabled, this file is not created automatically anymore.
Note, this is independent from running the container image. There the file is always created.

The Github runners were probably upgraded some time in the last weeks and currently they are running Docker 24 which enables BuildKit by default.
On older Docker installations, this can be reproduced by setting DOCKER_BUILDKIT=1.

I didn't find any reference on why the file is not created anymore when using BuildKit. It's probably no issue for most image builds. In our case, we use the Pacman package manager which relies on /etc/mtab.

Therefore, we simply create the file (as symlink to /proc/mounts) if it doesn't exist yet.

When building the image with BuildKit (which is enabled by default in
newer Docker versions), /etc/mtab is not created automatically anymore.
Since Pacman relies on it, we create the file if necessary.
@eht16 eht16 force-pushed the ci_builders_create_mtab branch from 52a5e88 to f8e81de Compare October 1, 2023 11:04
@eht16 eht16 merged commit 1531822 into master Oct 11, 2023
@eht16 eht16 deleted the ci_builders_create_mtab branch October 11, 2023 10:44
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