Skip to content

Conversation

@tenthirtyam
Copy link
Collaborator

@tenthirtyam tenthirtyam commented Dec 11, 2025

Description

Warning

End-to-end testing is still pending.

Extends the vsphere-clone builder to support using remote OVF and OVA files as clone sources, in addition to the current support for standard virtual machine templates. Remote OVF/OVA support enables users to deploy virtual machines directly from OVF/OVA files accessible via HTTP/HTTPS URLs using vSphere's native OVF Manager capabilities available in the vmware/govmomi SDK.

It adds a new remote_source configuration block, updates documentation, and implements comprehensive validation and tests to ensure correct usage and mutual exclusivity between template and remote_source.

Resolved Issues

Closes #566

Rollback Plan

Revert commit.

Changes to Security Controls

None.

@tenthirtyam tenthirtyam added this to the v2.1.0 milestone Dec 11, 2025
@tenthirtyam tenthirtyam self-assigned this Dec 11, 2025
@tenthirtyam tenthirtyam added enhancement Enhancement builder/vsphere-clone Builder: vsphere-clone labels Dec 11, 2025
@tenthirtyam tenthirtyam force-pushed the feat/remote-ovf-clone-source branch 2 times, most recently from d0b7a85 to 392c0a9 Compare December 15, 2025 22:27
@tenthirtyam tenthirtyam changed the title [wip] feat: add remote ovf/ova source support feat: add remote ovf/ova source support Dec 15, 2025
@tenthirtyam tenthirtyam marked this pull request as ready for review December 15, 2025 22:44
@tenthirtyam tenthirtyam requested a review from a team as a code owner December 15, 2025 22:44
Copilot AI review requested due to automatic review settings December 15, 2025 22:44
Copy link

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 extends the vsphere-clone builder to support deploying VMs from remote OVF/OVA files accessible via HTTP/HTTPS URLs, in addition to the existing template-based cloning. The implementation uses vSphere's native OVF Manager capabilities and includes comprehensive validation, authentication support, TLS configuration, credential sanitization, and extensive test coverage.

Key Changes:

  • Adds remote_source configuration block with URL, authentication credentials, and TLS options
  • Implements mutual exclusivity between template and remote_source
  • Adds OVF deployment functionality in the driver layer with progress monitoring
  • Includes comprehensive credential sanitization to prevent exposure in logs and error messages

Reviewed changes

Copilot reviewed 20 out of 20 changed files in this pull request and generated no comments.

Show a summary per file
File Description
builder/vsphere/clone/step_clone.go Adds RemoteSourceConfig struct, OVF deployment logic, and credential sanitization
builder/vsphere/driver/driver.go Implements OVF deployment, progress monitoring, error handling, and validation
builder/vsphere/driver/driver_mock.go Adds mock implementations for OVF operations
builder/vsphere/driver/driver_test.go Comprehensive tests for OVF functionality
builder/vsphere/clone/step_clone_test.go Tests for remote source detection and deployment
builder/vsphere/clone/config.go Updates code generation directive
docs-partials/**/*.mdx Documentation for remote source configuration
Test files HTTP server, integration tests, and credential handling tests

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tenthirtyam tenthirtyam marked this pull request as draft December 15, 2025 23:00
@tenthirtyam tenthirtyam force-pushed the feat/remote-ovf-clone-source branch 2 times, most recently from 11f4a7c to 8b1e9e3 Compare January 28, 2026 13:00
Extends the `vsphere-clone` builder to support using remote OVF and OVA files as clone sources, in addition to the current support for standard virtual machine templates. Remote OVF/OVA support enables users to deploy virtual machines directly from OVF/OVA files accessible via HTTP/HTTPS URLs using vSphere's native OVF Manager capabilities available in the `vmware/govmomi` SDK.

Signed-off-by: Ryan Johnson <ryan@tenthirtyam.otg>
@tenthirtyam tenthirtyam force-pushed the feat/remote-ovf-clone-source branch from 8b1e9e3 to 605173b Compare January 29, 2026 02:36
@github-actions github-actions bot added documentation Documentation needs-review Needs Review size/xl Relative Sizing: Extra-Large labels Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builder/vsphere-clone Builder: vsphere-clone documentation Documentation enhancement Enhancement needs-review Needs Review size/xl Relative Sizing: Extra-Large

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extend the vsphere-clone builder to support using remote OVF and OVA files as clone sources

2 participants