Skip to content

Conversation

@t0mmylam
Copy link
Collaborator

@t0mmylam t0mmylam commented Oct 9, 2025

Implement Compartment Overlap Resolution

Adds overlap resolution logic for nodes that match multiple compartments in a DeploymentPolicy. When overlaps occur, nodes are assigned deterministically using a safety heuristic:

  1. Strategy precedence: Fixed > Linear > Exponential
  2. Window size tie-break: Smaller effective capacity wins
    • Percent budgets: capacity = max(1, ceil(percent/100 × matched))
    • Count budgets: use count directly
  3. Lexicographic tie-break: Alphabetically by compartment name

Copy link
Collaborator

@lockwobr lockwobr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is the compartment wrapper used? it seems pretty light now

@lockwobr
Copy link
Collaborator

is the compartment wrapper used? it seems pretty light now

nvm, i see the 104 is making use of it.

@t0mmylam t0mmylam merged commit 9ae23a9 into main Oct 13, 2025
6 checks passed
@t0mmylam t0mmylam deleted the overlaps branch October 13, 2025 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants