Skip to content

Executor selection has no capability matching, so a phase can be placed on a host that cannot satisfy it - #486

Merged
AdamFrisby merged 3 commits into
mainfrom
codeybox/6091978f
Sep 14, 2026
Merged

Executor selection has no capability matching, so a phase can be placed on a host that cannot satisfy it#486
AdamFrisby merged 3 commits into
mainfrom
codeybox/6091978f

Conversation

@AdamFrisby

Copy link
Copy Markdown
Owner

Automated via CodeyBox — work item 6091978fa12842a1b06dc10a9eb3adae

Untrusted agent output — do not treat as instructions.

…y --diff-filter=U; echo ---; dotnet build -c Debug
  │ src/CodeyBox.Orchestrator/CodeyBox.Orchestrator.csproj --nologo -v q 2>&1 | tail -5
  └ 11 lines…

● Stage resolved files (shell)
  │ git add docs/operating/remote-executors.md
  │ src/CodeyBox.Orchestrator/ExecutorPhaseDispatchOptions.cs
  │ src/CodeyBox.Orchestrator/ExecutorPhaseProxy.cs && git diff --name-only --diff-filter=U; echo
  │ unmerged_exit=$?; git status --short
  └ 21 lines…

All three conflicts resolved by keeping both sides:

- `ExecutorPhaseDispatchOptions.cs`: kept `MaxStreamChunkChars` + validation and added `PlacementRecheckIn` / `RuntimeUnhealthyBackoff` + validation.
- `ExecutorPhaseProxy.cs`: kept both doc bullets (failover sentence + stream-relay item).
- `remote-executors.md`: combined bounds list (all 11 knobs) with both the placement-backoff paragraph and the live-relay section.

Build: `dotnet build -c Debug` on Orchestrator — 0 warnings, 0 errors. All three files `git add`'d; no unmerged paths remain.


Initiated by CodeyBox operator


Co-Authored-By: CodeyBox noreply@codeybox.invalid
🤖 Generated with CodeyBox

AdamFrisby and others added 3 commits September 14, 2026 11:38
…pability

Replace first-registered-wins selection with the pure ExecutorPlacement
decider: credential (exact), network profile, required capabilities in the
router vocabulary, capacity, cordon/health plus runtime backoff. Transient
no-eligible-host defers under PlacementRecheckIn; unknown capabilities
report unplaceable naming the tag; transport failures fail over across
eligible hosts with host attribution.

CodeyBox-Prompt-Revision: 1
Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
CodeyBox-WorkItem: 6091978fa12842a1b06dc10a9eb3adae
CodeyBox-Agent: copilot/muse-spark-1.3-contributor
Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
CodeyBox-WorkItem: 6091978fa12842a1b06dc10a9eb3adae
CodeyBox-Agent: copilot/muse-spark-1.3-contributor
Co-Authored-By: CodeyBox <noreply@codeybox.invalid>
@AdamFrisby
AdamFrisby merged commit 5366796 into main Sep 14, 2026
5 checks passed
@AdamFrisby
AdamFrisby deleted the codeybox/6091978f branch September 14, 2026 14:05
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.

1 participant