8000 ci: fix nightly lock-suite helper discovery by adityamukho · Pull Request #889 · project-minigraf/minigraf · GitHub
[go: up one dir, main page]

Skip to content

ci: fix nightly lock-suite helper discovery - #889

Open
adityamukho wants to merge 1 commit into
mainfrom
fix/887-docker-lock-helper
Open

adityamukho wants to merge 1 commit into
mainfrom
fix/887-docker-lock-helper

Conversation

@adityamukho
@adityamukho adityamukho commented Nov 13, 2026 •
Copy link
Copy Markdown
Collaborator

The nightly Docker and NFS lock suites build pid_ns_helper successfully but look for it in the wrong directory when test executables run under target/debug/build/.../out.

Set an explicit workflow-wide CARGO_TARGET_DIR and MINIGRAF_PID_NS_HELPER so both jobs find the built helper independently of the test executable layout. This follows the existing coverage workflow's helper override.

Validation:

  • Reproduced the Docker helper lookup failure from the reported nested layout; it passes using the exact workflow environment.
  • Both existing NFS helper tests pass from the same relocated layout.
  • Workflow YAML parses, both branch patches are identical, and git diff --check passes.
  • Claude reviewed the published file contents: no actionable findings.

Full Docker and mounted-NFS execution requires CI infrastructure and was not run locally.

Fixes #887.
Fixes #888.

Companion v3 PR: #890.

@codecov
codecov Bot commented Nov 13, 2026
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

This branch has not been deployed

No deployments
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.

Nightly NFS lock-suite failure Nightly Docker lock-suite failure

1 participant

0