prometheus/prometheus

promql: option to preserve comments in parser

Aperta

#15.871 aperta il 28 gen 2025

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Go (10.408 fork)batch import
component/promqlhelp wantedkind/featurenot-as-easy-as-it-lookspriority/P3

Metriche repository

Star
 (64.042 stelle)
Metriche merge PR
 (Merge medio 11g 5h) (118 PR mergiate in 30 g)

Descrizione

Proposal

I have a requirement to analyse and modify promql expressions to enforce some business-specific rules. This is being achieved by parsing to the AST, analysing the tree through traversal, and optionally applying modifications to the AST before rendering it back to a string.

Comments are currently stripped in this process by a hard-coded skip in the parser.

We would like this to be an option somewhere so that we can preserve these comments through the parse->AST->string process, as these comments can contain useful debugging/tracing information provided by the original query issuer.

Guida contributor

Direzione di ricerca
Esamina la logica di skip hardcoded nel parser.parse alle righe 356-363. Comprendi come i commenti vengono attualmente rimossi e come aggiungere un'opzione per preservarli. Cerca i pattern di opzioni del parser esistenti nel codebase.
Tech stack
go
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50