-
Notifications
You must be signed in to change notification settings - Fork 24
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
possible tls issue with traefik-route #328
Comments
Alessandro Cabbia has found that in some cases we generate duplicate tls routers. It may be an instance of this issue:
|
to reproduce (using grafana in this case)
then inspect the grafana file where there are 3 routers instead of 2
the above comes from https://github.com/canonical/grafana-k8s-operator/blob/main/src/charm.py#L1325-L1380 |
another diff compared to other ingresses,
|
@PietroPasotti https://github.com/canonical/traefik-k8s-operator/blob/main/src/charm.py#L812-L814 seems |
Duplicate In tempo we had no |
Bug Description
in
charm._update_dynamic_config_route
, we inject tls config regardless of whether TLS is enabled.It's unclear whether that's a bug and if so what the consequences are.
To Reproduce
read the code
Environment
n/a
Relevant log output
Additional context
we should minimally add some documentation and find out what's going on and why
The text was updated successfully, but these errors were encountered: