Skip to content

[BUGFIX] PrometheusTimeSeriesQuery: Grafana migration: don't match non-prom queries when type is set#632

Merged
AntoineThebaud merged 1 commit intomainfrom
antoinethebaud/prom-ts-query-migrate-fix
Apr 17, 2026
Merged

[BUGFIX] PrometheusTimeSeriesQuery: Grafana migration: don't match non-prom queries when type is set#632
AntoineThebaud merged 1 commit intomainfrom
antoinethebaud/prom-ts-query-migrate-fix

Conversation

@AntoineThebaud
Copy link
Copy Markdown
Contributor

@AntoineThebaud AntoineThebaud commented Apr 16, 2026

Description

Closes perses/perses#4013.

This PR reduces the "greediness" of the PrometheusTimeSeriesQuery migration logic by ensuring it's not matched when the datasource's type is explicitly set with another value than "prometheus". It doesn't change the current behavior when type is not set.

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

…n-prom queries when `type` is set

Signed-off-by: AntoineThebaud <antoine.thebaud@yahoo.fr>
@AntoineThebaud AntoineThebaud marked this pull request as ready for review April 16, 2026 21:29
@AntoineThebaud AntoineThebaud added this pull request to the merge queue Apr 17, 2026
Merged via the queue into main with commit 8def69e Apr 17, 2026
16 checks passed
@AntoineThebaud AntoineThebaud deleted the antoinethebaud/prom-ts-query-migrate-fix branch April 17, 2026 20:23
@rickardsjp
Copy link
Copy Markdown
Contributor

Thanks @AntoineThebaud!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PrometheusTimeSeriesQuery: migration greedily matches non-Prometheus targets

3 participants