Skip to content
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

loki-mixin: fix label selectors + logs dashboard #4618

Merged
merged 1 commit into from
Nov 3, 2021
Merged

loki-mixin: fix label selectors + logs dashboard #4618

merged 1 commit into from
Nov 3, 2021

Conversation

cstyan
Copy link
Contributor

@cstyan cstyan commented Nov 3, 2021

Simplify standard label selectors (datsource, cluster, namespace) on multiple dashboards + fix broken logs dashboard.

The logs dashboard seems to be completely broken without this PR, are we actively using it at all?

Signed-off-by: Callum Styan callumstyan@gmail.com

multiple dashboards + fix broken logs dashboard.

Signed-off-by: Callum Styan <callumstyan@gmail.com>
@cstyan cstyan requested a review from a team as a code owner November 3, 2021 05:16
Copy link
Contributor

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

why did we remove templateLabels ? is it not used ?

Copy link
Contributor

@kavirajk kavirajk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

Q: Does this anyway break our internal dashboards or alerts (once its pulled it by /lib/loki-cloud-mixin)

@owen-d owen-d merged commit 5640cd0 into grafana:main Nov 3, 2021
@cstyan
Copy link
Contributor Author

cstyan commented Nov 3, 2021

@cyriltovena why did we remove templateLabels ? is it not used ? The common template labels, cluster/namespace/logs datsource, are all defined in a common place that every dashboard can pull them in from dashboard-utils.libsonnet. Adding them to templating again manually causes problems.

@kavirajk Does this anyway break our internal dashboards or alerts (once its pulled it by /lib/loki-cloud-mixin) it shouldn't I tested this internally by modifying these same files but in our vendor directory. As usual, if you do see anything that's broken feel free to let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants