Skip to content

fix(scheduler): backport victim-invariant prefilter to v0.9#1568

Merged
enoodle merged 13 commits intokai-scheduler:v0.9from
enoodle:erez/v0.9/pod-invariant-predicate-implement
May 11, 2026
Merged

fix(scheduler): backport victim-invariant prefilter to v0.9#1568
enoodle merged 13 commits intokai-scheduler:v0.9from
enoodle:erez/v0.9/pod-invariant-predicate-implement

Conversation

@enoodle
Copy link
Copy Markdown
Collaborator

@enoodle enoodle commented May 8, 2026

Description

Backport the victim-invariant pre-predicate guard to solver-backed actions on v0.9 so reclaim, preempt, and consolidation skip solver work for jobs already blocked by failures that evicting victims cannot fix.

This backport includes:

  • the victim-invariant session hook
  • predicate classification for missing PVCs, missing required ConfigMaps, and tasks larger than any node
  • action-level early skip wiring for reclaim, preempt, and consolidation
  • the candidate-only pre-filter cache used by the action guard and normal allocation path
  • the release-branch changelog line

This backport intentionally excludes the main-branch design-doc and benchmark-only commits.

Related Issues

Fixes #1502

Checklist

Note: Ensure your PR title follows the Conventional Commits format (e.g., feat(scheduler): add new feature)

  • Self-reviewed
  • Added/updated tests (if needed)
  • Updated documentation (if needed)

Breaking Changes

None.

Additional Notes

Validation:

  • go test ./pkg/scheduler/framework ./pkg/scheduler/actions/common ./pkg/scheduler/actions/reclaim ./pkg/scheduler/actions/preempt ./pkg/scheduler/actions/consolidation ./pkg/scheduler/plugins/predicates ./pkg/scheduler/k8s_internal/predicates

Reviewer guidance:

  • this is a release-branch backport only
  • the branch was cleaned to keep only the scheduler fix stack plus the changelog line
  • design-doc and benchmark-only history from the main branch was intentionally not carried here

enoodle added 12 commits May 7, 2026 15:56
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 8, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 67075973-aac1-4b0e-8bc1-e87ec5bfec3f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@enoodle enoodle marked this pull request as ready for review May 11, 2026 08:02
@enoodle enoodle enabled auto-merge (squash) May 11, 2026 11:43
@enoodle enoodle merged commit 6a18396 into kai-scheduler:v0.9 May 11, 2026
7 of 8 checks passed
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.

2 participants