Fix RayJob submitter pod affinity to use default config#6897
Merged
Conversation
Update the test to verify that the submitter pod template uses the default affinity from K8s plugin config rather than copying the head pod's affinity. Add clarifying comment for the affinity/tolerations assignment. Signed-off-by: Kevin Su <kevin@union.ai> Signed-off-by: Kevin Su <pingsutw@apache.org>
jeevb
reviewed
Feb 5, 2026
jeevb
approved these changes
Feb 5, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TL;DR
Fix the RayJob submitter pod template to use the default affinity from K8s plugin config, and update the corresponding test to verify this behavior correctly.
Type
Are all requirements met?
Complete description
The submitter pod template for RayJob was already using
config.GetK8sPluginConfig().DefaultAffinity, but the test was incorrectly verifying that the submitter pod affinity matched the head pod's affinity instead of the default config affinity. This PR:DefaultAffinityin the K8s plugin config and verify the submitter pod template uses it, rather than setting affinity on the head pod and expecting it to be copiedTracking Issue
NA
Follow-up issue
NA