Skip to content

Conversation

@AntoineThebaud
Copy link
Contributor

@AntoineThebaud AntoineThebaud commented Nov 24, 2025

Description

Closes perses/perses#1523.
Closes perses/perses#3509 (engine query param can be passed).

I guess this piece of code could be moved to plugin-system in the future if other datasources start to support query params. Since it's Prometheus only for now I prefer to have it here.

Screenshots

New section at the end of the datasource editor:

Screenshot 2025-11-24 164405

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.

UI Changes

  • Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
  • Code follows the UI guidelines.

Signed-off-by: Antoine THEBAUD <[email protected]>
@AntoineThebaud AntoineThebaud changed the title Antoinethebaud/prom query params [FEATURE] PrometheusDatasource: support query params Nov 24, 2025
Signed-off-by: Antoine THEBAUD <[email protected]>
@AntoineThebaud AntoineThebaud force-pushed the antoinethebaud/prom-query-params branch from 83e646b to f0b6d9a Compare November 24, 2025 16:18
@AntoineThebaud AntoineThebaud marked this pull request as ready for review November 24, 2025 17:02
@AntoineThebaud AntoineThebaud changed the title [FEATURE] PrometheusDatasource: support query params [FEATURE] Prometheus: support query params Nov 25, 2025
@shahrokni shahrokni self-requested a review November 28, 2025 08:39
- dont save empty keys
- avoid using index as key for loop-generated components
- remove useless check

Signed-off-by: Antoine THEBAUD <[email protected]>
@AntoineThebaud AntoineThebaud added this pull request to the merge queue Nov 28, 2025
Merged via the queue into main with commit 5f2dff4 Nov 28, 2025
13 checks passed
@AntoineThebaud AntoineThebaud deleted the antoinethebaud/prom-query-params branch November 28, 2025 13:43
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.

Prometheus: support query engine selection Prometheus: support query params

3 participants