This repository has been archived by the owner on Mar 6, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
-Short description: Makes easier to create a new VM and attach it to the vApp by providing the vapp template name and VM template name as input parameters. This also works by proving a vapp name and VM name to create a VM based on any VM from a different vApp.
-Detailed description: Adding a couple of methods to extend the functionality of vApp. Define method create_vm_from_template, this is an extension from add_vm in order to simplify the VM creation. In this method, a vapp template is retrieved from a catalog to be used as an input parameter for add_vm. But create_vm_from_template facilitates the process by accepting the vapp template name as a str parameter. The other method get_template, is used simply to get the tempalte resource the add_vm methods needs as a paramter.
For me as a developer was complex to figure out a way to create a VM based on a template and attached to my vApp by looking at the defined methods. I guess this method will make this operation more accesible and understandable for consumers.
This change is