Skip to content

Read apt's *** marker when the installed version is the candidate - #13

Merged
marcos-mendez merged 3 commits into
masterfrom
fix/policy-installed-candidate
Sep 30, 2026
Merged

marcos-mendez merged 3 commits into
masterfrom
fix/policy-installed-candidate

Conversation

@marcos-mendez

Copy link
Copy Markdown
Collaborator

The boot gate's first run that reached the archive (after Keel-Linux/tracker#5 was fixed on the public services VM, 2026-09-30) failed with the candidate inithooks 2.3.6+keel5 does not come from https://archive.keellinux.org at priority 1001, although it does. apt prints the installed version as *** 2.3.6+keel5 1001, with both sources under it, and bt_policy_verdict compared the version to the first field. The old fixture wrote two separate lines, which apt never does for an installed candidate. The new test is apt's real output from that run; it fails on the old code and passes on the new.

navigator added 3 commits September 30, 2026 02:22
bt_policy_verdict looked for the candidate as the first field of its version line, but apt prints the installed version as ' *** 2.3.6+keel5 1001', so a current appliance, whose installed version is the candidate, failed the check. The gate never reached it until tracker#5 was fixed on 2026-09-30; its first run past the archive failed here. The old fixture wrote two lines where apt writes one; the new test uses apt's real output and fails on the old code.
From the review of #13: the version line's number is the highest priority of its sources, so matching it proved nothing about where the candidate comes from; any repository claiming our origin gets our pin. The parse now requires '<pin> <uri>' inside the candidate's block, and a new test refuses an installed candidate from another source at the same pin.
kcov counted the lines of the inline awk program as unexecuted bash and put boot-test-lib.sh under the 97 percent gate. The program moves to tests/lib/candidate-source.awk, beside the library, with its explanation, and the library finds it through BT_LIB_DIR.
@marcos-mendez
marcos-mendez merged commit b60f091 into master Sep 30, 2026
3 checks passed
@marcos-mendez
marcos-mendez deleted the fix/policy-installed-candidate branch September 30, 2026 02:36
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.

1 participant