Update GKE Terraform module to sync up with EKS#132
Merged
Conversation
Bring the GKE infrastructure module in sync with the EKS module by upgrading all upstream dependencies and adding production-ready defaults. - Add versions.tf with pinned provider constraints - Upgrade GKE engine module 24.1.0 -> ~> 35.0 - Upgrade network module 6.0.0 -> ~> 12.0 - Upgrade memorystore module 7.0.0 -> ~> 12.0 - Upgrade sql-db module 13.0.1 -> ~> 23.0 - Upgrade default PostgreSQL from v14 to v17 - Add prefix, labels, and node pool sizing variables - Add database and Redis configurability variables - Fix conditional outputs for optional resources - Enable deletion_protection by default Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Bring the GKE Helm module in sync with the EKS Helm module by adding Vector DaemonSet and Datadog Agent support with GCP-native patterns. - Add Vector DaemonSet log collection with Workload Identity - Add Datadog Agent DaemonSet for APM tracing - Add extra_helm_values for custom YAML overrides - Add force_update, wait, timeout variables - Add dependency_update to Helm release - Add versions.tf with provider constraints - Mark sensitive variables appropriately Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Update the gke-complete example to demonstrate new variables added in the GKE infrastructure and Helm module updates. - Use prefix variable and show optional node pool/database config - Demonstrate Vector DaemonSet and Datadog variables - Show extra_helm_values pattern - Add descriptions to all outputs - Mark sensitive outputs appropriately Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The Helm chart uses imageRegistry as a global prefix combined with image.repository to form the full image reference. Add a separate image_registry variable (analogous to ecr_registry in the EKS module) so the chart correctly constructs image paths for GCR/Artifact Registry. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.