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

feat: add confidential computing example #421

Conversation

arthurlapertosa
Copy link
Contributor

Add confidential computing example with custom SA and Cloud HSM encrypted disk.

@arthurlapertosa
Copy link
Contributor Author

/gcbrun

@arthurlapertosa arthurlapertosa marked this pull request as ready for review August 20, 2024 17:34
@arthurlapertosa arthurlapertosa requested a review from a team as a code owner August 20, 2024 17:34
@arthurlapertosa
Copy link
Contributor Author

@apeabody @erlanderlo could you please take a look?

@apeabody
Copy link
Contributor

/gcbrun

Copy link
Collaborator

@erlanderlo erlanderlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! mostly LGTM with a couple minor things to consider

examples/confidential_computing/outputs.tf Outdated Show resolved Hide resolved
examples/confidential_computing/variables.tf Outdated Show resolved Hide resolved
examples/confidential_computing/main.tf Outdated Show resolved Hide resolved
examples/confidential_computing/main.tf Show resolved Hide resolved
@arthurlapertosa
Copy link
Contributor Author

@apeabody could you please rerun the checks? I'm not sure why lint failed though (it's succeeding locally in my machine), would you have a clue? Thanks!

@apeabody
Copy link
Contributor

/gcbrun

Co-authored-by: Andrew Peabody <[email protected]>
@apeabody
Copy link
Contributor

/gcbrun

Copy link
Collaborator

@erlanderlo erlanderlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. One additional item wrt the SA.

@arthurlapertosa
Copy link
Contributor Author

@apeabody could you run /gcbrun again, please?

@apeabody
Copy link
Contributor

apeabody commented Sep 6, 2024

/gcbrun

Copy link
Collaborator

@erlanderlo erlanderlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for the contribution!

Copy link
Contributor

@apeabody apeabody left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed by @erlanderlo

@apeabody apeabody merged commit ddb6985 into terraform-google-modules:master Sep 9, 2024
4 checks passed
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