-
Notifications
You must be signed in to change notification settings - Fork 27
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
restart on traefik-route initiated static config changes #325
Conversation
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.
First round done, two things come up. Will see it again once CI is happy.
🤦 Co-authored-by: Mateusz Kulewicz <[email protected]>
…aefik-k8s-operator into traefik-route-static-config
…aefik-k8s-operator into traefik-route-static-config
CI is stuck because of microsoft/linux-package-repositories#130 now trying to run them locally, if they pass I vote we merge on red |
only test failing locally is unrelated to these changes. (it's on forward-auth) |
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.
This PR adds a 'static' field to the traefik-route interface and instructs Traefik to take whatever's in there and put it into the static config and restart itself whenever there's a change.