added reverse-dns capabilities for cloud servers

Author: kwminnick

README.md

@@ -46,6 +46,15 @@ You'll find the complete documentation on the shell by running ``rackdns help``:
     		domain-modify       Modify a domain.
     		domain-show         Show details about the given domain
     		limits              List all applicable limits.
+    		rdns-create         Add a reverse DNS record to the specified domain.
+    		rdns-delete         Remove one or all PTR records associated with a
+                        		Rackspace Cloud device. Use the optional ip parameter
+                        		to specify a specific record to delete. Omitting this
+                        		parameter removes all PTR records associated with the
+                        		specified device.
+    		rdns-list           List all PTR records configured for the specified
+                        		Cloud Server.
+    		rdns-modify         Modify a reverse DNS record to the specified domain.
     		record-create       Add new record to the specified domain.
     		record-delete       Delete a record of the specified domain.
     		record-list         Print a list of records for the given domain.

dnsclient/v1_0/records.py

@@ -73,7 +73,28 @@ def create(self, args, domainId):
                 "priority" : args.priority                
             } ]
         }
-        return self._create_async('/domains/%s/records' % base.getid(domainId), body, return_raw=False, response_key="")
+        url = '/domains/%s/records' % base.getid(domainId)
+        
+        if args.type == "PTR":
+            url = '/rdns'
+            body = {
+                "recordsList" : {
+                    "records" : [ {
+                                   "name" : args.name,
+                                   "comment" : args.comment,
+                                   "ttl" : int(args.ttl),
+                                   "type" : args.type,
+                                   "data" : args.data               
+                                   } ]
+                    },
+                "link" : {
+                    "content" : "",
+                    "href" : args.server_href,
+                    "rel" : "cloudServersOpenStack"
+                }
+            }      
+        
+        return self._create_async(url, body, return_raw=False, response_key="")
 
     def modify(self, args, domainId):
         """
@@ -98,7 +119,30 @@ def modify(self, args, domainId):
             "data" : args.data,
             "priority" : args.priority                
         }
-        return self._update('/domains/%s/records/%s' % (base.getid(domainId), base.getid(args.record_id)) , body, return_raw=False, response_key="")
+        url = '/domains/%s/records/%s' % (base.getid(domainId), base.getid(args.record_id))
+        
+        if hasattr(args, 'type'):
+            if args.type == "PTR":
+                url = '/rdns'
+                body = {
+                        "recordsList" : {
+                                "records" : [ {
+                                   "name" : args.name,
+                                   "id" : args.record_id,
+                                   "comment" : args.comment,
+                                   "ttl" : int(args.ttl),
+                                   "type" : args.type,
+                                   "data" : args.data               
+                                   } ]
+                        },
+                        "link" : {
+                                  "content" : "",
+                                  "href" : args.server_href,
+                                  "rel" : "cloudServersOpenStack"
+                        }
+                }      
+
+        return self._update(url, body, return_raw=False, response_key="")
 
     def delete(self, domainId, recordId):
         """
@@ -108,3 +152,23 @@ def delete(self, domainId, recordId):
         :param recordId: The ID of the :class:`Record` to delete.
         """
         self._delete("/domains/%s/records/%s" % (base.getid(domainId), base.getid(recordId)))
+        
+    def rdns_list(self, href):     
+        """
+        List all PTR records configured for the specified Cloud device.
+
+        :param href: The href of the device to get .
+        :rtype: :class:`Record`
+        """
+        return self._list("/rdns/cloudServersOpenStack?href=%s" % href, "records") 
+
+    def rdns_delete(self, href, ip):
+        """
+        Remove one or all PTR records associated with a Rackspace Cloud device. 
+        Use the optional ip query parameter to specify a specific record to delete. 
+        Omitting this parameter removes all PTR records associated with the specified device.
+
+        :param href: The ID of the device to delete.
+        :param ip: The ip of the specific record to delete.
+        """
+        self._delete("/rdns/cloudServersOpenStack?href=%s&ip=%s" % (href, ip))

dnsclient/v1_0/shell.py

@@ -117,7 +117,7 @@ def do_record_list(cs, args):
     utils.print_list(record_list, columns)
 
 @utils.arg('domain',
-           metavar='name',
+           metavar='domain',
            help="Specifies the domain or subdomain. Must be a valid existing domain (example.com)")
 @utils.arg('--name',
            metavar='<name>',
@@ -190,3 +190,85 @@ def do_record_delete(cs, args):
     """Delete a record of the specified domain."""
     domainId = utils.find_resource(cs.domains, args.domain)
     cs.records.delete(domainId, args.record_id)
+
+@utils.arg('domain',
+           metavar='domain',
+           help="Specifies the domain or subdomain. Must be a valid existing domain (example.com)")
+@utils.arg('--name',
+           metavar='<name>',
+           required=True,
+           help="The full name of the new record (ftp.example.com)")
+@utils.arg('--data',
+           metavar='<data>',
+           required=True,
+           help="The data field, must be a valid IPv4 or IPv6 IP address")
+@utils.arg('--server-href',
+           metavar='<server_href>',
+           required=True,
+           help="The link to the Rackspace Cloud Server that this will reference.")
+@utils.arg('--ttl',
+           default=3600,
+           metavar='<ttl>',
+           help="If specified, must be greater than 300. The default value, if not specified, is 3600.")
+@utils.arg('--comment',
+           default=None,
+           metavar='<comment>',
+           help="If included, its length must be less than or equal to 160 characters.")
+def do_rdns_create(cs, args):
+    """Add a reverse DNS record to the specified domain."""
+    args.priority = None
+    args.type = "PTR"
+    do_record_create(cs, args)
+
+@utils.arg('href',
+           metavar='href',
+           help="The link to the Rackspace Cloud Server that this will reference.")    
+def do_rdns_list(cs, args):
+    """List all PTR records configured for the specified Cloud Server."""
+    record_list = cs.records.rdns_list(args.href)
+    columns = ['ID', 'Name', 'Type', "Data", "TTL", "Comment"]
+    utils.print_list(record_list, columns)
+    
+@utils.arg('domain',
+           metavar='domain',
+           help="Specifies the domain or subdomain. Must be a valid existing domain (example.com)")
+@utils.arg('--record-id',
+           metavar='<record_id>',
+           required=True,
+           help="The id of the PTR record to modify.")
+@utils.arg('--name',
+           metavar='<name>',
+           required=True,
+           help="The full name of the new record (ftp.example.com)")
+@utils.arg('--data',
+           metavar='<data>',
+           required=True,
+           help="The data field, must be a valid IPv4 or IPv6 IP address")
+@utils.arg('--server-href',
+           metavar='<server_href>',
+           required=True,
+           help="The link to the Rackspace Cloud Server that this will reference.")
+@utils.arg('--ttl',
+           default=3600,
+           metavar='<ttl>',
+           help="If specified, must be greater than 300. The default value, if not specified, is 3600.")
+@utils.arg('--comment',
+           default=None,
+           metavar='<comment>',
+           help="If included, its length must be less than or equal to 160 characters.")
+def do_rdns_modify(cs, args):
+    """Modify a reverse DNS record to the specified domain."""
+    args.priority = None
+    args.type = "PTR"
+    do_record_modify(cs, args)
+
+@utils.arg('href',
+           metavar='href',
+           help="The link to the Rackspace Cloud Server that this will reference.")
+@utils.arg('--ip',
+           metavar='<ip>',
+           default="",
+           help="Use the optional ip parameter to specify a specific record to delete.  Omitting this parameter removes all PTR records associated with the specified device.")    
+def do_rdns_delete(cs, args):
+    """Remove one or all PTR records associated with a Rackspace Cloud device.  Use the optional ip parameter to specify a specific record to delete.  Omitting this parameter removes all PTR records associated with the specified device."""
+    cs.records.rdns_delete(args.href, args.ip)

setup.py

@@ -36,7 +36,7 @@ def read_file(file_name):
     #test_suite="nose.collector",
     cmdclass=setup.get_cmdclass(),
     classifiers=[
-        "Development Status :: 5 - Production/Stable",
+        "Development Status :: 4 - Beta",
         "Environment :: Console",
         "Intended Audience :: Developers",
         "Intended Audience :: Information Technology",