Skip to content

feat: clone from .ovf/.ova#373

Merged
kp2099 merged 1 commit intomainfrom
feat/clone-from-ovf-ova
Oct 16, 2025
Merged

feat: clone from .ovf/.ova#373
kp2099 merged 1 commit intomainfrom
feat/clone-from-ovf-ova

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

@tenthirtyam tenthirtyam commented Aug 20, 2025

Description

Add support for cloning virtual machines from .ovf and .ova files using ovftool, in addition to .vmx files.

Also reorganizes the examples and includes examples for vmware-vmx which include the capabilities added in this pull request.

Resolved Issues

Closes #9

Rollback Plan

Revert commit.

Changes to Security Controls

None.

@tenthirtyam tenthirtyam added this to the v2.y.z milestone Aug 20, 2025
@tenthirtyam tenthirtyam self-assigned this Aug 20, 2025
@tenthirtyam tenthirtyam added enhancement Enhancement builder/vmware-vmx Builder: vmware-vmx refactor Refactor labels Aug 20, 2025
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch from 1eb22a6 to 7efa6db Compare August 25, 2025 16:50
@tenthirtyam tenthirtyam changed the title feat: clone from .ovf/.ova [wip] feat: clone from .ovf/.ova Aug 28, 2025
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch 6 times, most recently from 9ceb886 to fb63642 Compare September 2, 2025 17:22
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch 2 times, most recently from 46f0b3d to be5e16c Compare October 1, 2025 03:00
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch 15 times, most recently from 394acca to 9ce0a6f Compare October 11, 2025 15:25
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch 3 times, most recently from a28903d to df78c0e Compare October 13, 2025 19:07
@tenthirtyam tenthirtyam changed the title [wip] feat: clone from .ovf/.ova feat: clone from .ovf/.ova Oct 13, 2025
@tenthirtyam tenthirtyam modified the milestones: v2.y.z, v2.0.0 Oct 13, 2025
@tenthirtyam tenthirtyam marked this pull request as ready for review October 13, 2025 19:10
@tenthirtyam tenthirtyam requested a review from a team as a code owner October 13, 2025 19:10
Copilot AI review requested due to automatic review settings October 13, 2025 19:10
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for cloning virtual machines from .ovf and .ova files using ovftool, in addition to the existing .vmx file support. The changes also reorganize the examples directory and add comprehensive examples for the vmware-vmx builder.

  • Add .ovf/.ova cloning capability using ovftool with new configuration parameters
  • Reorganize examples into separate iso/ and vmx/ directories with platform-specific configurations
  • Update documentation to reflect the new cloning capabilities

Reviewed Changes

Copilot reviewed 30 out of 31 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
builder/vmware/vmx/step_clone_vmx.go Core implementation of .ovf/.ova cloning using ovftool
builder/vmware/vmx/config.go Added guest_os_type and version configuration parameters with validation
builder/vmware/vmx/config.hcl2spec.go Generated HCL2 specification for new configuration fields
builder/vmware/vmx/builder.go Integration of new parameters into the build process
example/vmx/ New vmware-vmx examples with support for .vmx, .ovf, and .ova sources
example/iso/ Reorganized vmware-iso examples with updated variable structure
docs-partials/ Updated documentation to reflect new source file support

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@tenthirtyam tenthirtyam requested review from kp2099 and removed request for Copilot October 13, 2025 19:11
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch from df78c0e to 723031d Compare October 13, 2025 19:13
@tenthirtyam tenthirtyam requested a review from Copilot October 13, 2025 19:14
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 30 out of 31 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch from 723031d to 66c7f83 Compare October 13, 2025 19:17
@tenthirtyam tenthirtyam requested a review from Copilot October 13, 2025 19:17
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 30 out of 31 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Add support for cloning virtual machines from .ovf and .ova files using ovftool, in addition to .vmx files.

Signed-off-by: Ryan Johnson <ryan@tenthirtyam.org>
@tenthirtyam tenthirtyam force-pushed the feat/clone-from-ovf-ova branch from 66c7f83 to 3f5dc81 Compare October 13, 2025 19:21
@tenthirtyam tenthirtyam requested a review from Copilot October 13, 2025 19:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 30 out of 31 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@tenthirtyam tenthirtyam requested a review from Copilot October 13, 2025 19:22
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 30 out of 31 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@kp2099 kp2099 merged commit 80323e6 into main Oct 16, 2025
14 checks passed
@kp2099 kp2099 deleted the feat/clone-from-ovf-ova branch October 16, 2025 11:07
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

builder/vmware-vmx Builder: vmware-vmx enhancement Enhancement refactor Refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vmware-vmx: builder ova import functionality

3 participants