-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Apply new labels model to container cluster and edgenetwork resources #11320
Apply new labels model to container cluster and edgenetwork resources #11320
Conversation
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_edgenetwork_network" "primary" {
labels = # value needed
}
Resource: resource "google_edgenetwork_subnet" "primary" {
labels = # value needed
}
|
Tests analyticsTotal tests: 3877 Click here to see the affected service packages
Tests were added that are skipped in VCR:
Action takenFound 81 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_edgenetwork_network" "primary" {
labels = # value needed
}
Resource: resource "google_edgenetwork_subnet" "primary" {
labels = # value needed
}
|
Tests analyticsTotal tests: 3877 Click here to see the affected service packages
Tests were added that are skipped in VCR:
Action takenFound 19 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
f6e6d8f
to
7811bc5
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_edgenetwork_network" "primary" {
labels = # value needed
}
Resource: resource "google_edgenetwork_subnet" "primary" {
labels = # value needed
}
|
Tests analyticsTotal tests: 3877 Click here to see the affected service packages
Tests were added that are skipped in VCR:
Action takenFound 11 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
7811bc5
to
91009b9
Compare
91009b9
to
0732dfa
Compare
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_edgenetwork_network" "primary" {
labels = # value needed
}
Resource: resource "google_edgenetwork_subnet" "primary" {
labels = # value needed
}
|
1 similar comment
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_edgenetwork_network" "primary" {
labels = # value needed
}
Resource: resource "google_edgenetwork_subnet" "primary" {
labels = # value needed
}
|
Tests analyticsTotal tests: 3886 Click here to see the affected service packages
Tests were added that are skipped in VCR:
Action takenFound 13 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
1 similar comment
Tests analyticsTotal tests: 3886 Click here to see the affected service packages
Tests were added that are skipped in VCR:
Action takenFound 13 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
|
@rileykarson This PR has been waiting for review for 3 weekdays. Please take a look! Use the label |
75ce1f6
into
GoogleCloudPlatform:FEATURE-BRANCH-major-release-6.0.0
Part of hashicorp/terraform-provider-google#16219
Apply new labels model to
google_container_cluster
,google_edgenetwork_network
andgoogle_edgenetwork_subnet
Release Note Template for Downstream PRs (will be copied)