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

Add advanced machine feature options #236

Merged
merged 1 commit into from
Feb 14, 2022
Merged

Add advanced machine feature options #236

merged 1 commit into from
Feb 14, 2022

Conversation

timstoop
Copy link
Contributor

Allow creation of instance templates that either have nested virtualization enabled or support SMT.

@comment-bot-dev
Copy link

comment-bot-dev commented Feb 14, 2022

Thanks for the PR! 🚀
✅ Lint checks have passed.

@timstoop
Copy link
Contributor Author

I have no permission to check why the test is failing.

@morgante
Copy link
Contributor

estInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │ Error: Error creating instance: googleapi: Error 400: The selected machine type (n1-standard-1) does not support SMT settings., badRequest
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │ 
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │   with module.instance_simple.module.compute_instance.google_compute_instance_from_template.compute_instance[2],
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │   on ../../../modules/compute_instance/main.tf line 41, in resource "google_compute_instance_from_template" "compute_instance":
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │   41: resource "google_compute_instance_from_template" "compute_instance" {
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │ 
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: ╵
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: ╷
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │ Error: Error creating instance: googleapi: Error 400: The selected machine type (n1-standard-1) does not support SMT settings., badRequest
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │ 
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │   with module.instance_simple.module.compute_instance.google_compute_instance_from_template.compute_instance[0],
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │   on ../../../modules/compute_instance/main.tf line 41, in resource "google_compute_instance_from_template" "compute_instance":
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │   41: resource "google_compute_instance_from_template" "compute_instance" {
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: │ 
TestInstanceSimpleModule 2022-02-14T17:35:04Z command.go:179: ╵
TestInstanceSimpleModule 2022-02-14T17:35:04Z retry.go:99: Returning due to fatal error: FatalError{Underlying: error while running command: exit status 1; ╷
│ Error: Error creating instance: googleapi: Error 400: The selected machine type (n1-standard-1) does not support SMT settings., badRequest
│ 
│   with module.instance_simple.module.compute_instance.google_compute_instance_from_template.compute_instance[1],
│   on ../../../modules/compute_instance/main.tf line 41, in resource "google_compute_instance_from_template" "compute_instance":
│   41: resource "google_compute_instance_from_template" "compute_instance" {
│ 
╵
╷
│ Error: Error creating instance: googleapi: Error 400: The selected machine type (n1-standard-1) does not support SMT settings., badRequest
│ 
│   with module.instance_simple.module.compute_instance.google_compute_instance_from_template.compute_instance[3],
│   on ../../../modules/compute_instance/main.tf line 41, in resource "google_compute_instance_from_template" "compute_instance":
│   41: resource "google_compute_instance_from_template" "compute_instance" {
│ 
╵
╷
│ Error: Error creating instance: googleapi: Error 400: The selected machine type (n1-standard-1) does not support SMT settings., badRequest
│ 
│   with module.instance_simple.module.compute_instance.google_compute_instance_from_template.compute_instance[2],
│   on ../../../modules/compute_instance/main.tf line 41, in resource "google_compute_instance_from_template" "compute_instance":
│   41: resource "google_compute_instance_from_template" "compute_instance" {
│ 
╵
╷
│ Error: Error creating instance: googleapi: Error 400: The selected machine type (n1-standard-1) does not support SMT settings., badRequest
│ 
│   with module.instance_simple.module.compute_instance.google_compute_instance_from_template.compute_instance[0],
│   on ../../../modules/compute_instance/main.tf line 41, in resource "google_compute_instance_from_template" "compute_instance":
│   41: resource "google_compute_instance_from_template" "compute_instance" {
│ 
╵}

@timstoop
Copy link
Contributor Author

All checks passed, going to remove the unnecessary whiteline at the end of the file in my next push and then leave it for your approval!

tags = var.tags
labels = var.labels
access_config = [local.access_config]
enable_nested_virtualization = var.enable_nested_virtualization
Copy link
Contributor

Choose a reason for hiding this comment

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

For future reference, it's encouraged to hard code values like this in examples.

@morgante morgante merged commit 4f21330 into terraform-google-modules:master Feb 14, 2022
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