Skip to content

Conversation

@muffl0n
Copy link
Contributor

@muffl0n muffl0n commented Jan 31, 2025

Description

Fixes the helm chart for "http-webhook". It's removing support for older Kubernetes versions (pre 1.22) ruthlessly but supporting those would bloat the template and feels unnecessary.

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

Signed-off-by: Sven Schliesing <sven@schliesing.de>
serviceName and servicePort were removed and pathType is required in 1.22: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#ingress-v122

Signed-off-by: Sven Schliesing <sven@schliesing.de>
Otherwise it would break the yaml:
---
# SPDX-License-Identifier: Apache-2.0apiVersion: networking.k8s.io/v1
kind: Ingress
---

Signed-off-by: Sven Schliesing <sven@schliesing.de>
@netlify
Copy link

netlify bot commented Jan 31, 2025

Deploy Preview for docs-securecodebox ready!

Name Link
🔨 Latest commit 1e854a3
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/67a4c053d525ae0008c93568
😎 Deploy Preview https://deploy-preview-2878--docs-securecodebox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
@J12934 J12934 self-assigned this Feb 6, 2025
@J12934 J12934 added the bug Bugs label Feb 6, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2025

@J12934 J12934 merged commit 6a5a325 into secureCodeBox:main Feb 6, 2025
54 checks passed
@J12934
Copy link
Member

J12934 commented Feb 6, 2025

Perfect :)
Thanks @muffl0n 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bugs

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants