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

Test failures with bcachefs #9887

Open
Chwiggy opened this issue May 9, 2024 · 7 comments
Open

Test failures with bcachefs #9887

Chwiggy opened this issue May 9, 2024 · 7 comments

Comments

@Chwiggy
Copy link

Chwiggy commented May 9, 2024

What is the bug?

During the build process of GDAL 3.8.5 30 to 40 pytests fail on systems running on bcachefs. By majority these are tests concerning checksums on files involved in I/O operations.

This bug is not reproducible on other filesystems, like ext4

Steps to reproduce the issue

Tentative: Try and build gdal 3.8.5 on a machine running bcachefs.

Versions and provenance

OS info

- system: `"x86_64-linux"`
 - host os: `Linux 6.8.8, NixOS, 24.05 (Uakari), 24.05.20240502.63c3a29`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - nixpkgs: `/nix/store/p69bcs7ma6ijj8v9xsrg3nq3nn8ryn95-source`
 - filesystem: bcachefs

GDAL version

3.8.5

Provenance

nixpkgs:unstable

Additional context

relevant issue on nixpkgs: NixOS/nixpkgs#302137
full build logs: https://gist.github.com/Chwiggy/e18dcb59ae47b78c3edbcd965d09d6ac

@rouault
Copy link
Member

rouault commented May 9, 2024

couldn't that be a defect (or "feature") of bcachefs itself, given that the tests pass on a variety of more standard filesystems?

@Chwiggy
Copy link
Author

Chwiggy commented May 9, 2024

could very well be. Would still be useful for someone who has more insight into gdal's testing setup, to figure out if these failed tests break because the test setup makes broken assumptions about how filesystems behave, if these tests might be skipped or need to be modified if building against specific filesystems, or if this is a flaw with bcachefs, that needs to be fixed within bcachefs

@Chwiggy
Copy link
Author

Chwiggy commented May 9, 2024

also notably, this seems to not have been an issue before gdal 3.8.4, and gdal 3.8.4 built with a bit of coercion in the end.

@rouault
Copy link
Member

rouault commented May 9, 2024

also notably, this seems to not have been an issue before gdal 3.8.4

could you run a git bisect session then ?

@rouault rouault changed the title Build failure with bcachefs Test failures with bcachefs May 9, 2024
@nh2
Copy link

nh2 commented Jul 12, 2024

NixOS/nixpkgs#302137 (comment)

=========================== short test summary info ============================
FAILED gcore/vsiaz.py::test_vsiaz_fake_write - TypeError: Failed expected string as 'msg' parameter, got 'int' instead.
FAILED gcore/vsioss.py::test_vsioss_6 - AssertionError: (1, 2)

It would be very surprising if a Python type confusing string and int was caused by the underlying file sytem.

@Chwiggy Does NixOS Hydra even use bcachefs?

@rouault
Copy link
Member

rouault commented Jul 12, 2024

FAILED gcore/vsiaz.py::test_vsiaz_fake_write - TypeError: Failed expected string as 'msg' parameter, got 'int' instead.
FAILED gcore/vsioss.py::test_vsioss_6 - AssertionError: (1, 2)

Some of those vsi tests are known to randomly fail for unknown reasons

@nh2
Copy link

nh2 commented Jul 12, 2024

@rouault Do you have a list ones of all those that are flaky?

We should disable them in NixOS (and all other Linux distributions likely want to do the same).

It might even make sense to disable them in the upstream package, adding a flag or so to enable flaky tests.

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

No branches or pull requests

3 participants