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

Merged
adityamukho merged 1 commit into
mainfrom
fix/887-docker-lock-helper
Nov 14, 2026
Merged

adityamukho merged 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!

@adityamukho
adityamukho force-pushed the fix/887-docker-lock-helper branch from 5f3e38f to 220a1ed Compare November 14, 2026 02:18
@adityamukho
adityamukho merged commit 82e1ea5 into main Nov 14, 2026
16 of 17 checks passed
@adityamukho
adityamukho deleted the fix/887-docker-lock-helper branch November 14, 2026 02:29
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