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

policies: add firstnodex to the known policies #1161

Merged
merged 1 commit into from Apr 6, 2024

Conversation

trws
Copy link
Member

@trws trws commented Apr 6, 2024

problem: the firstnodex policy was added and tested, but couldn't be selected by users because it wouldn't be seen as a known policy.

solution: accept the firstnodex value as a known policy in known_match_policy

problem: the firstnodex policy was added and tested, but couldn't be
selected by users because it wouldn't be seen as a known policy.

solution: accept the firstnodex value as a known policy in
`known_match_policy`
@trws trws requested a review from milroy April 6, 2024 03:27
Copy link
Member

@milroy milroy left a comment

Choose a reason for hiding this comment

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

Thanks for the quick PR. LGTM!

@trws trws added the merge-when-passing mergify.io - merge PR automatically once CI passes label Apr 6, 2024
Copy link

codecov bot commented Apr 6, 2024

Codecov Report

Merging #1161 (a587a97) into master (d0b9ac3) will not change coverage.
The diff coverage is 100.0%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1161   +/-   ##
======================================
  Coverage    73.9%   73.9%           
======================================
  Files         103     103           
  Lines       14395   14395           
======================================
  Hits        10643   10643           
  Misses       3752    3752           
Files Coverage Δ
resource/policies/dfu_match_policy_factory.cpp 92.8% <100.0%> (ø)

@mergify mergify bot merged commit 6e6576d into flux-framework:master Apr 6, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-when-passing mergify.io - merge PR automatically once CI passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants