Skip to content

fix: explicitly specify TCP protocol for helm SSA compatibility (#692)#1901

Merged
alvneiayu merged 1 commit intobitnami-labs:mainfrom
leantos:fix/issue-692-tcp-protocol
Mar 12, 2026
Merged

fix: explicitly specify TCP protocol for helm SSA compatibility (#692)#1901
alvneiayu merged 1 commit intobitnami-labs:mainfrom
leantos:fix/issue-692-tcp-protocol

Conversation

@leantos
Copy link
Copy Markdown
Contributor

@leantos leantos commented Feb 27, 2026

Description of the change
This PR adds the explicit protocol: TCP field to the port definitions in the Sealed Secrets Helm chart.

While TCP is the default protocol in Kubernetes, Server-Side Apply (SSA) on clusters running version 1.19 or earlier can reject manifests if the protocol is not explicitly defined. This change ensures the Helm chart is compatible with modern GitOps workflows (like ArgoCD or Flux using SSA) across all supported Kubernetes versions.

Targeted Files:

  • templates/deployment.yaml: Added protocol: TCP to the http and metrics container ports.
  • templates/service.yaml: Added protocol: TCP to the main service and metrics service port definitions.

Benefits

  • Resolves deployment failures when using Server-Side Apply on older Kubernetes clusters (<= 1.19).
  • Improves manifest explicitness and follows best practices for cross-version compatibility.

Applicable issues

Checklist

  • I have read the CONTRIBUTING doc.
  • My commit is Signed-off-by using git commit -s.
  • (If applicable) Table of Contents in README.md or values.txt has been updated via doctoc.
  • Changes are focused solely on the Helm YAML templates as requested.

…ami-labs#692)

Signed-off-by: Emmanuel Tom Jose <84658038+leantos@users.noreply.github.com>
@alvneiayu alvneiayu merged commit 255d92b into bitnami-labs:main Mar 12, 2026
15 checks passed
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Mar 13, 2026
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [bitnami-labs/sealed-secrets](https://github.com/bitnami-labs/sealed-secrets) | patch | `v0.36.0` → `v0.36.1` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>bitnami-labs/sealed-secrets (bitnami-labs/sealed-secrets)</summary>

### [`v0.36.1`](https://github.com/bitnami-labs/sealed-secrets/blob/HEAD/RELEASE-NOTES.md#v0361)

[Compare Source](bitnami-labs/sealed-secrets@v0.36.0...v0.36.1)

- Doc/issue 501 all namespaces ([#&#8203;1900](bitnami-labs/sealed-secrets#1900))
- Bump go 1.26.1 ([#&#8203;1914](bitnami-labs/sealed-secrets#1914))
- Update actions/setup-go to v6.2.0 ([#&#8203;1906](bitnami-labs/sealed-secrets#1906))
- fix: explicitly specify TCP protocol for helm SSA compatibility ([#&#8203;692](bitnami-labs/sealed-secrets#692)) ([#&#8203;1901](bitnami-labs/sealed-secrets#1901))
- docs: document GKE Warden and RBAC restrictions ([#&#8203;1892](bitnami-labs/sealed-secrets#1892))
- Bump k8s.io/klog/v2 from 2.130.1 to 2.140.0 ([#&#8203;1913](bitnami-labs/sealed-secrets#1913))
- chore: remove note about deprecation of helm chart. ([#&#8203;1902](bitnami-labs/sealed-secrets#1902))
- Bump k8s.io/code-generator from 0.35.1 to 0.35.2 ([#&#8203;1909](bitnami-labs/sealed-secrets#1909))
- Bump k8s.io/client-go from 0.35.1 to 0.35.2 ([#&#8203;1908](bitnami-labs/sealed-secrets#1908))
- Bump distroless/static from `d90359c` to `28efbe9` in /docker ([#&#8203;1912](bitnami-labs/sealed-secrets#1912))
- Fix oci push action ([#&#8203;1899](bitnami-labs/sealed-secrets#1899))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever MR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My42NC4zIiwidXBkYXRlZEluVmVyIjoiNDMuNjQuMyIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90IiwiYXV0b21hdGlvbjpib3QtYXV0aG9yZWQiLCJkZXBlbmRlbmN5LXR5cGU6OnBhdGNoIl19-->
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.

Helm chart add protocols

2 participants