Skip to content

test(cli): add tests for provision and validate commands #613

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.

Sign up for GitHub

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

Closed
ArangoGutierrez wants to merge 3 commits into from
Closed

test(cli): add tests for provision and validate commands #613

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
30d594d
feat(cli): add validate command for environment file validation
ArangoGutierrez Jan 26, 2026
60d47f4
feat(cli): add provision command for standalone provisioning
ArangoGutierrez Jan 26, 2026
5f931ab
test(cli): add tests for provision and validate commands
ArangoGutierrez Feb 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 39 additions & 3 deletions cmd/cli/main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,17 @@
"github.com/NVIDIA/holodeck/cmd/cli/cleanup"
"github.com/NVIDIA/holodeck/cmd/cli/create"
"github.com/NVIDIA/holodeck/cmd/cli/delete"
"github.com/NVIDIA/holodeck/cmd/cli/describe"

Check failure on line 25 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/describe: import lookup disabled by -mod=vendor

Check failure on line 25 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / golang / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/describe: import lookup disabled by -mod=vendor
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

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

This import references a package "github.com/NVIDIA/holodeck/cmd/cli/describe" that does not appear to exist in the codebase. The command is registered on line 116 but the package is missing.

Suggested change
"github.com/NVIDIA/holodeck/cmd/cli/describe"

Copilot uses AI. Check for mistakes.
"github.com/NVIDIA/holodeck/cmd/cli/dryrun"
"github.com/NVIDIA/holodeck/cmd/cli/get"

Check failure on line 27 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/get: import lookup disabled by -mod=vendor

Check failure on line 27 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / golang / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/get: import lookup disabled by -mod=vendor
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

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

This import references a package "github.com/NVIDIA/holodeck/cmd/cli/get" that does not appear to exist in the codebase. The command is registered on line 118 but the package is missing.

Suggested change
"github.com/NVIDIA/holodeck/cmd/cli/get"

Copilot uses AI. Check for mistakes.
"github.com/NVIDIA/holodeck/cmd/cli/list"
oscmd "github.com/NVIDIA/holodeck/cmd/cli/os"
"github.com/NVIDIA/holodeck/cmd/cli/provision"
"github.com/NVIDIA/holodeck/cmd/cli/scp"

Check failure on line 31 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/scp: import lookup disabled by -mod=vendor

Check failure on line 31 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / golang / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/scp: import lookup disabled by -mod=vendor
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

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

This import references a package "github.com/NVIDIA/holodeck/cmd/cli/scp" that does not appear to exist in the codebase. The command is registered on line 122 but the package is missing.

Suggested change
"github.com/NVIDIA/holodeck/cmd/cli/scp"

Copilot uses AI. Check for mistakes.
"github.com/NVIDIA/holodeck/cmd/cli/ssh"

Check failure on line 32 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/ssh: import lookup disabled by -mod=vendor

Check failure on line 32 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / golang / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/ssh: import lookup disabled by -mod=vendor
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

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

This import references a package "github.com/NVIDIA/holodeck/cmd/cli/ssh" that does not appear to exist in the codebase. The command is registered on line 123 but the package is missing.

Suggested change
"github.com/NVIDIA/holodeck/cmd/cli/ssh"

Copilot uses AI. Check for mistakes.
"github.com/NVIDIA/holodeck/cmd/cli/status"
"github.com/NVIDIA/holodeck/cmd/cli/update"

Check failure on line 34 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/update: import lookup disabled by -mod=vendor

Check failure on line 34 in cmd/cli/main.go

View workflow job for this annotation

GitHub Actions / golang / Build 

cannot find module providing package github.com/NVIDIA/holodeck/cmd/cli/update: import lookup disabled by -mod=vendor
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

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

This import references a package "github.com/NVIDIA/holodeck/cmd/cli/update" that does not appear to exist in the codebase. The command is registered on line 125 but the package is missing.

Suggested change
"github.com/NVIDIA/holodeck/cmd/cli/update"

Copilot uses AI. Check for mistakes.
"github.com/NVIDIA/holodeck/cmd/cli/validate"
"github.com/NVIDIA/holodeck/internal/logger"

cli "github.com/urfave/cli/v2"
Expand Down Expand Up @@ -63,9 +70,22 @@
# List all environments
holodeck list

# List environments in JSON format
holodeck list -o json

# Get status of a specific environment
holodeck status <instance-id>

# SSH into an instance
holodeck ssh <instance-id>

# Run a command on an instance
holodeck ssh <instance-id> -- nvidia-smi

# Copy files to/from an instance
holodeck scp ./local-file.txt <instance-id>:/remote/path/
holodeck scp <instance-id>:/remote/file.log ./local/

# Delete an environment
holodeck delete <instance-id>

Expand Down Expand Up @@ -93,10 +113,17 @@
cleanup.NewCommand(log),
create.NewCommand(log),
delete.NewCommand(log),
describe.NewCommand(log),
dryrun.NewCommand(log),
get.NewCommand(log),
list.NewCommand(log),
oscmd.NewCommand(log),
provision.NewCommand(log),
scp.NewCommand(log),
ssh.NewCommand(log),
status.NewCommand(log),
update.NewCommand(log),
validate.NewCommand(log),
}

// Custom help template
Expand Down Expand Up @@ -129,18 +156,27 @@
# List all environments
{{.Name}} list

# List environments in JSON format
{{.Name}} list -o json

# Get status of a specific environment
{{.Name}} status <instance-id>

# SSH into an instance
{{.Name}} ssh <instance-id>

# Run a command on an instance
{{.Name}} ssh <instance-id> -- nvidia-smi

# Copy files to/from an instance
{{.Name}} scp ./local-file.txt <instance-id>:/remote/path/

# Delete an environment
{{.Name}} delete <instance-id>

# Clean up AWS VPC resources
{{.Name}} cleanup vpc-12345678

# Use a custom cache directory
{{.Name}} --cachepath /path/to/cache create -f env.yaml

For more information about a command, run:
{{.Name}} help <command>
`
Expand Down
Loading
Loading