Skip to content

Fix Helm provider v3 configuration - #246

Open
Renato L. de F. Cunha (renatolfc) wants to merge 2 commits into
microsoft:mainfrom
renatolfc:fix/issue-227-helm-provider
Open

Fix Helm provider v3 configuration#246
Renato L. de F. Cunha (renatolfc) wants to merge 2 commits into
microsoft:mainfrom
renatolfc:fix/issue-227-helm-provider

Conversation

@renatolfc

Copy link
Copy Markdown
Contributor

Fresh Terraform initialization can select Helm provider v3 because the repository allows versions newer than 2.7.1, but the shared services and AKS Kubernetes modules still use the removed v2 nested kubernetes block. The services module also declares backend state even though it is consumed as a child module.

This PR switches those active provider configurations to object syntax and leaves backend ownership with the root modules. Provider versions and deployment behavior are otherwise unchanged.

Closes #227.

Copilot AI 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.

Pull request overview

Updates Terraform Helm provider configuration in Kubernetes-related modules to be compatible with Helm provider v3+ (while still working for v2), and removes an invalid child-module backend declaration so backend configuration remains owned by root modules.

Changes:

  • Removed backend "kubernetes" from the services module to avoid “Backend configuration ignored” warnings when used as a child module.
  • Updated Helm provider configuration from the removed nested kubernetes { ... } block syntax to the v3-compatible kubernetes = { ... } object argument syntax in both affected modules.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/vibe_core/vibe_core/terraform/services/providers.tf Removes child-module backend config and switches Helm provider Kubernetes config to object syntax.
src/vibe_core/vibe_core/terraform/aks/modules/kubernetes/providers.tf Switches Helm provider Kubernetes config to object syntax for Helm provider v3 compatibility.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

Incompatible helm provider configuration in modules/kubernetes/providers.tf (Terraform v2.x+)

2 participants