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 3 default_route_action
subfields to google_compute_region_url_map
resource
#6674
Add 3 default_route_action
subfields to google_compute_region_url_map
resource
#6674
Conversation
Hello! I am a robot who works on Magic Modules PRs. I've detected that you're a community contributor. @ScottSuarez, a repository maintainer, has been assigned to assist you and help review your changes. ❓ First time contributing? Click here for more detailsYour assigned reviewer will help review your code by:
You can help make sure that review is quick by running local tests and ensuring they're passing in between each push you make to your PR's branch. Also, try to leave a comment with each push you make, as pushes generally don't generate emails. If your reviewer doesn't get back to you within a week after your most recent change, please feel free to leave a comment on the issue asking them to take a look! In the absence of a dedicated review dashboard most maintainers manage their pending reviews through email, and those will sometimes get lost in their inbox. |
Hello! I am a robot who works on Magic Modules PRs. I've detected that you're a community contributor. @slevenick, a repository maintainer, has been assigned to assist you and help review your changes. ❓ First time contributing? Click here for more detailsYour assigned reviewer will help review your code by:
You can help make sure that review is quick by running local tests and ensuring they're passing in between each push you make to your PR's branch. Also, try to leave a comment with each push you make, as pushes generally don't generate emails. If your reviewer doesn't get back to you within a week after your most recent change, please feel free to leave a comment on the issue asking them to take a look! In the absence of a dedicated review dashboard most maintainers manage their pending reviews through email, and those will sometimes get lost in their inbox. |
…_region_url_map` resource Enforce mutual exclusive relationships between `defaultRouteAction.weightedBackendServices[]`, `defaultUrlRedirect` and `defaultService`
…uding `defaultRouteAction.weightedBackendServices[]`
…p` resource, update generated test
…n_url_map` resource, update generated test
13937e4
to
919c511
Compare
Hi! I'm the modular magician. Your PR generated some diffs in downstreams - here they are. Diff report:Terraform GA: Diff ( 3 files changed, 1227 insertions(+), 50 deletions(-)) |
Tests analyticsTotal tests: Action takenTriggering VCR tests in RECORDING mode for the tests that failed during VCR. Click here to see the failed testsTestAccFirebaserulesRelease_BasicRelease|TestAccComputeInstance_soleTenantNodeAffinities|TestAccComputeRegionUrlMap_regionUrlMapDefaultRouteActionExample|TestAccBigtableAppProfile_bigtableAppProfileAnyclusterExample|TestAccBigtableAppProfile_bigtableAppProfileMulticlusterExample|TestAccCGCSnippet_eventarcWorkflowsExample |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
I'm removing reviewers while the PR is in draft to try and avoid spamming y'all 🙏 |
default_route_action.weighted_backend_services
field to google_compute_region_url_map
resourcedefault_route_action
subfields to google_compute_region_url_map
resource
Hello! I am a robot who works on Magic Modules PRs. I've detected that you're a community contributor. @megan07, a repository maintainer, has been assigned to assist you and help review your changes. ❓ First time contributing? Click here for more detailsYour assigned reviewer will help review your code by:
You can help make sure that review is quick by running local tests and ensuring they're passing in between each push you make to your PR's branch. Also, try to leave a comment with each push you make, as pushes generally don't generate emails. If your reviewer doesn't get back to you within a week after your most recent change, please feel free to leave a comment on the issue asking them to take a look! In the absence of a dedicated review dashboard most maintainers manage their pending reviews through email, and those will sometimes get lost in their inbox. |
mmv1/templates/terraform/examples/region_url_map_default_route_action.tf.erb
Show resolved
Hide resolved
name = "<%= ctx[:vars]['region_url_map_name'] %>" | ||
description = "a description" | ||
|
||
default_route_action { |
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.
Can you write a test that updates several of the fields in here (particularly the lists like retry_policy & weighted_backend_services) to add items, remove items and/or remove the list entirely?
Updates on these sorts of fields can be complicated and produce perma-diffs for users depending on how the API is implemented, and we lose a lot of possible fixes once we release the resource with a certain implementation
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.
Will do soon, after the choice about using sets is clearer! after I make the PR conform to google_compute_url_map
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.
Tests added in 06750b7
…d_services` blocks and nested blocks
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 ( 4 files changed, 1421 insertions(+), 61 deletions(-)) |
Tests analyticsTotal tests: Action takenTriggering VCR tests in RECORDING mode for the tests that failed during VCR. Click here to see the failed testsTestAccFirebaserulesRelease_BasicRelease|TestAccComputeRegionUrlMap_regionUrlMapDefaultRouteActionExample|TestAccComputeRegionUrlMap_errorWhenUseID |
Tests passed during RECORDING mode: Tests failed during RECORDING mode: Please fix these to complete your PR |
…ror_policy`, and`weighted_backend_services` fields within `default_route_action`
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 ( 4 files changed, 1546 insertions(+), 49 deletions(-)) |
…ion of this resource
…default_route_action`, add field to `retryPolicy`
This resource is a bit of a pain and this PR is a bit messy, apologies! I've made changes so that this PR adds fields to
|
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 ( 4 files changed, 1553 insertions(+), 49 deletions(-)) |
Tests analyticsTotal tests: Action takenTriggering VCR tests in RECORDING mode for the tests that failed during VCR. Click here to see the failed testsTestAccFirebaserulesRelease_BasicRelease|TestAccComputeRegionUrlMap_defaultRouteAction_full_update|TestAccComputeForwardingRule_update |
Thanks! |
…map` resource (GoogleCloudPlatform#6674) * Add `defaultRouteAction.weightedBackendServices[]` to `google_compute_region_url_map` resource Enforce mutual exclusive relationships between `defaultRouteAction.weightedBackendServices[]`, `defaultUrlRedirect` and `defaultService` * Add generated test for `google_compute_region_url_map` resource, including `defaultRouteAction.weightedBackendServices[]` * Add `defaultRouteAction.retryPolicy` to `google_compute_region_url_map` resource, update generated test * Add `defaultRouteAction.requestMirrorPolicy` to `google_compute_region_url_map` resource, update generated test * Add missing field descriptions * Add missing `properties` field from `requestMirrorPolicy` definition * Update example (an generated tests) to have multiple `weighted_backend_services` blocks and nested blocks * Convert `defaultRouteAction.weightedBackendServices` and `defaultRouteAction.weightedBackendServices.headerAction.*` lists to sets, add test showing issue with permadiff * Remove use of sets, to match global version of the resource * Add `conflicts` field between `default_url_redirect` and `default_route_action` To match UrlMap resource : https://github.com/hashicorp/magic-modules/blob/4f1ef3974f99d6a9efa95ea0284ee831cc63d2f5/mmv1/products/compute/api.yaml#L18556-L18557 * Remove validation for field not added in this PR * Update acceptance test to set and update `retry_policy`, `request_mirror_policy`, and`weighted_backend_services` fields within `default_route_action` * Add missing `at_least_one_of` fields, for parity with the global version of this resource * Update `at_least_one_of` field to reference fields currently within `default_route_action`, add field to `retryPolicy`
…map` resource (GoogleCloudPlatform#6674) * Add `defaultRouteAction.weightedBackendServices[]` to `google_compute_region_url_map` resource Enforce mutual exclusive relationships between `defaultRouteAction.weightedBackendServices[]`, `defaultUrlRedirect` and `defaultService` * Add generated test for `google_compute_region_url_map` resource, including `defaultRouteAction.weightedBackendServices[]` * Add `defaultRouteAction.retryPolicy` to `google_compute_region_url_map` resource, update generated test * Add `defaultRouteAction.requestMirrorPolicy` to `google_compute_region_url_map` resource, update generated test * Add missing field descriptions * Add missing `properties` field from `requestMirrorPolicy` definition * Update example (an generated tests) to have multiple `weighted_backend_services` blocks and nested blocks * Convert `defaultRouteAction.weightedBackendServices` and `defaultRouteAction.weightedBackendServices.headerAction.*` lists to sets, add test showing issue with permadiff * Remove use of sets, to match global version of the resource * Add `conflicts` field between `default_url_redirect` and `default_route_action` To match UrlMap resource : https://github.com/hashicorp/magic-modules/blob/4f1ef3974f99d6a9efa95ea0284ee831cc63d2f5/mmv1/products/compute/api.yaml#L18556-L18557 * Remove validation for field not added in this PR * Update acceptance test to set and update `retry_policy`, `request_mirror_policy`, and`weighted_backend_services` fields within `default_route_action` * Add missing `at_least_one_of` fields, for parity with the global version of this resource * Update `at_least_one_of` field to reference fields currently within `default_route_action`, add field to `retryPolicy`
…map` resource (GoogleCloudPlatform#6674) * Add `defaultRouteAction.weightedBackendServices[]` to `google_compute_region_url_map` resource Enforce mutual exclusive relationships between `defaultRouteAction.weightedBackendServices[]`, `defaultUrlRedirect` and `defaultService` * Add generated test for `google_compute_region_url_map` resource, including `defaultRouteAction.weightedBackendServices[]` * Add `defaultRouteAction.retryPolicy` to `google_compute_region_url_map` resource, update generated test * Add `defaultRouteAction.requestMirrorPolicy` to `google_compute_region_url_map` resource, update generated test * Add missing field descriptions * Add missing `properties` field from `requestMirrorPolicy` definition * Update example (an generated tests) to have multiple `weighted_backend_services` blocks and nested blocks * Convert `defaultRouteAction.weightedBackendServices` and `defaultRouteAction.weightedBackendServices.headerAction.*` lists to sets, add test showing issue with permadiff * Remove use of sets, to match global version of the resource * Add `conflicts` field between `default_url_redirect` and `default_route_action` To match UrlMap resource : https://github.com/hashicorp/magic-modules/blob/4f1ef3974f99d6a9efa95ea0284ee831cc63d2f5/mmv1/products/compute/api.yaml#L18556-L18557 * Remove validation for field not added in this PR * Update acceptance test to set and update `retry_policy`, `request_mirror_policy`, and`weighted_backend_services` fields within `default_route_action` * Add missing `at_least_one_of` fields, for parity with the global version of this resource * Update `at_least_one_of` field to reference fields currently within `default_route_action`, add field to `retryPolicy`
Closes hashicorp/terraform-provider-google#10660
This PR adds multiple new fields to the RegionUrlMap resource:
I've added the other fields under
defaultRouteAction
, but wanted to split the work across 2 PRs so it was easier to reviewIf this PR is for Terraform, I acknowledge that I have:
make test
andmake lint
to ensure it passes unit and linter tests.Release Note Template for Downstream PRs (will be copied)