Repository navigation
load balance individual hosts when clusters are to big to distribute - #14000
DaanHoogland wants to merge 1 commit into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## main #14000 +/- ##
=============================================
- Coverage 19.73% 3.41% -16.33%
=============================================
Files 6371 487 -5884
Lines 575784 41863 -533921
Branches 70478 7912 -62566
=============================================
- Hits 113644 1429 -112215
+ Misses 449788 40234 -409554
+ Partials 12352 200 -12152
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| @@ -88,37 +88,54 @@ public List<HostVO> getHostsToRebalance(ManagementServerHostVO ms, int avLoad) { | |||
| hostToClusterMap = sortByClusterSize(hostToClusterMap); | |||
There was a problem hiding this comment.
does the cluster sort here actually reorder anything? its compare never says one cluster comes first, so the order looks random and which cluster gets split depends on that
There was a problem hiding this comment.
fair, shpould look into that, but does this im[pact this PR, @Damans227 ?
Description
This PR...
Fixes: #9640
Note 1: this is usefull for smaller installations and/or installations that have a small number of clusters in comparison to the number of hosts.
Note 2: this does not affect or enable realtime rebalancing of direct agents, only attached agents are impacted.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?