Add WorkloadIdentityConfig to Compute Instance and InstanceTemplate#18121
Add WorkloadIdentityConfig to Compute Instance and InstanceTemplate#18121draksbahijakub-google wants to merge 2 commits into
Conversation
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @BBBmau, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit e23f870: Diff reportYour PR generated the following diffs in downstream repositories:
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_compute_instance_from_machine_image" "primary" {
workload_identity_config {
identity = # value needed
identity_certificate_enabled = # value needed
}
}
Resource: resource "google_compute_instance_from_template" "primary" {
workload_identity_config {
identity = # value needed
identity_certificate_enabled = # value needed
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 5 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
🟢 All tests passed! View the recording VCR build log or the debug logs folder for detailed results. @draksbahijakub-google, @BBBmau VCR tests complete for e23f870! |
e23f870 to
22d00f9
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 22d00f9: Diff reportYour PR generated the following diffs in downstream repositories:
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_compute_instance_from_machine_image" "primary" {
workload_identity_config {
identity = # value needed
identity_certificate_enabled = # value needed
}
}
Resource: resource "google_compute_instance_from_template" "primary" {
workload_identity_config {
identity = # value needed
identity_certificate_enabled = # value needed
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 🟢 All tests passed in Replaying mode! No Recording was needed. View the replaying VCR build log @draksbahijakub-google, @BBBmau VCR tests complete for 22d00f9! |
BBBmau
left a comment
There was a problem hiding this comment.
please address the missing fields in the test report as well as the missing fields in documentation
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 760b69f: Diff reportYour PR generated the following diffs in downstream repositories:
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_compute_instance_from_template" "primary" {
workload_identity_config {
identity = # value needed
identity_certificate_enabled = # value needed
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 2 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
🟢 All tests passed! View the recording VCR build log or the debug logs folder for detailed results. @draksbahijakub-google, @BBBmau VCR tests complete for 760b69f! |
760b69f to
233a0d3
Compare
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 233a0d3: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 2 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
🟢 All tests passed! View the recording VCR build log or the debug logs folder for detailed results. @draksbahijakub-google, @BBBmau VCR tests complete for 233a0d3! |
@BBBmau Added missing tests. When it comes to detected missing docs - I verified that in both cases these docs are not listing all instances properties, but sending users to check general instances documentation for supported properties, so I left these 2 docs not changed to stay consistent. |
Add WorkloadIdentityConfig to Compute Instance and InstanceTemplate
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.