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

Allow template names to include a VM folder path #48

Merged
merged 3 commits into from
Jan 12, 2019
Merged

Conversation

thheinen
Copy link
Collaborator

@thheinen thheinen commented Jan 9, 2019

Description

VMs or templates to be cloned can now be inside of folders or subfolders.

Issues Resolved

Until now, templates or VMs to be cloned had to be placed in the vSphere's datacenter root directly. For better organization, most setups use folders to group templates - often being nested several levels deep.

Due to the find_vm method being non-recursive, using anything within folders was not possible before (#29). By converting the lookup to use the inventory path instead (format: "DATACENTER/vm/FOLDER/VM"), this now provides the ability to use both templates within the datacenter root (just as before) as well as within subfolders.

Check List

@thheinen thheinen requested a review from a team as a code owner January 9, 2019 13:25
@ticobotbot

This comment was marked as outdated.

Signed-off-by: theinen <[email protected]>
… template, if no specific folder was given

Signed-off-by: theinen <[email protected]>
@tas50 tas50 changed the title Fix bug, so template names can include a VM folder path Allow template names to include a VM folder path Jan 12, 2019
@tas50 tas50 merged commit 1e71b1f into chef:master Jan 12, 2019
@thheinen thheinen deleted the bug-29 branch January 13, 2019 17:31
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