-
Notifications
You must be signed in to change notification settings - Fork 925
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
[LIBCLOUD-780] Add Rackspace RDNS support #652
Conversation
This depends on #650 going in first. |
Hrm... testing this against the real API there's some issues still. Closing it for now, will reopen when I sort 'em out. |
b7d239b
to
122ffc6
Compare
The documented response for listing PTR records is apparently not what is actually returned by the real API. Adjusted the code/tests to match reality. |
New DNS driver methods: * ex_iterate_ptr_records * ex_get_ptr_record * ex_create_ptr_record * ex_update_ptr_record * ex_delete_ptr_record This should cover all of the functionality offered by the Rackspace DNS API in regards to RDNS.
122ffc6
to
8ecd5d2
Compare
self.connection.set_context({'resource': 'record', 'id': record_id}) | ||
response = self.connection.request( | ||
action='/rdns/%s/%s' % (service_name, record_id)).object | ||
item = next(iter(response['recordsList']['records'])) |
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.
IIRC, this is only available in python 2.6+
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.
next
, that is
EDIT: Confirmed
#650 is closed. |
LGTM 👍 thanks Greg |
Great, thanks! :) |
New DNS driver methods:
This should cover all of the functionality offered by the Rackspace
DNS API in regards to RDNS.