Skip to content
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 support for custom error response policies to net_lb_app_ext module #2916

Conversation

peter-norton
Copy link
Contributor

Changes for the net_lb_app_ext module:

  • Adds support for custom error response policies in path rules
  • Implements default custom error response policy functionality in path matchers and url map config

See the example for the google_compute_url_map resource here: https://registry.terraform.io/providers/hashicorp/google/6.13.0/docs/resources/compute_url_map#example-usage---url-map-custom-error-response-policy


Checklist

I applicable, I acknowledge that I have:

  • Read the contributing guide
  • Ran terraform fmt on all modified files
  • Regenerated the relevant README.md files using tools/tfdoc.py
  • Made sure all relevant tests pass

This update introduces the ability to define a default custom error response policy in the URL map module. It includes support for specifying error services and error response rules with match response codes, paths, and override response codes. This enhancement increases flexibility in handling custom error responses.
Modified the error_service assignment to include a fallback lookup mechanism for backend IDs when the value is not null. This ensures robustness and avoids null references while maintaining existing behavior.
Copy link

google-cla bot commented Feb 20, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@ludoo
Copy link
Collaborator

ludoo commented Feb 20, 2025

Very nice PR thanks :) Can you sign the CLA?

@ludoo ludoo enabled auto-merge (squash) February 20, 2025 19:32
@ludoo ludoo merged commit 24b7297 into GoogleCloudPlatform:master Feb 20, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants