-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 the "google_compute_global_address" datasource #759
Conversation
Acceptance result: TF_ACC=1 go test $(go list ./... | grep -v 'vendor') -v -run=TestAccDataSourceComputeGlobalAddress -timeout 120m
? github.com/terraform-providers/terraform-provider-google [no test files]
=== RUN TestAccDataSourceComputeGlobalAddress
--- PASS: TestAccDataSourceComputeGlobalAddress (26.33s)
PASS
ok github.com/terraform-providers/terraform-provider-google/google 26.362s
testing: warning: no tests to run
PASS
ok github.com/terraform-providers/terraform-provider-google/scripts 0.028s [no tests to run] |
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.
Great work. Only one suggestion.
} | ||
} | ||
|
||
func testAccCheckDataSourceComputeGlobalAddressDestroy(resource_name string) resource.TestCheckFunc { |
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.
You should be able to get the same result by using this method instead: testAccCheckComputeGlobalAddressDestroy
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.
i've "fixup" the code
--- PASS: TestAccDnsRecordSet_basic (22.47s) |
* Add the "compute_global_address" datasource * Add a basic test for the "compute_global_address" datasource * Include the "compute_global_address" in the provider * Add docs for the "compute_global_address" datasource
Send external gateway interface id if empty
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
Same as #748. This PR will be useful for users that want to expose the public IP of their HTTP load balancers.