Query: Enable promql-experimental-functions#8191
Query: Enable promql-experimental-functions#8191heysujal wants to merge 9 commits intothanos-io:mainfrom
Conversation
Signed-off-by: Sujal Gupta <sujalgupta6100@gmail.com>
Signed-off-by: Sujal Gupta <sujalgupta6100@gmail.com>
Signed-off-by: Sujal Gupta <sujalgupta6100@gmail.com>
Adjust the default get-config timeout to match the default get-config interval. Signed-off-by: Michael Hoffmann <mhoffmann@cloudflare.com> Signed-off-by: Sujal Gupta <sujalgupta6100@gmail.com>
Signed-off-by: Sujal Gupta <sujalgupta6100@gmail.com>
| activeQueryDir := cmd.Flag("query.active-query-path", "Directory to log currently active queries in the queries.active file.").Default("").String() | ||
|
|
||
| featureList := cmd.Flag("enable-feature", "Comma separated experimental feature names to enable.The current list of features is empty.").Hidden().Default("").Strings() | ||
| featureList := cmd.Flag("enable-feature", "Comma separated experimental feature names to enable. The current list of features is: promql-experimental-functions (enables experimental PromQL functions).").Hidden().Default("").Strings() |
There was a problem hiding this comment.
Does this flag still need to be hidden, I wonder
There was a problem hiding this comment.
The idea was just to support it, so I am not sure either.
| *enableTargetPartialResponse, | ||
| *enableMetricMetadataPartialResponse, | ||
| *enableExemplarPartialResponse, | ||
| false, |
There was a problem hiding this comment.
if --enable-feature=promql-experimental-functions is set, this will still take it as false.
maybe we can intercept by checking from here if the set value is promql-experimental-functions and pass true/false based on that.
There was a problem hiding this comment.
@Saumya40-codes , so like writing an expression here?
| DefaultTenant string | ||
| TenantCertField string | ||
| EnableXFunctions bool | ||
| EnableQueryExperimentalFunctions bool |
There was a problem hiding this comment.
we can use this field at cmd/thanos/query_frontend.go now
e.g. https://github.com/thanos-io/thanos/blob/main/cmd/thanos/query_frontend.go#L161
Signed-off-by: Sujal Gupta <55016909+heysujal@users.noreply.github.com>
|
Fixed in another PR. Thank you for your contribution! |
Changes
Fixes #7640
In
thanos/query.goAdded entry for the same in
pkg/api/query/engine.go,pkg/queryfrontend/config.go, andtest/e2e/e2ethanos/services.goVerification