Move VolumeBalancer wakeup interval to storage.proto config as VolumeBalancerWakeupInterval#5681
Conversation
…BalancerWakeupInterval Original value is kept as default
|
Note This is an automated comment that will be appended during run. 🟢 linux-x86_64-relwithdebinfo target: cloud/tasks/,cloud/storage/ (test time: 263s): all tests PASSED for commit 0008cc5.
🟢 linux-x86_64-relwithdebinfo target: cloud/disk_manager/ (test time: 222s): all tests PASSED for commit 0008cc5.
🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 4740s): all tests PASSED for commit 0008cc5.
🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 1921s): some tests FAILED for commit 0008cc5.
🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 704s): some tests FAILED for commit 0008cc5.
🔴 linux-x86_64-relwithdebinfo target: cloud/blockstore/ (test time: 701s): some tests FAILED for commit 0008cc5.
|
|
|
||
| ctx.Schedule(Timeout, new TEvents::TEvWakeup); | ||
| ctx.Schedule( | ||
| StorageConfig->GetVolumeBalancerWakeupInterval(), |
There was a problem hiding this comment.
You can use UpdateCountersInterval instead
|
|
||
| ctx.Schedule(Timeout, new TEvents::TEvWakeup()); | ||
| ctx.Schedule( | ||
| StorageConfig->GetVolumeBalancerWakeupInterval(), |
There was a problem hiding this comment.
UpdateCountersInterval is a good choice here
There was a problem hiding this comment.
Mass test failures and conflicts
efimov-ivan-yteam
deleted the
users/efimov-ivan/move-balancer-wakeup-interval-to-config
branch
2 participants
Moved hard-coded value to config