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

feat: Add routing policy wrr and geo to google_dns_record_set #67

Conversation

dennislapchenko
Copy link
Contributor

@dennislapchenko dennislapchenko commented Oct 5, 2023

Add the ability to set routing_policy with wrr and geo blocks.
As per resource docs here.
It is not a fully implementation of all fields in routing_policy, just the basic geo and wrr usage.
It can be iterated upon for a full coverage of routing_policy options.

Initially played with condition validation but is not really required as it well done in the resource implementation itself.

@dennislapchenko dennislapchenko requested review from imrannayer and a team as code owners October 5, 2023 14:11
@dennislapchenko dennislapchenko force-pushed the feat/add_routing_policy_wrr_and_geo_to_dns-record-set branch from 1097b82 to 3e5619f Compare October 5, 2023 14:28
@imrannayer
Copy link
Collaborator

@dennislapchenko can you plz generate document

@dennislapchenko
Copy link
Contributor Author

@dennislapchenko can you plz generate document

Done

@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer imrannayer merged commit 1eed464 into terraform-google-modules:master Nov 9, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants