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

Fix NnfAccess ClientMount count #426

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Fix NnfAccess ClientMount count #426

merged 1 commit into from
Dec 9, 2024

Conversation

bdevcich
Copy link
Contributor

@bdevcich bdevcich commented Dec 6, 2024

This was a workaround to account for creating multiple OSTs in an
unorthodox manner in the Servers resource. Flux won't create multiple
allocation sets on a single rabbit, but rather use the count when there
are multiple allocations on a single rabbit.

This workaround causes a bug and is not needed.

Signed-off-by: Blake Devcich [email protected]

This was a workaround to account for creating multiple OSTs in an
unorthodox manner in the Servers resource. Flux won't create multiple
allocation sets on a single rabbit, but rather use the count when there
are multiple allocations on a single rabbit.

This workaround causes a bug and is not needed.

Signed-off-by: Blake Devcich <[email protected]>
@bdevcich bdevcich merged commit 4c04a02 into master Dec 9, 2024
3 checks passed
@bdevcich bdevcich deleted the remove-access-count branch December 9, 2024 16:05
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.

3 participants