Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

peer-pods: pass policy hash via userdata #941

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

burgerdev
Copy link
Contributor

  1. Add some plumbing to forward the agent policy to remote hypervisor
  2. Add a policy-hash field to the agent config that's set via userdata.
  3. Measure the agent config into PCR10.
    • The daemon config contains a bunch of hard-to-predict network config, but we should double-check whether it could be an attack vector.
  4. Add hash-from-agent-config as an option in the hash-verification.

@burgerdev burgerdev added the no changelog PRs not listed in the release notes label Oct 18, 2024
@burgerdev burgerdev force-pushed the burgerdev/user-data branch 2 times, most recently from aeacb19 to d6dd9c1 Compare October 18, 2024 11:36
@burgerdev burgerdev marked this pull request as ready for review October 24, 2024 15:24
@burgerdev
Copy link
Contributor Author

I tested this (with runtime binaries and CAA container image manually replaced) and got a pod that verifies the policy and measures PCR10 as expected.

@Freax13 Freax13 self-requested a review October 25, 2024 12:00
@3u13r 3u13r mentioned this pull request Oct 29, 2024
3 tasks
@burgerdev burgerdev changed the base branch from main to burgerdev/clean-patches November 4, 2024 21:55
@burgerdev burgerdev force-pushed the burgerdev/clean-patches branch 2 times, most recently from d11aa21 to 3d721ad Compare November 5, 2024 09:44
Base automatically changed from burgerdev/clean-patches to main November 5, 2024 10:37
@burgerdev
Copy link
Contributor Author

Thanks for the re-review! I'm holding this PR back until we have a working peerpods setup on main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants