feat: switch to ovh provider for buckets#85
Conversation
fredleger
changed the title
|
@Libr4rian i put it on Work in Progress for now don't hesitate to ping me when ready, i can help you if you need |
|
@Libr4rian any update here ? I would love to merge them. |
Yes you can ! |
fredleger
changed the title
fredleger
left a comment
fredleger
left a comment
There was a problem hiding this comment.
Since we have a breaking change we must before merging this one, upgrade all OVH deps in this repo. Then rebase this PR and merge it.
There was a problem hiding this comment.
Pull request overview
This pull request migrates the OVH bucket module from using the OpenStack provider to the native OVH provider version 2.1. The change modernizes the infrastructure code by using OVH's dedicated Terraform resources instead of the generic OpenStack API.
Changes:
- Upgraded OVH provider from version 0.26.0 to 2.1 and removed OpenStack provider dependency
- Replaced
openstack_objectstorage_container_v1resource withovh_cloud_project_storageresource - Refactored variables: renamed
regiontoregion_name, removedovh_write_user, and addedcloud_project_id,owner_id,versioning_enabled, andencryption_enabled
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| ovh/bucket/providers.tf | Updated OVH provider version constraint from ~> 0.26.0 to ~> 2.1 and removed OpenStack provider |
| ovh/bucket/variables.tf | Renamed region to region_name, removed ovh_write_user, and added new variables for cloud project ID, owner ID, versioning, and encryption |
| ovh/bucket/bucket.tf | Replaced OpenStack container resource with OVH cloud project storage resource, including versioning and encryption configuration |
| ovh/bucket/main.tf | Updated variable reference from var.region to var.region_name |
| ovh/bucket/outputs.tf | Updated resource references from openstack_objectstorage_container_v1.bucket to ovh_cloud_project_storage.storage |
| ovh/bucket/README.md | Updated documentation to reflect new provider version, variables, and resources |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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>
No description provided.