Skip to content

Removed deprecated -querier.split-queries-by-day #3813

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

Merged

Conversation

pracucci
Copy link
Contributor

What this PR does:
We deprecated -querier.split-queries-by-day in Cortex 0.4.0 but we never removed it. I guess it's time to do it.

Which issue(s) this PR fixes:
N/A

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
@pracucci pracucci merged commit 46a8a65 into cortexproject:master Feb 12, 2021
@pracucci pracucci deleted the remove-deprecated-split-queries-by-day branch February 12, 2021 10:41
CharlieTLe added a commit that referenced this pull request Dec 24, 2024
...interval`

Query-frontend: removed `-querier.split-queries-by-day` (deprecated in Cortex 0.4.0). Please use `-querier.split-queries-by-interval` instead. #3813
CharlieTLe added a commit to CharlieTLe/cortex that referenced this pull request Dec 24, 2024
...interval`

Query-frontend: removed `-querier.split-queries-by-day` (deprecated in Cortex 0.4.0). Please use `-querier.split-queries-by-interval` instead. cortexproject#3813
Signed-off-by: Charlie Le <charlie_le@apple.com>
CharlieTLe added a commit that referenced this pull request Dec 24, 2024
...interval`

Query-frontend: removed `-querier.split-queries-by-day` (deprecated in Cortex 0.4.0). Please use `-querier.split-queries-by-interval` instead. #3813
Signed-off-by: Charlie Le <charlie_le@apple.com>
friedrichg pushed a commit that referenced this pull request Dec 30, 2024
* Update arguments.md

Removed obsolete `-promql.lookback-delta` option (deprecated since Cortex 1.2, replaced with `-querier.lookback-delta`). #3144

Signed-off-by: Charlie Le <3375195+CharlieTLe@users.noreply.github.com>
Signed-off-by: Charlie Le <charlie_le@apple.com>

* Rename `-querier.split-queries-by-day` to `-querier.split-queries-by-...

...interval`

Query-frontend: removed `-querier.split-queries-by-day` (deprecated in Cortex 0.4.0). Please use `-querier.split-queries-by-interval` instead. #3813
Signed-off-by: Charlie Le <charlie_le@apple.com>

* Fix typo and add description for `max_samples_per_query`

Flag `ingester.max-samples-per-query` was removed in #4812 as part of the chunk storage removal.

Signed-off-by: Charlie Le <charlie_le@apple.com>

* Remove 'Storage' section from arguments documentation

Removed in 73567bc#diff-ed3c6bb08ff3f85373c673fd92d1429599243584fc46d51a3221782263f26821L130

Signed-off-by: Charlie Le <charlie_le@apple.com>

* Update arguments.md

* Remove deprecated flag

Signed-off-by: Charlie Le <charlie_le@apple.com>

---------

Signed-off-by: Charlie Le <3375195+CharlieTLe@users.noreply.github.com>
Signed-off-by: Charlie Le <charlie_le@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants