-
Notifications
You must be signed in to change notification settings - Fork 117
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
drone chart updates #53
Conversation
- update to the latest version of the drone docker image, 2.11.1 - use /healthz for livenessProbe and add readinessProbe - update the chart to be compatibile with newer versions of kubernetes that require `pathType` for each specified path in an ingress - update templates/NOTES.txt and templates/ingress.yaml based on files generated by running `helm create drone` with helm version 3.8.2
charts/drone/values.yaml
Outdated
@@ -48,7 +48,8 @@ ingress: | |||
hosts: | |||
- host: chart-example.local | |||
paths: | |||
- "/" | |||
- path: "/*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think wildcards are not supported in ingress paths. At least they don't work with my traefik ingress controller and the kubernetes documentation doesn't seem to mention them.
- path: /
pathType: Prefix
... should basically do the same thing and a lot of charts use it as suggested setting.
Everything else works fine (k3s v1.22.7)! Thanks for updating the chart, @jimsheldon 💯 🥇
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the feedback @pauvos! I'm not sure what documentation I was reading, but I agree that pathType: Prefix
makes more sense as a default. I have pushed a commit with this change, which also adds some comments.
update to the latest version of the drone docker image, 2.11.1
use /healthz for livenessProbe and add readinessProbe
update the chart to be compatibile with newer versions of kubernetes that
require
pathType
for each specified path in an ingressupdate templates/NOTES.txt and templates/ingress.yaml based on files
generated by running
helm create drone
with helm version 3.8.2