Skip to content
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

provider/ovh: new provider, resource ovh_domain_zone #14775

Closed
wants to merge 5 commits into from
Closed

provider/ovh: new provider, resource ovh_domain_zone #14775

wants to merge 5 commits into from

Conversation

remijouannet
Copy link

@remijouannet remijouannet commented May 23, 2017

Hi,

after the pull request of @yanndegat i recreate my pull request to add this ressource to ovh provider

i'v written that simple plugin https://github.com/remijouannet/terraform-provider-ovh
with only one resource just to manipulate OVH domains,
so my PR include only one resource

however I don't know if the acceptance tests i wrote are enough or not,
i didn't write the terraform documentation yet as i don't know if my PR require more work or not

here is a little README:
howto

if you want to use the OVH API you have to generate:

An Application Key
An Application Secret Key
A Consumer Key

you can generate all three on this page
https://eu.api.ovh.com/createToken/

the following rights are needed for this plugin

GET : /domain/zone/*
PUT : /domain/zone/*
POST : /domain/zone/*
DELETE : /domain/zone/*
GET : /me

ovh.tf example

provider "ovh" {
application_key = "azrzqrgqvvdsfgsfffgc"
application_secret = "aztfqsqfsdcsdqezrfdvcx"
consumer_key = "aergfvdsrgtfbvcretfgd"
}

resource "ovh_domain_zone_record" "test" {
zone = "testdemo.ovh"
subDomain = "test"
fieldType = "A"
ttl = "3600"
target = "0.0.0.0"
}

testacc

export OVH_APPLICATION_KEY="arqzergfazfeef"
export OVH_APPLICATION_SECRET="aertedytfghftyhytghbgfv"
export OVH_CONSUMER_KEY="tyfghjhuyghvbjhuytghbjygh"
export OVH_ZONE="testdemo.ovh"

then use the makefile : make testacc TEST=./builtin/providers/ovh

@remijouannet remijouannet changed the title provider/ovh: new provider, resource ovh_domain_zon provider/ovh: new provider, resource ovh_domain_zone May 23, 2017
@stack72
Copy link
Contributor

stack72 commented May 24, 2017

Hi @remijouannet

Thanks for the work here - please can you enhance the PR with some documentation as well?

Thanks

Paul

@remijouannet
Copy link
Author

Hi,
done, is that enough ?

@stack72
Copy link
Contributor

stack72 commented Jun 27, 2017

@yanndegat thoughts on this PR would be awesome :)

@yanndegat
Copy link
Contributor

hi @stack72. will do
hi @remijouannet could you rebase your PR on the new https://github.com/terraform-providers/terraform-provider-ovh/ project please ?
and we'll start the review from there.

@stack72
Copy link
Contributor

stack72 commented Jul 5, 2017

@stack72 stack72 closed this Jul 5, 2017
@ghost
Copy link

ghost commented Apr 8, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 8, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants