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

Pick default datastore for extra disks #897

Merged
merged 2 commits into from
Nov 11, 2019
Merged

Conversation

koikonom
Copy link
Contributor

@koikonom koikonom commented Nov 4, 2019

When cloning virtual machines it is possible to define additional disks
to be created. If a datastore is not defined either for the disk or for
the virtual machine then the creation of the additional virtual machines
will fail.

This change introduces a fallback where if a datastore is not set we
pick the first datastore found that is associated with the virtual
machine.

When cloning virtual machines it is possible to define additional disks
to be created. If a datastore is not defined either for the disk or for
the virtual machine then the creation of the additional virtual machines
will fail.

This change introduces a fallback where if a datastore is not set we
pick the first datastore found that is associated with the virtual
machine.
@koikonom koikonom requested a review from a team November 4, 2019 21:59
@ghost ghost added the size/xs Relative Sizing: Extra-Small label Nov 4, 2019
@koikonom
Copy link
Contributor Author

koikonom commented Nov 4, 2019

Fixes #883

@koikonom koikonom merged commit c26e5f0 into master Nov 11, 2019
@koikonom koikonom deleted the gh883-fallback-datastore branch November 11, 2019 16:12
@vkmellon
Copy link

Hi! When it will be released?

@ghost ghost locked and limited conversation to collaborators Apr 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/xs Relative Sizing: Extra-Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants