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

[MCAPI-54] feat: add validation logic for zero boot volume case #471

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

okozachenko1203
Copy link
Member

@okozachenko1203 okozachenko1203 commented Jan 9, 2025

when both boot volume size and flavor's root_gb are zero, cluster node vm creation will be failed.
This commit prevents cluster creation with such params.

@okozachenko1203 okozachenko1203 requested a review from mnaser January 9, 2025 03:03
@okozachenko1203 okozachenko1203 changed the title [MCAPI-54] feat: add validation logic for zero boot vol case [MCAPI-54] feat: add validation logic for zero boot volume case Jan 9, 2025
@okozachenko1203
Copy link
Member Author

recheck

@okozachenko1203
Copy link
Member Author

recheck

1 similar comment
@mnaser
Copy link
Member

mnaser commented Jan 15, 2025

recheck

when both boot volume size and flavor's root_gb are zero, cluster node vm
creation will be failed.
This commit prevents cluster creation with such params.
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