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

vApp template capture support #652

Merged
merged 10 commits into from
Mar 8, 2024
Merged

Conversation

Didainius
Copy link
Collaborator

@Didainius Didainius commented Mar 1, 2024

This PR adds functionality to create vApp template from existing vApp or standalone VMs (which also have vApps, they're just hidden). The functionality is called "capture vApp template" and the functionality is introduced using Catalog.CaptureVappTemplate, Catalog.CaptureVappTemplateAsync and type types.CaptureVAppParams

Additionally there are some covenience changes in the code:

  • VApp.GetParentVDC becomes public (previously it was private method)
  • New method Org.GetVAppByHref to retrieve a vApp by given HREF
  • New methods VAppTemplate.GetCatalogItemHref and VAppTemplate.GetCatalogItemId that can return
    related catalog item ID and HREF of a vApp template

Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
@Didainius Didainius marked this pull request as ready for review March 5, 2024 05:10
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Copy link
Collaborator

@lvirbalas lvirbalas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Collaborator

@adambarreiro adambarreiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First scan! Just some thoughts

Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Copy link
Collaborator

@adambarreiro adambarreiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
Signed-off-by: Dainius Serplis <dserplis@vmware.com>
@Didainius Didainius merged commit e0a8d86 into vmware:main Mar 8, 2024
2 checks passed
@Didainius Didainius deleted the capture-template branch March 8, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants