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 with injected Transport errors #286

Open
sourcefrog opened this issue Mar 2, 2025 · 1 comment
Open

Test with injected Transport errors #286

sourcefrog opened this issue Mar 2, 2025 · 1 comment

Comments

@sourcefrog
Copy link
Owner

sourcefrog commented Mar 2, 2025

Conserve aspires to be robust against IO errors in archive storage, but this could be better tested.

In particular, cargo-mutants points out that some of the code paths that are meant to handle this in blockdir.rs aren't well tested.

I started using failpoints but perhaps a better approach is to allow configuring some errors into the Transport, since we already have that chokepoint.

For now some mutants are turned off in mutants.toml; they could be re-enabled when this is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant