-
Notifications
You must be signed in to change notification settings - Fork 462
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
Remove extend_writes
from Distributor YAML config
#2551
Remove extend_writes
from Distributor YAML config
#2551
Conversation
This is mentioned to be removed in 2.3, so this is a good time to do it. Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
355da3f
to
fd0f008
Compare
CHANGELOG.md
Outdated
@@ -14,6 +14,7 @@ | |||
* [CHANGE] Change default value for `-distributor.ha-tracker.max-clusters` to `100` to provide a DoS protection. #2465 | |||
* [CHANGE] Experimental block upload API exposed by compactor has changed: Previous `/api/v1/upload/block/{block}` endpoint for starting block upload is now `/api/v1/upload/block/{block}/start`, and previous endpoint `/api/v1/upload/block/{block}?uploadComplete=true` for finishing block upload is now `/api/v1/upload/block/{block}/finish`. #2486 | |||
* [CHANGE] Compactor: changed `-compactor.max-compaction-time` default from `0s` (disabled) to `1h`. When compacting blocks for a tenant, the compactor will move to compact blocks of another tenant or re-plan blocks to compact at least every 1h. #2514 | |||
* [CHANGE] Distributor: removed previously deprecated `extend_writes` (see #3719) YAML key from the Distributor config. #2551 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem like that issue/PR exists in Mimir. There is a related Cortex issue but it doesn't seem like it deprecates the option.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just force-pushed, I referenced the wrong issue. Good catch :)
Signed-off-by: Oleg Zaytsev <mail@olegzaytsev.com>
@56quarters re-requested review since I also removed the deprecated flag. |
Oh, right! I guess that's kind of important 😆 |
This is mentioned to be removed in 2.3, so this is a good time to do it.
Signed-off-by: Oleg Zaytsev mail@olegzaytsev.com