Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@yawangwang
Copy link
Collaborator

@yawangwang yawangwang commented Jul 26, 2024

This PR introduces the following changes:

  1. Upgrade confidentialcomputing client library to the latest version v1.6.0 with SEVSNP and TDX support.
  2. Convert sevsnp proto from go-sev-guest repo to API protos such that new confidentialcomputing client library can understand.
  3. Update some deprecated methods to the recommended version.
  4. Manual e2e tests - ran gotpm binary from a SNP machine to fetch token from the staging endpoint. Token successfully returned with
    {"aud":"https://sts.googleapis.com","exp":1722293238,"iat":1722289638,"iss":"https://staging-confidentialcomputing.sandbox.googleapis.com","nbf":1722289638,"sub":"https://www.googleapis.com/compute/v1/projects/yawangwang-test-project/zones/us-central1-a/instances/ubuntu2204-sevsnp-cvm2","eat_profile":"https://cloud.google.com/confidential-computing/confidential-vm/docs/token-claims","secboot":true,"oemid":11129,"hwmodel":"GCP_AMD_SEV_SNP","swname":"GCE","attester_tcb":["AMD","GOOGLE"],"dbgstat":"enabled","submods":{},"google_service_accounts":["[email protected]"]}

@yawangwang yawangwang force-pushed the translate_snp_to_api_proto branch from 308c2ce to ea7f086 Compare July 26, 2024 16:37
@yawangwang yawangwang marked this pull request as ready for review July 26, 2024 16:43
@yawangwang yawangwang requested a review from JoshuaKrstic July 26, 2024 16:44
@yawangwang yawangwang force-pushed the translate_snp_to_api_proto branch 4 times, most recently from a25fad4 to c907ed1 Compare July 26, 2024 19:31
@yawangwang yawangwang force-pushed the translate_snp_to_api_proto branch from c907ed1 to c484fc2 Compare July 26, 2024 20:05
@yawangwang
Copy link
Collaborator Author

/gcbrun

@yawangwang yawangwang merged commit b30d373 into google:main Aug 13, 2024
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.

3 participants