Skip to content

Conversation

@aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Nov 14, 2025

I'm proposing to add a blog post on the PromQL info function! During the last PromCon, a revelation was that the info function is too poorly marketed. One potential remediation mentioned was writing a blog post :)

Signed-off-by: Arve Knudsen <[email protected]>
@aknuds1 aknuds1 changed the title Add info function blog post Add info function blog post Nov 14, 2025
@aknuds1 aknuds1 changed the title Add info function blog post Add PromQL info function blog post Nov 14, 2025
Copy link
Member

@ywwg ywwg left a comment

Choose a reason for hiding this comment

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

This is great! I made a few comments

aknuds1 and others added 2 commits November 17, 2025 17:30
Co-authored-by: Owen Williams <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
Co-authored-by: Owen Williams <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
Copy link
Member

@beorn7 beorn7 left a comment

Choose a reason for hiding this comment

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

Sorry for the avalanche of comments. Here are my main lines of thought (from which most comments derive):

  • We should really focus on the cases where info is much more simple than the traditional query:
    1. All data labels are copied. (You have covered this case, but always further down; it should be the very first case to show, though.)
    2. Filtering for label value. (You haven't covered this case at all.)
  • Rather than reinforcing the misconception that info is merely solving a syntactic problem (which could be read as info being merely syntactic sugar), we should clarify that the problem of the complex join queries is caused by the amount of information the user has to specify and that info solves this by deducing this information from other sources (or just using heuristics for now, which also explains the shortcomings of the current implementation).
  • We should be very upfront about the missing parts, both that they are relevant but also that we want to work on them. That's partially already the case, but as explained in comments, it could be clearer. What we should also add is that the tooling outside of PromQL proper doesn't really handle the info function appropriately yet, e.g. autocomplete or query builders. (Many issues Patrick reported were about exactly those deficiencies. But for him, it appeared as fundamental problems of info and not as an incomplete implementation.)

@aknuds1 aknuds1 force-pushed the arve/info-blog-post branch from bdab42e to 2f68dcb Compare November 27, 2025 09:58
Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
@aknuds1 aknuds1 requested a review from beorn7 November 27, 2025 16:02
Signed-off-by: Arve Knudsen <[email protected]>
@aknuds1 aknuds1 requested a review from ywwg November 27, 2025 16:08
@beorn7
Copy link
Member

beorn7 commented Nov 27, 2025

Will have a look ASAP. (I'm saying that to many these days… it might not be very soon…)

@aknuds1
Copy link
Contributor Author

aknuds1 commented Nov 27, 2025

Thanks @beorn7!

beorn7
beorn7 previously requested changes Dec 2, 2025
Copy link
Member

@beorn7 beorn7 left a comment

Choose a reason for hiding this comment

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

A few more things, but looking very good already.

Note that the date in the filename needs to be updated to the actual date of publication.

@aknuds1 aknuds1 requested a review from beorn7 December 2, 2025 19:21
Copy link
Member

@ywwg ywwg left a comment

Choose a reason for hiding this comment

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

Can we make "Giving Feedback" a heading so it's more eyecatching? Right now it's buried at the end of the post -- make it really easy for people who want to talk back to do so.

@aknuds1 aknuds1 requested a review from ywwg December 3, 2025 08:13
@aknuds1
Copy link
Contributor Author

aknuds1 commented Dec 3, 2025

Can we make "Giving Feedback" a heading so it's more eyecatching?

@ywwg @beorn7 I moved requests for feedback to a single section "Giving Feedback", just before the "Conclusion" section. WDYT?

@beorn7
Copy link
Member

beorn7 commented Dec 6, 2025

I'm traveling until Dec 16. My remaining items were not very significant, and I trust you to address them appropriately, so as soon as the other reviewers are happy, assume I am, too, and don't wait for my explicit approval.

Copy link
Member

@ywwg ywwg left a comment

Choose a reason for hiding this comment

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

this looks great, thank you!

@aknuds1 aknuds1 dismissed beorn7’s stale review December 16, 2025 14:09

Dismissing stale review as @beorn7 says it's good to go, given Owen's approval.

Signed-off-by: Arve Knudsen <[email protected]>
Signed-off-by: Arve Knudsen <[email protected]>
@aknuds1 aknuds1 merged commit a9b9330 into main Dec 16, 2025
5 checks passed
@aknuds1 aknuds1 deleted the arve/info-blog-post branch December 16, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants