Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport #19004 to release/1.5.x #19130

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Conversation

angrycub
Copy link
Contributor

[core] Honor job's namespace when checking distinct_hosts feasibility
Manual backport because of conflicts in upgrade-specific.mdx

[core] Honor job's namespace when checking `distinct_hosts` feasibility
* Update distinct_host feasibility checking to honor the job's namespace. Fixes #9792
* Added test to verify original condition and that fix resolved it.
* Added documentation

Manual backport due to merge conflicts in upgrade-specific.mdx
Comment on lines +31 to +32
[tasks within a group are scheduled on the same client node][group].
</Warning>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
[tasks within a group are scheduled on the same client node][group].
</Warning>
[tasks within a group are scheduled on the same client node][group].
</Warning>

I'm not sure if it matters, so feel free to ignore this, but I usually see these block surrounded by line breaks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's only required to stand off the opening tag when there's markdown inside the component. The closing one doesn't need a standoff.

@angrycub angrycub merged commit eb43489 into release/1.5.x Nov 20, 2023
21 checks passed
@angrycub angrycub deleted the cv-backport-19004-1.5.x branch November 20, 2023 19:03
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