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

Move RCAR handshake from AA to CDH #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

Xynnn007
Copy link
Member

@Xynnn007 Xynnn007 commented Jan 8, 2025

This is to support such a scenario:

  1. We have an AA running aside, and have different one-shot CDH to connect to different KBSes.

This feature enables dependencies to leverage ttrpc capabilities to
connect to AA via ttrpc, thus `aa_ttrpc` is a better name for future
extension.

Signed-off-by: Xynnn007 <[email protected]>
AAEvidenceProvider gets evidence via ttrpc from AA. This patch also does
some refactoring upon the code structure of ttrpc to avoid duplication
of ttrpc files.

Signed-off-by: Xynnn007 <[email protected]>
When we use one AA and multiple CDH to connect to KBSes in one
confidential VM, AATokenProvider requires the KBS be the same. If we
use AAEvidenceProvider, the different target KBS address can be
specified in CDH's config and share a same AA.

This will also make the workflow simpler. Before this commit, if we want
to get a confidential resource from KBS, AA should connect to a KBS to
get a token. Then CDH get the token from AA. Then CDH access the KBS.

After this commit, only CDH will access the KBS.

This change will require the KBS that does RCAR and stores resource be
the same in CoCo, s.t. background check model. which is up to now
nearly all the deployments we have met.

This will not break the passport model because kbs_protocol crate still
provides a way for developers to integrate either passport model or
background check model in their own code.

Signed-off-by: Xynnn007 <[email protected]>
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