Skip to content

feat: share resource constrained executor check#19

Merged
intel352 merged 1 commit into
mainfrom
refactor/resource-constrained-executor-core
May 25, 2026
Merged

feat: share resource constrained executor check#19
intel352 merged 1 commit into
mainfrom
refactor/resource-constrained-executor-core

Conversation

@intel352
Copy link
Copy Markdown
Contributor

Summary

  • Add core executor capability DTOs for resource-limit placement checks.
  • Add shared helpers for CPU/memory limits requiring a resource-constrained executor and for matching worker executor capabilities to placement requirements.

Verification

  • RED: GOWORK=off go test ./protocol -run TestExecutorCapabilitiesHaveResourceConstrainedMatch -count=1 failed before implementation with missing API symbols.
  • GOWORK=off go test ./protocol -run TestExecutorCapabilitiesHaveResourceConstrainedMatch -count=1
  • git diff --check
  • GOWORK=off go test ./... -count=1

Adversarial Review

  • Kept provider capability reports out of this helper because support/degraded validation is a separate workflow-compute policy path.
  • Preserved existing semantics: CPU or memory limits require constrained execution; workspace-only limits do not.
  • Legacy tier-only workers are still handled when explicit executor refs are absent.

@intel352 intel352 merged commit d51e6f3 into main May 25, 2026
4 checks passed
@intel352 intel352 deleted the refactor/resource-constrained-executor-core branch May 25, 2026 07:57
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