-
Notifications
You must be signed in to change notification settings - Fork 144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding gateway and gateway interface resources #358
Conversation
* ovh_cloud_project_gateway * ovh_cloud_project_gateway_interface
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
thanks for your pull request.
Can you add acceptance tests for each of the new resources,
and also documentation pages?
Moreover, if you add two new resources, having datasources can be useful has well.
You can see an example of acceptance tests, documentation pages and datasources in another merged PRs and in the code of this repository.
Thanks :)
@durandx : did you know that you can already create a gateway with the openstack provider ? This page may help https://docs.ovh.com/gb/en/api/terraform-at-ovhcloud/ |
@scraly Thank you for your review and advices for this pull request. Unfortunately, right now, I have no time to work on acceptance tests and documentation + datasources... @yomovh Yes, that's what we used in the first place, but we were facing a problem for attaching an interface for a gateway created with openstack provider (maybe because we need to define a size for the gateway). Using the OVH API solved this problem, that's why we added this resource (in the ovh provider) using the OVH API. |
Thanks for the contribution :) Thanks :) |
Hi, I succesfully created a gateway but when I want to create a gateway_interface I have an issue:
|
Hi, We didn't run into this issue, probably because we used more than 1 subnet for the gateway. We have declared the first subnet to the Gateway resource (with ovh_cloud_project_gateway.gateway) and the others subnets have been attached to the gateway_interface (with ovh_cloud_project_gateway_interface.gateway_interface). |
Any update regarding this MR ? would be great to have a native interraction of OVH API for gateway in the provider |
Hi @rbeuque74 , can you have a look? thanks! :-) |
i've made the part of the MR with test and doc update |
Signed-off-by: DrummyFloyd <[email protected]> Related: ovh#358
Signed-off-by: DrummyFloyd <[email protected]> Related: ovh#358
Signed-off-by: DrummyFloyd <[email protected]> Related: ovh#358
* feat(cloud): add gateway ressource Signed-off-by: DrummyFloyd <[email protected]> Related: #358
Closing this PR as it has been replaced by: #697 |
Adding 2 new resources :