Skip to content

feat: switch to ovh provider for private subnet#84

Merged
fredleger merged 7 commits intohoverkraft-tech:mainfrom
Libr4rian:ovh-subnet
Mar 17, 2026
Merged

feat: switch to ovh provider for private subnet#84
fredleger merged 7 commits intohoverkraft-tech:mainfrom
Libr4rian:ovh-subnet

Conversation

@Libr4rian
Copy link
Contributor

No description provided.

@Libr4rian Libr4rian changed the title chore: switch to ovh provider for private subnet feat: switch to ovh provider for private subnet Nov 10, 2025
@fredleger fredleger changed the title feat: switch to ovh provider for private subnet WIP: feat: switch to ovh provider for private subnet Dec 17, 2025
Copy link
Contributor

@fredleger fredleger left a comment

Choose a reason for hiding this comment

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

this is related to #85
we must have a full change of ovh provider and indicate a breaking change in version number

@fredleger fredleger assigned fredleger and unassigned Libr4rian Jan 28, 2026
@fredleger fredleger requested a review from Copilot January 28, 2026 15:15
@fredleger fredleger changed the title WIP: feat: switch to ovh provider for private subnet feat: switch to ovh provider for private subnet Jan 28, 2026
Copy link
Contributor

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

Switches the ovh/pci-private-subnet Terraform module from managing subnets via the OpenStack provider to using the OVH provider’s ovh_cloud_project_network_private_subnet resource.

Changes:

  • Replaced the OpenStack subnet resource with the OVH private subnet resource and updated outputs accordingly.
  • Updated module inputs to match OVH subnet arguments (project/service name, region, CIDR, allocation range, DHCP/gateway toggles).
  • Removed the OpenStack provider requirement and bumped the OVH provider constraint; updated generated README tables.

Reviewed changes

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

Show a summary per file
File Description
ovh/pci-private-subnet/variables.tf Reworked inputs for OVH subnet resource (project id, region, CIDR, start/end, DHCP flags).
ovh/pci-private-subnet/providers.tf Dropped OpenStack provider and updated OVH provider version constraint.
ovh/pci-private-subnet/private-subnet.tf Switched resource implementation from OpenStack subnet to OVH private subnet.
ovh/pci-private-subnet/outputs.tf Updated outputs to reference the OVH subnet resource attributes.
ovh/pci-private-subnet/README.md Updated generated provider/resource/input tables for OVH provider usage.

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

Libr4rian and others added 6 commits March 15, 2026 20:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@fredleger fredleger self-requested a review March 17, 2026 17:43
@fredleger fredleger merged commit 0e9a973 into hoverkraft-tech:main Mar 17, 2026
1 check passed
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.

4 participants