-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
Add cold storage support #148
Conversation
/test all |
@dm3ch we can not run tests for this PR at the moment. |
@goruha Merged current master |
/test all |
68fc51b
to
f088595
Compare
@goruha Yep, merged the wrong branch. Now dropped unwanted changes from this PR. |
/test all |
@goruha I'm not sure how to better fix this test. I even don't understand why the test tries to enable cold storage if I haven't modified any tests and made it disabled by default. |
@dm3ch it does not enable cold storage.
I think you have to make this block |
Ok, wrapped to dynamic block |
3cb86c6
to
470ba57
Compare
/test all |
@dm3ch LGTM Thanks for your contribution. |
what
cluster_config.cold_storage_options.enabled
why