Closed
Bug 598800
Opened 14 years ago
Closed 13 years ago
Server needs to do proportional allocation to nodes
Categories
(Cloud Services :: Server: Registration, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: telliott, Assigned: telliott)
References
Details
(Keywords: push-needed, Whiteboard: [qa-])
Attachments
(2 files)
(deleted),
patch
|
mconnor
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jrconlin
:
review+
|
Details | Diff | Splinter Review |
With the introduction of different-capacity servers, we need to assign to nodes based on a proportion of expected maximum.
Assignee | ||
Comment 1•14 years ago
|
||
I think this is actually easier than I expected. Instead of ordering by actives, we order by actives/max_capacity.
If this turns out to be too inefficient (shouldn't be with only a couple hundred rows), we can have the nightly actives update add % to a new column and order by that.
This doesn't address how we get max_capacity into available_nodes (and mandates that max_capacity is now NOT NULL). We probably need to add it to the master config file for each node.
Assignee | ||
Comment 3•14 years ago
|
||
This requires the changes in 599133 or things will break horribly
Attachment #488268 -
Flags: review?(mconnor)
Updated•14 years ago
|
Attachment #488268 -
Flags: review?(mconnor) → review+
Assignee | ||
Comment 4•14 years ago
|
||
Updated•14 years ago
|
Whiteboard: [qa-]
Assignee | ||
Comment 5•14 years ago
|
||
Also need to do this to sreg
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•14 years ago
|
||
changes the select over to the reg version. Also appears to have removed a bunch of spaces.
Attachment #520900 -
Flags: review?(jrconlin)
Comment 7•13 years ago
|
||
Comment on attachment 520900 [details] [diff] [review]
Uses the reg algorithm to assign nodes
Wow. loads of whitespace diffs.
Attachment #520900 -
Flags: review?(jrconlin) → review+
Comment 8•13 years ago
|
||
Why this is REOPENED ?
Assignee | ||
Updated•13 years ago
|
Status: REOPENED → RESOLVED
Closed: 14 years ago → 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•