-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Closed
Description
Bug Report
With #4177. There is a confusion on grafana being isRequired while jaeger being not, even though they do the same things. The change would be to remove isRequired for grafana and update the tests as they expect grafana field to be there always.
Also, we are not really using grafana and jaeger fields, and just checking if they are empty. Using a boolean would be a better choice here.
What is the issue?
How can it be reproduced?
Logs, error output, etc
(If the output is long, please create a gist and
paste the link here.)
linkerd check output
your output here ...
Environment
- Kubernetes Version:
- Cluster Environment: (GKE, AKS, kops, ...)
- Host OS:
- Linkerd version:
Possible solution
Additional context
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels