Skip to content

Unify per-service KV sync for all delivery modes - #305

Merged
sehkone merged 1 commit into
mainfrom
sehkone/issue-299
Feb 22, 2026
Merged

Unify per-service KV sync for all delivery modes#305
sehkone merged 1 commit into
mainfrom
sehkone/issue-299

Conversation

@sehkone

@sehkone sehkone commented Feb 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove DeliveryMode::RemoteBootstrap filter from per-service KV writes (eab, http_responder_hmac, trust) so LocalFile services also receive per-service KV data
  • Keep secret_id KV write conditional on RemoteBootstrap only (LocalFile stores it in local files)
  • Rename internal functions/types to reflect delivery-mode-agnostic semantics (remoteservice)

Test plan

  • cargo clippy -- -D warnings passes
  • cargo fmt -- --check passes
  • cargo test --test bootroot_rotate — 14/14 passed
  • cargo test --test bootroot_service — 17/17 passed
  • E2E tests do not assert on changed output strings; no E2E changes needed

Closes #299

@sehkone
sehkone force-pushed the sehkone/issue-299 branch 3 times, most recently from 60151e8 to 98696bc Compare February 22, 2026 11:03
Remove the RemoteBootstrap filter from per-service KV writes
(eab, http_responder_hmac, trust) so LocalFile services also
receive per-service KV data. This is a prerequisite for dynamic
ctmpl templates that read from per-service KV.

The secret_id KV write remains conditional on RemoteBootstrap,
since LocalFile services store secret_id in local files.

Closes #299
@sehkone
sehkone merged commit 5e55415 into main Feb 22, 2026
13 checks passed
@sehkone
sehkone deleted the sehkone/issue-299 branch February 22, 2026 11:22
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.

Unify per-service KV sync for both LocalFile and RemoteBootstrap services

1 participant