Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"unhide" query.instant.default.max_source_resolution Flag #6261

Open
rekup opened this issue Apr 4, 2023 · 1 comment
Open

"unhide" query.instant.default.max_source_resolution Flag #6261

rekup opened this issue Apr 4, 2023 · 1 comment

Comments

@rekup
Copy link

rekup commented Apr 4, 2023

Hello everyone

We are using thanos ruler for some recording rules which do use offset to compare certain values to historic data. The offset hits a date range where we do not have any raw data, but only downsampled data. As a result, the query does not return any results. After some search we discovered the hidden flag --query.instant.default.max_source_resolution which was introduced in #1431. After configuring this option, our queries do work. I was wondering whether we could make this hidden flag a regular one by "unhiding" it

Describe the solution you'd like

Setting --query.instant.default.max_source_resolution to e.g. 1h resolves our issue but this flag is hidden by default and there is also no documentation for this flag.

Additional context

I am not sure why exactly this feature was implemented as a hidden flag. In my opinion, this feature could be interesting for others as well, and it might make sense to add some documentation for and make it a regular flag. Any opinions on this?
Everyone agrees, I would be happy to add documentation for this flag and "unhide" it.

Thanks,
Reto

@matej-g
Copy link
Collaborator

matej-g commented Apr 19, 2023

Hey @rekup,
Looks like this older comment might give some insight on why #1431 (review), I'd say if those reasons are still valid it makes sense to keep it hidden. Alternatively, we can document this use case and fix and still keep it hidden.

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

No branches or pull requests

2 participants