terraform-google-modules / terraform-google-vm

Provisions VMs in Google Cloud
https://registry.terraform.io/modules/terraform-google-modules/vm/google
Apache License 2.0
220 stars 368 forks source link

feat(TPG>=5.36)!: add confidential_instance_type to instance_template module #416

Closed arthurlapertosa closed 1 month ago

arthurlapertosa commented 1 month ago

add confidential_instance_type to instance_template module, with example and tests

daniel-cit commented 1 month ago

/gcbrun

daniel-cit commented 1 month ago

@apeabody Could you PTAL ?

apeabody commented 1 month ago

/gcbrun

apeabody commented 1 month ago

FYI @arthurlapertosa

Checking for documentation generation
diff -r '--exclude=.terraform' '--exclude=.kitchen' '--exclude=autogen' '--exclude=*.tfvars' '--exclude=*metadata.yaml' /workspace/modules/instance_template/README.md /tmp/tmp.4PKCCzT0D4/generate_docs/workspace/modules/instance_template/README.md
23c23
< | confidential\_instance\_type | Defines the confidential computing technology the instance uses. If this is set to "SEV\_SNP", var.min\_cpu\_platform will be automatically set to "AMD Milan". | `string` | `""` | no |
---
> | confidential\_instance\_type | Defines the confidential computing technology the instance uses. If this is set to "SEV\_SNP", var.min\_cpu\_platform will be automatically set to "AMD Milan". | `string` | `null` | no |
Error: Documentation generation has not been run, please run the
'make docker_generate_docs' command and commit the above changes.
arthurlapertosa commented 1 month ago

FYI @arthurlapertosa

Checking for documentation generation
diff -r '--exclude=.terraform' '--exclude=.kitchen' '--exclude=autogen' '--exclude=*.tfvars' '--exclude=*metadata.yaml' /workspace/modules/instance_template/README.md /tmp/tmp.4PKCCzT0D4/generate_docs/workspace/modules/instance_template/README.md
23c23
< | confidential\_instance\_type | Defines the confidential computing technology the instance uses. If this is set to "SEV\_SNP", var.min\_cpu\_platform will be automatically set to "AMD Milan". | `string` | `""` | no |
---
> | confidential\_instance\_type | Defines the confidential computing technology the instance uses. If this is set to "SEV\_SNP", var.min\_cpu\_platform will be automatically set to "AMD Milan". | `string` | `null` | no |
Error: Documentation generation has not been run, please run the
'make docker_generate_docs' command and commit the above changes.

Sorry, I forgot to update the documentation after changing the default value to null. Just did it.

apeabody commented 1 month ago

/gcbrun

arthurlapertosa commented 1 month ago

@apeabody could you please trigger the workflows?

apeabody commented 1 month ago

/gcbrun

apeabody commented 1 month ago

Thanks @arthurlapertosa! - Can you resolve the appropriate open comments?

arthurlapertosa commented 1 month ago

@erlanderlo could you please take a look at the open comments?

arthurlapertosa commented 1 month ago

@apeabody all open comments addressed. Please take another look!

apeabody commented 1 month ago

/gcbrun