feat(base-cluster/traefik): Changed ipFamilyPolicy to DualStack

[cknost]

ipFamilyPolicy for ipv6 Dual-Stack Ingress
https://github.com/traefik/traefik-helm-chart/blob/master/traefik/values.yaml#L825C5-L825C19

Summary by CodeRabbit

• New Features

  • Traefik ingress will prefer dual‑stack networking (IPv4/IPv6) when a static external IP is configured, improving compatibility in mixed IP environments.
  • No changes for deployments that don't set a static load‑balancer IP.

• Refactor

  • Ingress configuration now explicitly prefers dual‑stack when a static IP is present, yielding more predictable networking behavior across clusters.

[coderabbitai]

Walkthrough

Adds ipFamilyPolicy: PreferDualStack to the Traefik service values when .Values.ingress.IP is provided, alongside loadBalancerIP; no other templates, exports, or behavior changed.

Changes

Cohort / File(s)	Summary
Traefik Ingress Helm template charts/base-cluster/templates/ingress/traefik.yaml	Conditionally adds ipFamilyPolicy: PreferDualStack next to service.loadBalancerIP when .Values.ingress.IP is set; no other behavior changes.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant U as User Values (.Values)
    participant H as Helm Template
    participant K as Kubernetes Service (Traefik)

    U->>H: Provide ingress.IP
    alt .Values.ingress.IP is set
        H->>K: Render Service with\n`loadBalancerIP` and\n`ipFamilyPolicy: PreferDualStack`
        note right of K #D6EAF8: Dual-stack preference enabled
    else
        H->>K: Render Service without\n`ipFamilyPolicy` set
    end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I tweak the chart with careful knack,
A dual-stack hop along the track,
When an IP is placed in view,
PreferDualStack steps into cue.
Traefik smiles — the change is new. 🐇✨

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 clearly identifies the scope (base-cluster/traefik) and the primary intent—changing ipFamilyPolicy to enable dual-stack behavior—so it meaningfully summarizes the PR. It is slightly imprecise because the change sets ipFamilyPolicy to PreferDualStack rather than exactly "DualStack," but the intent is clear and relevant to the changeset.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch add_ipFamilyPolicy

---

📜 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 ad59bca and a7fdc04.

📒 Files selected for processing (1)

• charts/base-cluster/templates/ingress/traefik.yaml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

• charts/base-cluster/templates/ingress/traefik.yaml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: check licenses
• GitHub Check: lint helm chart (base-cluster)

---

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.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

charts/base-cluster/values.schema.json (1)

1429-1437: Good addition; enum matches Kubernetes values.

The property is well-scoped and consistent with ServiceSpec ipFamilyPolicy values.

If desired, consider exposing ipFamilies (array: ["IPv4","IPv6"]) alongside ipFamilyPolicy for environments that require explicit ordering.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between abebb3b and 194fef4.

📒 Files selected for processing (3)

• charts/base-cluster/templates/ingress/traefik.yaml (1 hunks)
• charts/base-cluster/values.schema.json (1 hunks)
• charts/base-cluster/values.yaml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: Update release-please config file for a possibly new chart
• GitHub Check: check licenses
• GitHub Check: lint helm chart (base-cluster)

🔇 Additional comments (1)

charts/base-cluster/values.yaml (1)

442-442: Sensible default.

PreferDualStack is safe on single‑stack clusters and enables dual‑stack where supported.

Please confirm this default is intended for both nginx and traefik providers, even though only the Traefik template currently consumes it.