Repository navigation
test: expand deadman VM regression coverage (04) - #3
Merged
Merged
Conversation
Expose one canonical deadman check and cover isolated recovery, disarm, and arm rejection scenarios without changing runtime helpers. Co-authored-by: Copilot App <[email protected]>
Co-authored-by: Copilot App <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Item04 only. Reuse the existing locked NixOS VM harness and expose it once as
checks.x86_64-linux.deadman; no runtime helper changes.Evidence
Final HEAD:
4eb84978fc6adfa82d6436264b2074e95ee3d55d.Final Linux/KVM run 35612136537 passed
nix flake check --no-update-lock-file --print-build-logsagainst the unchanged locked NixOS 26.05 test input. The logs show the actualchecks.x86_64-linux.deadmanbuild and all seven subtests completing in a 272.05-second VM script, including the full 70-second disarm wait. This was an actual VM execution, not evaluation-only or a mocked check.Initial run 35611400808 also passed at
b955cefbfe9f88be039a813598085cacf2363424; the only subsequent codebase change records that evidence in the changelog. Embedded Python parsing andgit diff --checkpassed locally. The worktree is clean.No live-host timer tests, no bootloader coverage, no changes to runtime helpers, the lockfile, LICENSE, operational playbook/prompt, or CONTRIBUTING; no later-plan items. Optional profile-write-failure injection remains outside this change. Mergeable with passing CI, but intentionally left open and unmerged for the parent session's ordered upgrade coordination.