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 dns support for dnspod provider #787
Conversation
Great, thanks. Is the PR already for a review, or are you still working on it? |
@Kami the pr is ready for review :) |
url, body, headers): | ||
body = self.fixtures.load('delete_record_record_does_not_exist.json') | ||
|
||
return httplib.OK, body, {}, httplib.responses[httplib.OK] |
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.
Just curious - does API indeed return 200 status code for all the different responses?
Thanks 👍 I've added some in-line comments. |
@jetbird Did you push your latest changes? Since I don't see any new commits (license header is still missing, etc.). |
@Kami done :) |
creates, removes uneeded default headers, calling response.object instead of response.parse_body
Closes #787 Signed-off-by: Tomaz Muraus <tomaz@tomaz.me>
Closes #787 Signed-off-by: Tomaz Muraus <tomaz@tomaz.me>
…ython versions Closes #787 Signed-off-by: Tomaz Muraus <tomaz@tomaz.me>
Closes #787 Signed-off-by: Tomaz Muraus <tomaz@tomaz.me>
Closes #787 Signed-off-by: Tomaz Muraus <tomaz@tomaz.me>
…m creates, removes uneeded default headers, calling response.object instead of response.parse_body Closes #787 Signed-off-by: Tomaz Muraus <tomaz@tomaz.me>
@jetbird Thanks. I've made some changes and fixes (6594a46, defff8a) and merged patch into trunk. As you can see in the second commit (defff8a), |
@jetbird Can you please test latest changes against the live provider API? |
Thanks for explaining the fixes :) Of course, I will test it now. |
@Kami Tested. It works fine for me :) |
@jetbird Great 👍 It would also be good if you can add some basic docs for the driver and contact the provider (tall them about Libcloud, driver, etc. and ask them if it's possible to mention it somewhere on the page / docs, etc.). |
DNS support for DNSPod provider
Description
I coded the dns driver for DNSPod on libcloud/dns/drivers/dnspod.py module. In order for this driver to work the libcloud/common/dnspod.py module is needed. The unitttests for this driver are coded in the libcloud/test/dns/test_dnspod.py module.
Status
Checklist (tick everything that applies)