Skip to content

Makefile: drop stale test-chain / test-persist targets - #4

Closed
HarryR wants to merge 1 commit into
mainfrom
remove-stale-chain-persist-tests
Closed

HarryR wants to merge 1 commit into
mainfrom
remove-stale-chain-persist-tests

Conversation

@HarryR

@HarryR HarryR commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • test-chain-% and test-persist-% booted a bare example binary as stage2: derive-probe (TPM derive determinism plus PCR15 key binding) and blockdev-probe (GPT rebuild, dm-crypt and a sentinel on a grown disk).
  • 9167e6e (XTS/meta_bg rework) deleted those probes, along with reboot-probe and tpm_derive::read_all_pcrs, but left the targets. Both have failed at cargo build --example ever since.
  • test-pivot is the full-chain test (stage0 -> UKI -> stage1 -> packed loader -> pivot-init) and covers their ground through the real loader. It is untouched.
  • Kept the shared STAGE0_* / STAGE1_* / SERVE_HOST / PERSIST_GROW, which test-pivot uses. Dropped the PAYLOAD / PERSIST_PAYLOAD defaults, which nothing else uses.
  • PCR-BINDING.md: removed the "How to reproduce / investigate" section, which pointed at the deleted target and probe. If that analysis is needed again, the test can be recreated.

Test plan

  • No remaining references to the removed targets, payloads or read_all_pcrs
  • make -n test-pivot-x86_64 still resolves
  • make ci

🤖 Generated with Claude Code

Both booted a bare example as stage2 (derive-probe, blockdev-probe);
9167e6e deleted those probes (and reboot-probe, read_all_pcrs) but left
the targets, so both have failed at `cargo build --example` since.
test-pivot is the full-chain test and covers their ground through the
real loader. Keep the shared STAGE0_*/STAGE1_*/SERVE_HOST/PERSIST_GROW
it uses; drop the PAYLOAD/PERSIST_PAYLOAD defaults.

PCR-BINDING.md: remove the reproduce section, which pointed at the
deleted target and probe.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@HarryR

HarryR commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator Author

Folded into #2 (one PR per repo for this changeset).

@HarryR HarryR closed this Nov 4, 2026
@HarryR
HarryR deleted the remove-stale-chain-persist-tests branch November 4, 2026 13:44
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