Repository navigation
Prepare disabled Mint media authentication and offline console workflow - #77
Conversation
|
Automatic CI verified on exact head c587dc1:
Local checks: all 84 mocked/static checks, focused actionlint and diff whitespace checks passed. The complete eight-file PR diff is within the exact infrastructure allowlist; mixed application, signing-key and unknown-path controls retain application CI. Live inventory, native-fixture and application workflow definitions are unchanged from main. No manual dispatch, new preparation job, runner operation, activation-setting change, media acquisition, guest execution, application build/package/release or merge was performed. The new workflow is disabled by default and still requires separate reviewed access-control and execution authorization. Native cancellation/cleanup/authentication remain unproven. #77 and #73 remain draft/open/unmerged. |
|
Reviewed and fixed at 5ac0559. Finding: the reused owned curl/GPG process helper filtered out RUNNER_TRACKING_ID, weakening the runner's independent orphan-process cleanup fallback after forced caller termination. The fixed child environment preserves that opaque marker only when supplied, without logging it or propagating unrelated GitHub credentials, SSH agent settings or loader injection variables. This is cleanup metadata, not authentication authority; SIGKILL/runner/native cleanup correctness remains unproven. The independently approved bundle must include the changed native helper. Added mocked checks cover tracking-marker presence/absence with the fixed environment and synthetic termination passing through real owned-child kill/wait/descriptor/selector cleanup logic. No process was spawned and no real signal was installed or sent. Local QA: 86 mocked/static tests passed (39 helper, 30 caller/workflow, 8 policy, 9 scope), focused actionlint and git diff --check passed. Exact-head CI:
The full ten-file diff remains infrastructure-only, with mixed application/key changes retaining normal CI. Workflow gates, live inventory/native-fixture/application definitions and activation settings are unchanged. No dispatch, runner operation, download, guest work, application build/package/release or merge was performed. #73 and #77 remain draft/open/unmerged. Remaining gates: verify live selected-workflow/group/environment controls and fresh final SHA/bundle before any separately authorized execution. Actual mirror/GPG/filesystem behavior, cancellation/child cleanup, evidence retrieval and media authentication remain native proof obligations. Local diagnostic completion is not authentication safety, cleanup proof, runner deregistration or whole-host restoration. |
|
Bounded execution authorization packet — preparation only, NOT authorization to execute PR #77 merged as 05a8142. Post-merge policy/mock checks succeeded (39098701813), and the scope check succeeded with checks/native/update-trust/release skipped (39098701877). PR #73 remains draft/unmerged. No activation settings were changed, runner registered/started, workflow dispatched, media acquired, guest run, application built/packaged or release prepared. Exact approved-code candidates (do not configure yet):
Execution is BLOCKED pending the following live controls and new explicit permission:
The one proposed run, only after explicit authorization, would permit:
Proposed future ordering (not performed): Pass criteria: one genuine signature/digest-authenticated Cinnamon ISO, durable run-bound evidence and an offline non-executed plan, plus separately verified job and runner closeout. No success claim for usable console/guest/updater behavior. Native authentication, filesystem, cancellation/cleanup and evidence retrieval still need observation; mocks are not proof of those properties. This packet authorizes NOTHING by itself. Current instruction remains preparation only: leave activation disabled, no runner operations/dispatch/download/guest execution, no #73 merge, no application builds/packages/releases. Reviewed source-file hashes:
|
|
Stopped native preparation run 39099802574 / job 117593558727:
Native authentication: NotRun; preparation result: failed/stopped. Source-descriptor cleanup is not independently proven by this log; whole-host restoration remains unproven. Retain checkout/diagnostic files, do not change permissions or delete uncertain state. The next minimal diagnostic is read-only UID/mode/owner inspection of the source path and its ancestors, not a rerun. No package/guest/update/build/release operation or #73 merge is authorized. |
Prepares a separate, disabled-by-default manual workflow for the reviewed Mint media-authentication and offline console-plan helpers. It does not activate or execute them.
The fixed workflow requires main source/workflow SHA approval, an independently reviewed caller/helper/key bundle hash, separate preparation flags, protected mint-host-access approval and the restricted sde-mint-isolated runner group. No dispatch inputs, arbitrary URLs/commands, reruns, guest launch, disk creation or package executor are exposed. The caller verifies bounded source snapshots before loading ports, runs the merged signature/digest/resource/ownership gates, and emits bounded sanitized evidence after descriptor closure. TERM/INT handling, source descriptor cleanup, complete manifest reads and failure sanitization include reviewed fault checks.
Automatic hosted CI runs mocked/static checks and lint only. Exact scope additions cover the workflow/caller/test; the complete ten-file diff skips application jobs on PR and future infrastructure-only main pushes. Mixed application, signing-key and unknown paths retain normal application CI. Existing live inventory, native-fixture and application workflow definitions are unchanged.
Validation: 86 local mocked/static checks passed (39 helper, 30 caller/workflow, 8 policy, 9 scope), focused actionlint and whitespace checks passed. Native cancellation/child cleanup, mirror/GPG/filesystem behavior, evidence retrieval and actual authentication remain unvalidated. Historical #72 findings are preserved.
Keep this PR draft and unmerged. No activation settings, runner registration/start, dispatch, media downloads, guest execution, PR #73 merge, application builds/packages or releases are authorized. Before any future run, the exact runner-group workflow restriction and environment controls must be reviewed separately; labels are routing only, and prior self-review exceptions are not reusable authorization.
Review follow-up 5ac0559 preserves the runner cleanup marker in the fixed curl/GPG child environment without propagating unrelated credentials or injection settings. Mocked coverage verifies marker presence/absence and synthetic termination through child cleanup. This is a best-effort fallback, not native cancellation or authentication proof. Dispatch gates and activation settings are unchanged.