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

fix: run 'cargo test' under 'nix flake check' with a non-debug profile #2340

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

thoughtpolice
Copy link
Member

Summary: The Nix CI has been failing recently due to (what I assume is) disk space issues. But only the flake check step is failing. Right now, nix flake check runs the Cargo tests with the debug profile to help get more debug info, which is even heftier in terms of debug info than the normal 'test' profile. For reference, a single build of 'cargo test' in a clean working copy results in a 15 gigabyte target/ directory.

Turn off the debug profile for nix flake check, which should hopefully stem the bleeding a bit. I believe the 'test' profile should still have enough symbols for backtraces, so panics should still be useful.

Change-Id: Idde10ac15847a1ad1e6f4e48a2497eca

Checklist

If applicable:

  • I have updated CHANGELOG.md
  • I have updated the documentation (README.md, docs/, demos/)
  • I have updated the config schema (cli/src/config-schema.json)
  • I have added tests to cover my changes

Summary: The Nix CI has been failing recently due to (what I assume is) disk
space issues. But only the `flake check` step is failing. Right now, `nix flake
check` runs the Cargo tests with the debug profile to help get more debug info,
which is even heftier in terms of debug info than the normal 'test' profile. For
reference, a single build of 'cargo test' in a clean working copy results in a
15 gigabyte `target/` directory.

Turn off the debug profile for `nix flake check`, which should hopefully stem
the bleeding a bit. I believe the 'test' profile should still have enough
symbols for backtraces, so panics should still be useful.

Signed-off-by: Austin Seipp <[email protected]>
Change-Id: Idde10ac15847a1ad1e6f4e48a2497eca
@thoughtpolice thoughtpolice enabled auto-merge (rebase) October 5, 2023 16:44
@thoughtpolice thoughtpolice merged commit d5b2f8e into main Oct 5, 2023
15 checks passed
@thoughtpolice thoughtpolice deleted the push-xzymutwrznyu branch October 5, 2023 16:48
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.

2 participants