-
Notifications
You must be signed in to change notification settings - Fork 368
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(TPG>=5.36)!: add confidential_instance_type to instance_template module #416
feat(TPG>=5.36)!: add confidential_instance_type to instance_template module #416
Conversation
/gcbrun |
@apeabody Could you PTAL ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @arthurlapertosa!
Please see the notes below:
/gcbrun |
FYI @arthurlapertosa
|
Sorry, I forgot to update the documentation after changing the default value to |
/gcbrun |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SEV vs SEV_SNP
test/integration/confidential_instance_template/confidential_instance_template_test.go
Outdated
Show resolved
Hide resolved
@apeabody could you please trigger the workflows? |
/gcbrun |
Thanks @arthurlapertosa! - Can you resolve the appropriate open comments? |
@erlanderlo could you please take a look at the open comments? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. one minor item
@apeabody all open comments addressed. Please take another look! |
/gcbrun |
add confidential_instance_type to instance_template module, with example and tests