Skip to content

Getting the full path and using a different condition to fix issue. #1099

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

Conversation

Krioks
Copy link

@Krioks Krioks commented Nov 8, 2021

SUMMARY

Fixes #1090 when creating / deleting a VM with vmware_guest it could be created/deleted in an another folder.
As mentioned in this issue, the fix was to compare VM paths correctly.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

vmware_guest causes an issue when creating / deleting a VM in folder with similar name, e.g. :

└───Project
    ├───aaa1
    ├───aaa111
    │               test1
    └───aaa1111
When I want to create a virtual machine with name test1 in the aaa1 folder, the virtual machine is created (customized) in the aaa111 folder, instead of the aaa1 folder. And, when I try to delete the test1 machine from the aaa1 folder, I will delete it from the aaa111 folder.

Using a new function instead of an old block of code.
Rewritten condition for getting a vm_obj.
@Krioks Krioks closed this Nov 11, 2021
@Krioks Krioks reopened this Nov 11, 2021
@Krioks Krioks force-pushed the #1090_correctly_getting_the_folder_name branch from 5c629fa to 24e7a5c Compare November 11, 2021 08:00
@goneri
Copy link
Member

goneri commented Jan 30, 2022

recheck

@mariolenz
Copy link
Collaborator

There hasn't been any activity on this PR for quite some time, so I'll close it for now. Feel free to re-open it or open a new one if you think it's important.

@mariolenz mariolenz closed this Apr 24, 2023
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.

vmware_guest create/delete VM in another folder
3 participants