Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(servarr): qbittorent ingress and jellyfin default value #41

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mbugeia
Copy link

@mbugeia mbugeia commented Nov 23, 2024

Kubito Helm Charts PR Checklist

Thank you for your contribution! Please ensure the following:

  • Chart Version: Updated version in Chart.yaml for the relevant chart.

  • README Update: If values.yaml was changed, run:

    readme-generator -v charts/your-chart/values.yaml -r charts/your-chart/README.md
  • Testing: Tested the chart locally, and it works as expected.

Changes Summary

Fix this 2 errors when installing the chart

* Service "servarr-jellyfin" is invalid: spec.externalTrafficPolicy: Invalid value: "Cluster": may only be set for externally-accessible services
 * Ingress.networking.k8s.io "servarr-qbittorrent" is invalid: spec.rules[0].http.paths[0].backend: Required value: port name or number is required

Qbittorent ingress was using a nonexistent value for the port number and a wrong service name

externalTrafficPolicy should only be set for LoadBalancer service, the default is ClusterIP

Related Issues

List any relevant issues, e.g., Fixes #123.

@mbugeia mbugeia changed the title Fix servarr ingress fix(servarr): qbittorent ingress and jellyfin default value Nov 23, 2024
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.

1 participant