-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Modify labels fields for DCL resources #8563
Modify labels fields for DCL resources #8563
Conversation
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
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. Terraform GA: Diff ( 53 files changed, 957 insertions(+), 181 deletions(-)) Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_assured_workloads_workload" "primary" {
effective_labels = # value needed
}
Resource: resource "google_cloudbuild_worker_pool" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_cloudbuildv2_connection" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_cloudbuildv2_repository" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_clouddeploy_delivery_pipeline" "primary" {
effective_annotations = # value needed
effective_labels = # value needed
}
Resource: resource "google_clouddeploy_target" "primary" {
effective_annotations = # value needed
effective_labels = # value needed
}
Resource: resource "google_container_aws_cluster" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_container_aws_node_pool" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_container_azure_cluster" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_container_azure_node_pool" "primary" {
effective_annotations = # value needed
}
Resource: resource "google_dataplex_asset" "primary" {
effective_labels = # value needed
}
Resource: resource "google_dataplex_lake" "primary" {
effective_labels = # value needed
}
Resource: resource "google_dataplex_zone" "primary" {
effective_labels = # value needed
}
Resource: resource "google_dataproc_workflow_template" "primary" {
effective_labels = # value needed
}
Resource: resource "google_eventarc_trigger" "primary" {
effective_labels = # value needed
}
Resource: resource "google_network_connectivity_hub" "primary" {
effective_labels = # value needed
}
Resource: resource "google_network_connectivity_spoke" "primary" {
effective_labels = # value needed
}
Resource: resource "google_privateca_certificate_template" "primary" {
effective_labels = # value needed
}
Resource: resource "google_recaptcha_enterprise_key" "primary" {
effective_labels = # value needed
}
|
Tests analyticsTotal tests:
|
4b0eb05
to
a8b9c32
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. Terraform GA: Diff ( 54 files changed, 824 insertions(+), 184 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 3 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccFolderIamPolicy_basic|TestAccSpannerInstanceIamPolicy|TestAccCloudbuildWorkerPool_basic |
Rerun these tests in REPLAYING mode to catch issues$\textcolor{green}{\textsf{All tests passed!}} |
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. Terraform GA: Diff ( 55 files changed, 831 insertions(+), 185 deletions(-)) |
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. Terraform GA: Diff ( 55 files changed, 909 insertions(+), 206 deletions(-)) |
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. Terraform GA: Diff ( 55 files changed, 890 insertions(+), 206 deletions(-)) |
Tests analyticsTotal tests: 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 testsTestAccDataprocWorkflowTemplate_basic|TestAccDataplexAsset_BasicAssetHandWritten |
Rerun these tests in REPLAYING mode to catch issues$\textcolor{green}{\textsf{All tests passed!}} |
Same here, can you add upgrade guide entries to the main branch? |
Thanks for the reminder. Created the PR-8608 to add the first version of guide for labels. Will add more details to the guide with more PRs being pushed to the 5.0.0 branch. |
Transform the top level "labels" and "annotations" fields for DCL resources. Currently all of labels fields for DCL resources are in the top level, so just handle this case now.
effective_labels
andeffective_annotations
Add
labels
orannotations
to acceptance tests for some DCL resources, which don't cover these filed in the tests. The tests for these fields already exist for most DCL resources having the labels fields.If this PR is for Terraform, I acknowledge that I have:
make test
andmake lint
in the generated providers to ensure it passes unit and linter tests.Release Note Template for Downstream PRs (will be copied)