Repository navigation
Conversation
CI (main) has failed twice in a row on 50b1a23 with three jobs: BDD conformance (2 scenarios), Test workspace (aarch64), and the meta-lane that requires every lane to pass. - The operations cookbook's TypeScript example still assigned a bare string env value; #5463 made EnvValue explicit, so the documented example no longer typechecks. Write MODE: literal("dev") and import literal. - mvmctl policy validate ./policy/api-group.toml is newly documented and sat outside the coverage ledger. Add a scenario that stages the guide's own group file in a scratch directory and validates it offline, then regenerate the ledger: the policy-validate examples move from uncovered debt to covered, and entries whose doc lines recent PRs removed leave the ledger. - a_console_spawned_shell_holds_no_capabilities read the pty master with read_to_end, which propagates EIO; EIO is how a pty master reports that every slave holder has closed it, so a fast-exiting shell failed the read on the aarch64 runner. Treat EIO as end of stream, matching pump_output's contract. Refs #5369
auser
force-pushed
the
fix/5369-bdd-aarch64-ci
branch
2 times, most recently
from
November 14, 2026 23:55
03cab35 to
86044aa
Compare
auser
changed the title
This branch has not been deployed
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
Since #5488 landed the console-EIO and cookbook env fixes and listed
mvmctl policy validate ./policy/api-group.tomlin the docs coverage ledger as uncovered debt, this PR completes that debt the way the ledger's header asks ("moving a line fromuncoveredtocoveredis the point of the file"):./policy/api-group.toml) in a scratch directory and runpolicy validateagainst it offline; exits 0 and printspolicy is validunder--verbose.policy validateexamples moveuncovered → covered; entries whose doc lines recent PRs removed leave the ledger.No production code changes; the earlier console/mdx fixes in this branch's history were identical to #5488 and dropped during the rebase.
Validation
just check::gatedclean (before rebase; the remaining diff is feature/toml/step files only, re-verified by the suite run above)Refs #5369