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

drop_common_labels reported as unknown function #3320

Closed
smarterclayton opened this Issue Oct 19, 2017 · 7 comments

Comments

Projects
None yet
3 participants
@smarterclayton
Copy link

smarterclayton commented Oct 19, 2017

Got an error on query drop_common_labels(openshift_sdn_pod_setup_errors) > 0 on v2.0.0-rc.1. I had just upgraded, but I did force refresh the page (in case something was cached).

Error executing query: parse error at char 19: unknown function with name "drop_common_labels"
@grobie

This comment has been minimized.

Copy link
Member

grobie commented Oct 19, 2017

drop_common_labels has been removed and is not longer available, see #3060.

@brian-brazil We need a better answer for this then "it's gone".

@smarterclayton

This comment has been minimized.

Copy link
Author

smarterclayton commented Oct 19, 2017

Ok, expected the docs to be updated. Also use it extremely often for very dense labels generated by kubernetes clusters (where there are several layers of labels being added which are mostly the same)

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Oct 19, 2017

You should be using by/without in your aggregations to select exactly which labels you want. drop_common_labels and keep_common are unsafe, as they don't always return the same labels.

@smarterclayton

This comment has been minimized.

Copy link
Author

smarterclayton commented Oct 19, 2017

It's more useful in the cases where I'm zeroing in on the labels I do want, by removing all the ones that don't change and thus cannot be useful in further refining. Not disagreeing it is redundant to other more focused tools, just identifying that it's a regular part of iterative flow of working through large metric sets to refine what I might need to search on, especially in the absence of knowing what the important labels are in an ad-hoc search.

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Oct 19, 2017

That's something that's best done in an exploration tool, it doesn't belong in the core language.

If you have lots of labels that are unchanging, it's possible that you have redundant target labels that should be trimmed.

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Oct 23, 2017

We already have bugs tracking 2.0 doc updates, there's nothing to be done here on the Prometheus side.

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 23, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 23, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.