Skip to content

Cloud DNS config addition to GKE Cluster module#3752

Merged
SwarnaBharathiMantena merged 1 commit into
GoogleCloudPlatform:developfrom
SwarnaBharathiMantena:swarnabm/cloud_dns_gkecluster_module
Mar 5, 2025
Merged

Cloud DNS config addition to GKE Cluster module#3752
SwarnaBharathiMantena merged 1 commit into
GoogleCloudPlatform:developfrom
SwarnaBharathiMantena:swarnabm/cloud_dns_gkecluster_module

Conversation

@SwarnaBharathiMantena

@SwarnaBharathiMantena SwarnaBharathiMantena commented Mar 4, 2025

Copy link
Copy Markdown
Contributor

What?

Cloud DNS is exposed as a DNS provider option in the gke-cluster module.

Why?

The Cloud DNS provider replaces the default kube-dns in-cluster provider with Google's managed Cloud DNS service, and is often preferable for larger-node-count clusters.

Testing

Manually tested provisioning/updates, make tests passed, pre-commit hooks ran fine, and CLA should be on file.

Submission Checklist

NOTE: Community submissions can take up to 2 weeks to be reviewed.

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cluster Toolkit Contribution guidelines #

@SwarnaBharathiMantena SwarnaBharathiMantena added the do-not-merge Block merging of this PR label Mar 4, 2025
@SwarnaBharathiMantena SwarnaBharathiMantena changed the title Swarnabm/cloud dns gkecluster module Cloud DNS config addition to GKE Cluster module Mar 4, 2025
@SwarnaBharathiMantena SwarnaBharathiMantena added release-key-new-features Added to release notes under the "Key New Features" heading. and removed do-not-merge Block merging of this PR labels Mar 5, 2025
Comment thread modules/scheduler/gke-cluster/main.tf
annuay-google
annuay-google previously approved these changes Mar 5, 2025

@annuay-google annuay-google left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SwarnaBharathiMantena SwarnaBharathiMantena force-pushed the swarnabm/cloud_dns_gkecluster_module branch from 16e0e35 to e57ac25 Compare March 5, 2025 18:16
@SwarnaBharathiMantena SwarnaBharathiMantena merged commit 6275d34 into GoogleCloudPlatform:develop Mar 5, 2025
@SwarnaBharathiMantena SwarnaBharathiMantena deleted the swarnabm/cloud_dns_gkecluster_module branch March 28, 2025 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-key-new-features Added to release notes under the "Key New Features" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants