Skip to content

Declarative storage network IP range#299

Open
bddvlpr wants to merge 1 commit into
apache:mainfrom
bddvlpr:feat/storage-network-range
Open

Declarative storage network IP range#299
bddvlpr wants to merge 1 commit into
apache:mainfrom
bddvlpr:feat/storage-network-range

Conversation

@bddvlpr

@bddvlpr bddvlpr commented Jul 1, 2026

Copy link
Copy Markdown

Adds a new cloudstack_storage_network_ip_range resource for managing a pod's storage network through Terraform. Follows the ip_range resource closely.

Acceptance tests cover creation, update and omitted VLAN cases. Resource documentation and sidebar entry are included.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Terraform resource to declaratively manage CloudStack storage network IP ranges at the pod level, aligning with existing IP-range-style resources and including docs/nav updates.

Changes:

  • Introduces cloudstack_storage_network_ip_range resource with CRUD + import support.
  • Adds acceptance tests covering create and update flows, plus a “VLAN omitted” creation case.
  • Adds resource documentation and registers it in the website sidebar.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
cloudstack/resource_cloudstack_storage_network_ip_range.go Implements the new resource schema and CRUD operations against CloudStack’s storage network IP range APIs.
cloudstack/resource_cloudstack_storage_network_ip_range_test.go Adds acceptance tests for create/update and creation without VLAN set.
cloudstack/provider.go Registers the new resource in the provider’s resource map.
website/docs/r/storage_network_ip_range.html.markdown Adds user-facing documentation for the new resource, including example and import instructions.
website/cloudstack.erb Adds the resource to the docs sidebar navigation.

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

Comment thread cloudstack/resource_cloudstack_storage_network_ip_range.go
@sureshanaparti sureshanaparti requested a review from vishesh92 July 10, 2026 08:22
@sureshanaparti sureshanaparti added this to the v0.7.0 milestone Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants