-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
API: remove configuration of Entrypoint and Middlewares #5119
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.
LGTM
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.
LGTM. Thanks! :)
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.
LGTM
This comment has been minimized.
This comment has been minimized.
This resolves "command traefik error: field not found, node: redirect" errors that put Traefik into an endless restart loop. See traefik/traefik#5119 for changes causing the BC break; this is a problem since Warden currently leverages the removed configuration settings in the TOML file that's generated and passed into the Traefik container. Thanks to @magedevjosh for finding and reporting this issue!
What does this PR do?
This PR removes the Middlewares and Entrypoint fields of the API handler configuration. As we're probably going to redo the routing and TLS handling (among other things) of other handlers (such as Ping, Rest provider, Prometheus metrics, etc) in the same way we're going to do it for the API handler, this PR also removes similarly extraneous fields for these as well.
Motivation
This change comes in preparation of a a rework of the API handler configuration, which will define how traffic is routed to it, how we handle TLS with it, etc.
We envision that the Entrypoint and Middlewares are not going to be part of its configuration anymore when we redo it, so remove them as of now, so that we don't have to break the configuration later on.
More
Additional Notes
Co-authored-by: Julien Salleyron julien.salleyron@gmail.com