Update operator versions and move them to variables

[Tzvonimir]

Summary by CodeRabbit

• New Features

  • Added configurable input variables for Helm chart versions (operator and zxporter), allowing users to specify which chart versions to deploy.

• Chores

  • Updated provider version constraints: devzero provider (0.1.5), helm provider (3.1.1).

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

This PR refactors hardcoded Helm chart versions into configurable Terraform variables. It introduces two new input variables for chart versions, updates provider version constraints, and modifies helm_release resources to reference these variables instead of literal values.

Changes

Cohort / File(s)	Summary
Provider and documentation updates README.md	Updated provider version constraints (devzero ≥ 0.1.1 → 0.1.5, helm ≥ 3.0.2 → 3.1.1) and added documentation for two new input variables: operator_chart_version (default "0.1.14") and zxporter_chart_version (default "0.0.26").
Variable definitions variables.tf	Added two new string variables: zxporter_chart_version and operator_chart_version, with descriptions and defaults matching README documentation.
Resource configuration main.tf	Updated helm_release resources to use the new variables for chart versions: zxporter version now uses var.zxporter_chart_version (was "0.0.22") and devzero_operator version now uses var.operator_chart_version (was "0.1.9").

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Changes follow a straightforward parameterization pattern across three files
• Variable definitions and provider constraints are uncomplicated
• Version attribute substitutions are direct and predictable

Possibly related PRs

• Enable single opertor to be installed #5: Modifies the same helm_release resources (zxporter and devzero_operator) by adding conditional counts to enable/disable releases.
• Upgrade helm versions #4: Changes version attributes of the same helm_release resources and further extracts them into variables.

Suggested reviewers

• lbagic
• MauroKinderknecht

Poem

🐰 Chart versions hop into variables today,
No more hardcoding in the Terraform way,
DevZero and Zxporter now flexibly dance,
With operators and helm at configs' command!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main changes: introducing new variables for Helm chart versions and updating provider constraints to use these variables instead of hard-coded values.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch tzvonimir/variable-cluster-version

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ecc8530 and 4ce14b9.

📒 Files selected for processing (3)

• README.md (2 hunks)
• main.tf (2 hunks)
• variables.tf (1 hunks)

🔇 Additional comments (4)

main.tf (1)

28-28: Helm release versions successfully externalized to variables.

The changes correctly reference var.zxporter_chart_version and var.operator_chart_version, enabling operators to customize chart versions without modifying the module source.

Also applies to: 69-69

variables.tf (1)

94-104: Version bump in chart defaults requires careful rollout planning.

The new variables introduce upgraded default chart versions (zxporter: 0.0.22 → 0.0.26; operator: 0.1.9 → 0.1.14). This means existing deployments using defaults will automatically pull newer chart versions. Verify that:

1. These chart versions have no breaking changes
2. The updated provider versions (devzero 0.1.5, helm 3.1.1) are compatible
3. Users are aware of and prepared for this upgrade

README.md (2)

39-40: Clarify provider version constraint strategy.

The provider version constraints have been tightened to specific versions (devzero 0.1.5, helm 3.1.1) rather than using >= (minimum version) constraints. This reduces flexibility for users with different versions. Consider whether:

1. These are pinned versions for testing purposes and should remain >= constraints
2. There are specific compatibility reasons requiring these exact versions with the new chart versions
3. Breaking changes exist in adjacent versions that justify the restriction

---

67-67: Input variables properly documented.

The new chart version variables are correctly added to the documentation with clear descriptions and defaults, maintaining consistency with existing variable documentation.

Also applies to: 71-71

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}