Skip to content

Remote vm cicd #4489

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Remote vm cicd #4489

wants to merge 7 commits into from

Conversation

LakshK98
Copy link
Contributor

@LakshK98 LakshK98 commented Jul 1, 2025

Description

This pull request introduces support for running CI/CD scripts on remote virtual machines (VMs) by adding new parameters and functionality across multiple scripts. Closes #3823

Script Enhancements for Remote VM Support:

  • Deployment Script Updates:

    • Enhanced scripts/deploy-ebpf.ps1.in to support a --remote_vm option for specifying remote VM IP address. Added logic for copying files to remote VMs using PowerShell remoting.
  • Setup Script Updates:

    • Modified scripts/setup_ebpf_cicd_tests.ps1 and scripts/execute_ebpf_cicd_tests.ps1 to support remote VM setup by adding an IsVMRemote parameter.

Testing

Local test runs on nebula vm.

Documentation

Added a remote-vm-setup.md file with instructions on how to setup a remote vm and run scripts on it.

Installation

NA

@Alan-Jowett Alan-Jowett requested a review from Copilot July 8, 2025 20:38
Copy link

@Copilot 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 executing CI/CD scripts and deployments on remote VMs by introducing new parameters, branching logic, and documentation.

  • Introduces an IsVMRemote switch and remote‐execution branches in VM test, setup, and execution scripts.
  • Extends the deployment script with a --remote_vm option to copy files via PowerShell remoting.
  • Provides a new remote-vm-setup.md guide and updates InstallEbpf.md to reference remote setup.

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
scripts/vm_run_tests.psm1 Added IsVMRemote param and branch for Invoke-Command -ComputerName
scripts/setup_ebpf_cicd_tests.ps1 Added IsVMRemote switch and a remote‐VM setup block
scripts/execute_ebpf_cicd_tests.ps1 Passed IsVMRemote into job scriptblock and parameters
scripts/deploy-ebpf.ps1.in Added --remote_vm option and WSMan copy logic
scripts/config_test_vm.psm1 Extended artifact export, install, and network init to handle remote sessions; added Invoke-CommandOnVM helper
docs/remote-vm-setup.md New documentation for configuring and using remote VMs
docs/InstallEbpf.md Updated link to include remote VM setup instructions
Comments suppressed due to low confidence (1)

scripts/vm_run_tests.psm1:8

  • New IsVMRemote logic branches have been added for Invoke-Command. Consider adding unit or integration tests to verify both remote and VM invocation paths.
    [Parameter(Mandatory = $false)][bool] $IsVMRemote = $false,

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.

For dev work flow, automation scripts (used in CICD) should work over IP addresses to communicate with test VMs
2 participants