Skip to content

WIP: feat: add ovh gateway module#83

Merged
fredleger merged 4 commits into
hoverkraft-tech:mainfrom
Libr4rian:ovh-gateway
Mar 17, 2026
Merged

WIP: feat: add ovh gateway module#83
fredleger merged 4 commits into
hoverkraft-tech:mainfrom
Libr4rian:ovh-gateway

Conversation

@Libr4rian
Copy link
Copy Markdown
Contributor

No description provided.

@fredleger fredleger changed the title feat: add ovh gateway module WIP: feat: add ovh gateway module Dec 17, 2025
Copy link
Copy Markdown
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.

same as #85. Sync all OVH provider versions before merging

Copy link
Copy Markdown
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

This PR introduces a new Terraform module for managing an OVH public cloud gateway, consistent with the existing OVH PCI module structure in this repository.

Changes:

  • Adds a new ovh/gateway Terraform module with variables, provider configuration, main locals, and the OVH gateway resource.
  • Exposes the created gateway’s ID via an output for use by consumers.
  • Documents the new module in a README generated with terraform-docs (structure aligned with other OVH modules, though currently with some formatting issues).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
ovh/gateway/variables.tf Defines common module inputs (name, customer, tags) plus gateway-specific inputs (cloud_project_id, model, region, network_uuid, subnet_id).
ovh/gateway/providers.tf Declares required Terraform and provider versions for ovh and time to support the gateway resource and timestamping.
ovh/gateway/outputs.tf Exposes the ovh_cloud_project_gateway.gateway.id so callers can reference the created gateway.
ovh/gateway/main.tf Sets up shared local interpolated_tags and a time_static.last_update resource, mirroring tagging patterns in other OVH modules.
ovh/gateway/gateway.tf Creates the ovh_cloud_project_gateway resource wired to the module inputs (project ID, model, region, network, subnet).
ovh/gateway/README.md Adds user-facing documentation for the new gateway module (requirements, providers, inputs/outputs), currently with naming and markdown table formatting issues.

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

Comment thread ovh/gateway/variables.tf Outdated
Comment thread ovh/gateway/providers.tf Outdated
Comment thread ovh/gateway/README.md Outdated
Comment thread ovh/gateway/README.md
Libr4rian and others added 3 commits March 15, 2026 20:55
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:37
@fredleger fredleger merged commit b388d21 into hoverkraft-tech:main Mar 17, 2026
1 check passed
@fredleger
Copy link
Copy Markdown
Contributor

@Libr4rian @ffederighi I merged it considering the "WIP" was a relicate of previous work. Was i right ?

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