Conversation
Implement the complete `civo vpc` command hierarchy using the VPC-prefixed methods from civogo SDK v0.7.0-alpha1. All commands target the /v2/vpc/ API path and follow existing CLI patterns for output formatting, batch deletion, region handling, and flag conventions. Command tree: civo vpc network ls | create | show | update | remove civo vpc subnet ls | create | show | remove | attach | detach civo vpc firewall ls | create | show | update | remove civo vpc firewall rule ls | create | remove civo vpc loadbalancer ls | create | show | update | remove civo vpc ip ls | reserve | show | update | assign | unassign | delete Implementation details: - 37 new files in cmd/vpc/ covering network, subnet, firewall, firewall rules, load balancer, and reserved IP resources - Wire VPCCmd into rootCmd via cmd/root.go - Subnet commands require --network flag to scope operations to a network - Load balancer create/update support repeatable --backend flag with format ip:source_port:target_port:protocol:health_check_port - IP assign uses --resource-id and --resource-type for generic resource assignment (not limited to instances) - Firewall list resolves network IDs to names via ListVPCNetworks - Firewall show includes inline rule listing - All remove commands support batch deletion with confirmation prompts - Three-way output support: json, custom fields, and human-readable
added 2 commits
March 3, 2026 15:14
- Fix gofmt alignment in var blocks for vpc_firewall_create.go and vpc_loadbalancer_update.go - Simplify createRules assignment to satisfy staticcheck QF1007
Member
|
Is this something we want? Aren't we making our CLI API more complicated? |
giornetta
approved these changes
Mar 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implement the complete
civo vpccommand hierarchy using the VPC-prefixed methods from civogo SDK v0.7.0-alpha1. All commands target the /v2/vpc/ API path and follow existing CLI patterns for output formatting, batch deletion, region handling, and flag conventions.Command tree:
civo vpc network ls | create | show | update | remove
civo vpc subnet ls | create | show | remove | attach | detach
civo vpc firewall ls | create | show | update | remove
civo vpc firewall rule ls | create | remove
civo vpc loadbalancer ls | create | show | update | remove
civo vpc ip ls | reserve | show | update | assign | unassign | delete
Implementation details: