)]}'
{"designateclient/osc/plugin.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from openstackclient.common import utils as oscutils"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"DEFAULT_DNS_API_VERSION \u003d \u00272\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"API_NAME \u003d \u0027dns\u0027"},{"line_number":24,"context_line":"API_VERSION_OPTION \u003d \u0027os_dns_api_version\u0027"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_e42746c8","line":21,"updated":"2015-08-19 16:55:14.000000000","message":"It would be a bit better if you used \u0027DEFAULT_API_VERSION\u0027 here","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        metavar\u003d\u0027\u003cdns-api-version\u003e\u0027,"},{"line_number":42,"context_line":"        default\u003dshell.env("},{"line_number":43,"context_line":"            \u0027OS_DNS_API_VERSION\u0027,"},{"line_number":44,"context_line":"            default\u003dDEFAULT_DNS_API_VERSION),"},{"line_number":45,"context_line":"        help\u003d\u0027DNS API version, default\u003d\u0027 +"},{"line_number":46,"context_line":"             DEFAULT_DNS_API_VERSION +"},{"line_number":47,"context_line":"             \u0027 (Env: OS_DNS_API_VERSION)\u0027)"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_ff403336","line":44,"updated":"2015-08-19 16:55:14.000000000","message":"If you change to DEFAULT_API_VERSION and remove this default, that will allow people to set dns_api_version in their clouds.yaml file.\n\nThis gets done here: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/shell.py#L301","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"d961c92c95b29ff41a31f1fc220e8cc231addaf7","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from openstackclient.common import utils as oscutils"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"DEFAULT_DNS_API_VERSION \u003d \u00272\u0027"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"API_NAME \u003d \u0027dns\u0027"},{"line_number":24,"context_line":"API_VERSION_OPTION \u003d \u0027os_dns_api_version\u0027"}],"source_content_type":"text/x-python","patch_set":22,"id":"da20952f_e2e7bc44","line":21,"updated":"2015-09-01 14:45:26.000000000","message":"This should be DEFAULT_API_VERSION","commit_id":"0633f8e5c61583dce1f2ec827c54c9c29513f0bc"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"d961c92c95b29ff41a31f1fc220e8cc231addaf7","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        metavar\u003d\u0027\u003cdns-api-version\u003e\u0027,"},{"line_number":42,"context_line":"        default\u003dshell.env("},{"line_number":43,"context_line":"            \u0027OS_DNS_API_VERSION\u0027,"},{"line_number":44,"context_line":"            default\u003dDEFAULT_DNS_API_VERSION),"},{"line_number":45,"context_line":"        help\u003d\u0027DNS API version, default\u003d\u0027 +"},{"line_number":46,"context_line":"             DEFAULT_DNS_API_VERSION +"},{"line_number":47,"context_line":"             \u0027 (Env: OS_DNS_API_VERSION)\u0027)"}],"source_content_type":"text/x-python","patch_set":22,"id":"da20952f_a2246460","line":44,"updated":"2015-09-01 14:45:26.000000000","message":"This should not be defaulted here because then it cannot be overridden by os-client-config.","commit_id":"0633f8e5c61583dce1f2ec827c54c9c29513f0bc"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"569319e2fb8948d3ad133069e75a19475790393f","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    \"\"\"Hook to add global options.\"\"\""},{"line_number":37,"context_line":"    parser.add_argument("},{"line_number":38,"context_line":"        \u0027--os-dns-api-version\u0027,"},{"line_number":39,"context_line":"        metavar\u003d\u0027\u003cdns-api-version\u003e\u0027,"},{"line_number":40,"context_line":"        help\u003d\u0027DNS API version, default\u003d\u0027 +"},{"line_number":41,"context_line":"             DEFAULT_API_VERSION +"},{"line_number":42,"context_line":"             \u0027 (Env: OS_DNS_API_VERSION)\u0027)"}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_562e65a6","line":39,"updated":"2015-09-01 17:27:39.000000000","message":"I see I should of been more specific about the default I was talking about here.  I meant the default on the shell.env call, but the argument default using the environment should technically stay.  So add here:\n\n    default\u003dshell.env(\u0027OS_DNS_API_VERSION\u0027),","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"ca51f31daa89fb99976e5ed1c5232c4eaebbbc99","unresolved":false,"context_lines":[{"line_number":36,"context_line":"    \"\"\"Hook to add global options.\"\"\""},{"line_number":37,"context_line":"    parser.add_argument("},{"line_number":38,"context_line":"        \u0027--os-dns-api-version\u0027,"},{"line_number":39,"context_line":"        metavar\u003d\u0027\u003cdns-api-version\u003e\u0027,"},{"line_number":40,"context_line":"        help\u003d\u0027DNS API version, default\u003d\u0027 +"},{"line_number":41,"context_line":"             DEFAULT_API_VERSION +"},{"line_number":42,"context_line":"             \u0027 (Env: OS_DNS_API_VERSION)\u0027)"}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_dbcff527","line":39,"in_reply_to":"da20952f_562e65a6","updated":"2015-09-01 20:05:53.000000000","message":"Done","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"03e686c8040e0c38c5f994b53d059a7dc1f1a2ac","unresolved":false,"context_lines":[{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":"from openstackclient.common import utils as oscutils"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from designate import shell"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"DEFAULT_API_VERSION \u003d \u00272\u0027"}],"source_content_type":"text/x-python","patch_set":25,"id":"da20952f_85e1f300","line":18,"updated":"2015-09-02 13:35:17.000000000","message":"shouldnt this be designateclient?","commit_id":"10fdb26def00338b8eb0b720966ce2807a56cb2f"}],"designateclient/v2/cli/recordsets.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        parser \u003d super(CreateRecordSetCommand, self).get_parser(prog_name)"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"        parser.add_argument(\u0027zone_id\u0027, help\u003d\"RecordSet ID\")"},{"line_number":73,"context_line":"        parser.add_argument(\u0027--name\u0027, help\u003d\"RecordSet Name\", required\u003dTrue)"},{"line_number":74,"context_line":"        parser.add_argument(\u0027--records\u0027, help\u003d\"RecordSet Records\","},{"line_number":75,"context_line":"                            nargs\u003d\u0027+\u0027, required\u003dTrue)"},{"line_number":76,"context_line":"        parser.add_argument(\u0027--type\u0027, help\u003d\"RecordSet Type\", required\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_44e072fa","line":73,"updated":"2015-08-19 16:55:14.000000000","message":"Maybe position on the name","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"}],"designateclient/v2/cli/reverse.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    columns \u003d [\u0027id\u0027, \u0027ptrdname\u0027, \u0027description\u0027, \u0027ttl\u0027]"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def get_parser(self, prog_name):"},{"line_number":35,"context_line":"        parser \u003d super(ListFloatingIPCommand, self).get_parser(prog_name)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        return parser"},{"line_number":38,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_24a3fe01","line":35,"updated":"2015-08-19 16:55:14.000000000","message":"maybe just don\u0027t bother defining this here.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"class SetFloatingIPCommand(show.ShowOne):"},{"line_number":64,"context_line":"    \"\"\"Set floatingip ptr record\"\"\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"    def get_parser(self, prog_name):"},{"line_number":67,"context_line":"        parser \u003d super(SetFloatingIPCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_4444f219","line":64,"updated":"2015-08-19 16:55:14.000000000","message":"Isn\u0027t this really a create?","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        ttl_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":77,"context_line":"        ttl_group.add_argument(\u0027--ttl\u0027, type\u003dint, help\u003d\"TTL\")"},{"line_number":78,"context_line":"        ttl_group.add_argument(\u0027--no-ttl\u0027, action\u003d\u0027store_true\u0027)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        return parser"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_04dd9a6e","line":78,"updated":"2015-08-19 16:55:14.000000000","message":"I don\u0027t get thise --no- options, why not just default\u003dNone on --ttl?  The default value probably is None anyway.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"76700b8dfbe747b7c2a1515df4b8492a7d9ed073","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        ttl_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":77,"context_line":"        ttl_group.add_argument(\u0027--ttl\u0027, type\u003dint, help\u003d\"TTL\")"},{"line_number":78,"context_line":"        ttl_group.add_argument(\u0027--no-ttl\u0027, action\u003d\u0027store_true\u0027)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        return parser"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"da20952f_2ca03ef4","line":78,"in_reply_to":"fa1b9901_04dd9a6e","updated":"2015-08-26 20:31:16.000000000","message":"this is a set - the resource still exits, just will no data until it is set","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":109,"context_line":"    def get_parser(self, prog_name):"},{"line_number":110,"context_line":"        parser \u003d super(UnsetFloatingIPCommand, self).get_parser(prog_name)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        parser.add_argument(\u0027floatingip_id\u0027, help\u003d\"Floating IP ID\")"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        return parser"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_04c4ba8e","line":112,"updated":"2015-08-19 16:55:14.000000000","message":"Unset is normally associated with a resource metadata.  This looks more like a delete to me.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"76700b8dfbe747b7c2a1515df4b8492a7d9ed073","unresolved":false,"context_lines":[{"line_number":109,"context_line":"    def get_parser(self, prog_name):"},{"line_number":110,"context_line":"        parser \u003d super(UnsetFloatingIPCommand, self).get_parser(prog_name)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        parser.add_argument(\u0027floatingip_id\u0027, help\u003d\"Floating IP ID\")"},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        return parser"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"da20952f_8caf0ae3","line":112,"in_reply_to":"fa1b9901_04c4ba8e","updated":"2015-08-26 20:31:16.000000000","message":"this is an unset - the resource still exits, just with no data after the unset","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"}],"designateclient/v2/cli/tlds.py":[{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"1d1ed5fa4c4613f296a6c1cffc4accdc1d64884e","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    def get_parser(self, prog_name):"},{"line_number":47,"context_line":"        parser \u003d super(ShowTLDCommand, self).get_parser(prog_name)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        parser.add_argument(\u0027id\u0027, help\u003d\"TLD ID\")"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        return parser"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_362ad17f","line":49,"updated":"2015-09-01 17:29:03.000000000","message":"this does not seem to be resolving name -\u003e id","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"ca51f31daa89fb99976e5ed1c5232c4eaebbbc99","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    def get_parser(self, prog_name):"},{"line_number":47,"context_line":"        parser \u003d super(ShowTLDCommand, self).get_parser(prog_name)"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"        parser.add_argument(\u0027id\u0027, help\u003d\"TLD ID\")"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        return parser"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_305f3077","line":49,"in_reply_to":"da20952f_362ad17f","updated":"2015-09-01 20:05:53.000000000","message":"This was a thing in the bindings, uploaded seperate patch for it. https://review.openstack.org/219425","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"}],"designateclient/v2/cli/zones.py":[{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class ListZonesCommand(lister.Lister):"},{"line_number":30,"context_line":"    \"\"\"List Zones\"\"\""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    columns \u003d [\u0027id\u0027, \u0027name\u0027, \u0027type\u0027, \u0027serial\u0027, \u0027status\u0027, \u0027action\u0027]"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"7a016987_e634254b","line":30,"updated":"2015-05-27 19:27:50.000000000","message":"In the other client descriptions, the resources don\u0027t seem to be capitalized, I think it\u0027d be good to conform to that convention.","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"class ListZonesCommand(lister.Lister):"},{"line_number":30,"context_line":"    \"\"\"List Zones\"\"\""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    columns \u003d [\u0027id\u0027, \u0027name\u0027, \u0027type\u0027, \u0027serial\u0027, \u0027status\u0027, \u0027action\u0027]"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"3afb71cf_f29bc6c7","line":30,"in_reply_to":"7a016987_e634254b","updated":"2015-06-08 10:46:06.000000000","message":"Done","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"class ShowZoneCommand(show.ShowOne):"},{"line_number":55,"context_line":"    \"\"\"Show Zone\"\"\""},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def get_parser(self, prog_name):"},{"line_number":58,"context_line":"        parser \u003d super(ShowZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":16,"id":"7a016987_e60b057c","line":55,"updated":"2015-05-27 19:27:50.000000000","message":"The \u0027show\u0027 command descriptions seem to normally say \u0027Show zone details\u0027 or \u0027Show a specific zone\u0027. I think it\u0027d be good to be more descriptive with all of the \u0027shows\u0027.","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"class ShowZoneCommand(show.ShowOne):"},{"line_number":55,"context_line":"    \"\"\"Show Zone\"\"\""},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def get_parser(self, prog_name):"},{"line_number":58,"context_line":"        parser \u003d super(ShowZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":16,"id":"3afb71cf_32639ede","line":55,"in_reply_to":"7a016987_e60b057c","updated":"2015-06-08 10:46:06.000000000","message":"Done","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"class CreateZoneCommand(show.ShowOne):"},{"line_number":73,"context_line":"    \"\"\"Create Domain\"\"\""},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    def get_parser(self, prog_name):"},{"line_number":76,"context_line":"        parser \u003d super(CreateZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":16,"id":"7a016987_c685a9cf","line":73,"updated":"2015-05-27 19:27:50.000000000","message":"For most resources, the help looks like \u0027Create new x\u0027, that should probably be followed, for consistency.","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"class SetZoneCommand(command.Command):"},{"line_number":98,"context_line":"    \"\"\"Set Zone\"\"\""},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def get_parser(self, prog_name):"},{"line_number":101,"context_line":"        parser \u003d super(SetZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":16,"id":"7a016987_4649b9b6","line":98,"updated":"2015-05-27 19:27:50.000000000","message":"All of the OS set helps seem to look like \u0027Set x properties\u0027","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":95,"context_line":""},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"class SetZoneCommand(command.Command):"},{"line_number":98,"context_line":"    \"\"\"Set Zone\"\"\""},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def get_parser(self, prog_name):"},{"line_number":101,"context_line":"        parser \u003d super(SetZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":16,"id":"3afb71cf_d2ec4a2b","line":98,"in_reply_to":"7a016987_4649b9b6","updated":"2015-06-08 10:46:06.000000000","message":"Done","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        LOG.info(\u0027Zone %s was deleted\u0027, parsed_args.id)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ExportZoneCommand(command.Command):"},{"line_number":159,"context_line":"    def get_parser(self, prog_name):"},{"line_number":160,"context_line":"        parser \u003d super(ExportZoneCommand, self).get_parser(prog_name)"},{"line_number":161,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"7a016987_9913965c","line":158,"updated":"2015-05-27 19:27:50.000000000","message":"These next 4 commands need \"help\" docstrings.","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        LOG.info(\u0027Zone %s was deleted\u0027, parsed_args.id)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ExportZoneCommand(command.Command):"},{"line_number":159,"context_line":"    def get_parser(self, prog_name):"},{"line_number":160,"context_line":"        parser \u003d super(ExportZoneCommand, self).get_parser(prog_name)"},{"line_number":161,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"3afb71cf_72c5968c","line":158,"in_reply_to":"7a016987_9913965c","updated":"2015-06-08 10:46:06.000000000","message":"Done","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":75,"context_line":"    def get_parser(self, prog_name):"},{"line_number":76,"context_line":"        parser \u003d super(CreateZoneCommand, self).get_parser(prog_name)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        parser.add_argument(\u0027--name\u0027, help\u003d\"Zone Name\", required\u003dTrue)"},{"line_number":79,"context_line":"        parser.add_argument(\u0027--email\u0027, help\u003d\"Zone Email\")"},{"line_number":80,"context_line":"        parser.add_argument(\u0027--type\u0027, help\u003d\"Zone Type\", default\u003d\u0027PRIMARY\u0027)"},{"line_number":81,"context_line":"        parser.add_argument(\u0027--ttl\u0027, type\u003dint, help\u003d\"Time To Live (Seconds)\")"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_4224edad","line":78,"updated":"2015-08-19 16:55:14.000000000","message":"Could use a positional for name since it is required.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":6662,"name":"James Li","email":"yueli.m@gmail.com","username":"james-li-3"},"change_message_id":"67687e5713d5d352f4c11dceaf86d368fe724a7b","unresolved":false,"context_lines":[{"line_number":76,"context_line":"        parser \u003d super(CreateZoneCommand, self).get_parser(prog_name)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        parser.add_argument(\u0027--name\u0027, help\u003d\"Zone Name\", required\u003dTrue)"},{"line_number":79,"context_line":"        parser.add_argument(\u0027--email\u0027, help\u003d\"Zone Email\")"},{"line_number":80,"context_line":"        parser.add_argument(\u0027--type\u0027, help\u003d\"Zone Type\", default\u003d\u0027PRIMARY\u0027)"},{"line_number":81,"context_line":"        parser.add_argument(\u0027--ttl\u0027, type\u003dint, help\u003d\"Time To Live (Seconds)\")"},{"line_number":82,"context_line":"        parser.add_argument(\u0027--description\u0027, help\u003d\"Description\")"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_3b0be32b","line":79,"updated":"2015-08-20 16:49:06.000000000","message":"I think email is required too, otherwise I got Error Validating \u0027Domain\u0027 object from api.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        data \u003d client.zones.create("},{"line_number":91,"context_line":"            parsed_args.name, parsed_args.type,"},{"line_number":92,"context_line":"            email\u003dparsed_args.email,"},{"line_number":93,"context_line":"            masters\u003dparsed_args.masters)"},{"line_number":94,"context_line":"        return zip(*sorted(six.iteritems(data)))"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_62d1f156","line":93,"updated":"2015-08-19 16:55:14.000000000","message":"ttl and description seem to be ignored.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        parser.add_argument(\u0027--ttl\u0027, type\u003dint, help\u003d\"Time To Live (Seconds)\")"},{"line_number":107,"context_line":"        description_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":108,"context_line":"        description_group.add_argument(\u0027--description\u0027, help\u003d\"Description\")"},{"line_number":109,"context_line":"        description_group.add_argument(\u0027--no-description\u0027, action\u003d\u0027store_true\u0027)"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        parser.add_argument(\u0027--masters\u0027, help\u003d\"Zone Masters\", nargs\u003d\u0027+\u0027)"},{"line_number":112,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_c2ff9da8","line":109,"updated":"2015-08-19 16:55:14.000000000","message":"Seems like if --description was not specified, that would be no-description","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":117,"context_line":""},{"line_number":118,"context_line":"        data \u003d {}"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        # TODO(kiall): API needs updating.. this get is silly"},{"line_number":121,"context_line":"        if parsed_args.name:"},{"line_number":122,"context_line":"            data[\u0027name\u0027] \u003d parsed_args.name"},{"line_number":123,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_c251ddd9","line":120,"updated":"2015-08-19 16:55:14.000000000","message":"Maybe create a function set and create can share for this.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    def get_parser(self, prog_name):"},{"line_number":146,"context_line":"        parser \u003d super(DeleteZoneCommand, self).get_parser(prog_name)"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"        parser.add_argument(\u0027id\u0027, help\u003d\"Zone ID\")"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        return parser"},{"line_number":151,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_22ac599f","line":148,"updated":"2015-08-19 16:55:14.000000000","message":"It is nice to be able to specify name or id for all these cases on the positional","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":170,"context_line":"        print(response.content)"},{"line_number":171,"context_line":""},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"class ImportZoneCommand(command.Command):"},{"line_number":174,"context_line":"    \"\"\"Import a zone\"\"\""},{"line_number":175,"context_line":"    def get_parser(self, prog_name):"},{"line_number":176,"context_line":"        parser \u003d super(ImportZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_f96cd336","line":173,"updated":"2015-08-19 16:55:14.000000000","message":"Maybe combine this with the \u0027zone create\u0027 with --import or --path","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":187,"context_line":"            LOG.info(\"Imported zone successfully\")"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"class AbandonZoneCommand(command.Command):"},{"line_number":191,"context_line":"    \"\"\"Abandon a zone\"\"\""},{"line_number":192,"context_line":"    def get_parser(self, prog_name):"},{"line_number":193,"context_line":"        parser \u003d super(AbandonZoneCommand, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_b91bbbc9","line":190,"updated":"2015-08-19 16:55:14.000000000","message":"maybe these two commands could be supported by the \u0027zone set\u0027 command with --abandon and --axfr","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        data \u003d client.zone_transfers.list_requests()"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"        cols \u003d self.columns"},{"line_number":266,"context_line":"        cols \u003d [\"id\"]"},{"line_number":267,"context_line":"        return cols, (utils.get_item_properties(s, cols) for s in data)"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_d9978f46","line":266,"updated":"2015-08-19 16:55:14.000000000","message":"why?","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":293,"context_line":""},{"line_number":294,"context_line":"        parser.add_argument(\u0027id\u0027, help\u003d\"Zone Transfer ID\")"},{"line_number":295,"context_line":"        description_group \u003d parser.add_mutually_exclusive_group()"},{"line_number":296,"context_line":"        description_group.add_argument(\u0027--description\u0027, help\u003d\"Description\")"},{"line_number":297,"context_line":"        description_group.add_argument(\u0027--no-description\u0027, action\u003d\u0027store_true\u0027)"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":20,"id":"fa1b9901_79bd23b6","line":296,"updated":"2015-08-19 16:55:14.000000000","message":"Does this ^ just need a default\u003dNone?","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"7c63bf00888a192958d1267d26cd501141617ab4","unresolved":false,"context_lines":[{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        client.zone_transfers.delete_request(parsed_args.id)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        LOG.info(\u0027Zone Transfer %s was deleted\u0027 % parsed_args.floatingip_id)"},{"line_number":354,"context_line":""},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"class AcceptTransferRequestCommand(command.Command):"}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_3875e031","line":353,"updated":"2015-09-01 17:05:49.000000000","message":"floatingip_id \u003d\u003e zone_transfer_request.id","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"ca51f31daa89fb99976e5ed1c5232c4eaebbbc99","unresolved":false,"context_lines":[{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        client.zone_transfers.delete_request(parsed_args.id)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        LOG.info(\u0027Zone Transfer %s was deleted\u0027 % parsed_args.floatingip_id)"},{"line_number":354,"context_line":""},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"class AcceptTransferRequestCommand(command.Command):"}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_5b4d25c3","line":353,"in_reply_to":"da20952f_36997120","updated":"2015-09-01 20:05:53.000000000","message":"Done","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"569319e2fb8948d3ad133069e75a19475790393f","unresolved":false,"context_lines":[{"line_number":350,"context_line":""},{"line_number":351,"context_line":"        client.zone_transfers.delete_request(parsed_args.id)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        LOG.info(\u0027Zone Transfer %s was deleted\u0027 % parsed_args.floatingip_id)"},{"line_number":354,"context_line":""},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"class AcceptTransferRequestCommand(command.Command):"}],"source_content_type":"text/x-python","patch_set":23,"id":"da20952f_36997120","line":353,"in_reply_to":"da20952f_3875e031","updated":"2015-09-01 17:27:39.000000000","message":"Good catch, in this case though parsed_args.id","commit_id":"c6fb2b6814bba1acfd20336eb136871727369837"}],"designateclient/v2/reverse.py":[{"author":{"_account_id":4894,"name":"stanzgy","email":"stanzgy@gmail.com","username":"stanzgy"},"change_message_id":"dfbd62d5062c6b5503e089941e285ba9d96bf9a8","unresolved":false,"context_lines":[{"line_number":14,"context_line":"# License for the specific language governing permissions and limitations"},{"line_number":15,"context_line":"# under the License."},{"line_number":16,"context_line":"from designateclient import client"},{"line_number":17,"context_line":"import json"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class FloatingIPController(client.Controller):"}],"source_content_type":"text/x-python","patch_set":14,"id":"1aca2d91_94723eac","line":17,"updated":"2015-05-08 03:27:01.000000000","message":"pep8 failed: H305  imports not grouped correctly","commit_id":"0f84b8c8068bf6582107bb7e869fcd736f813075"}],"doc/source/shell-v2.rst":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"::"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    $ source ~/openrc"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"We can now try to create a primary zone"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-rst","patch_set":20,"id":"fa1b9901_a4d94e24","line":19,"updated":"2015-08-19 16:55:14.000000000","message":"As a plugin, you\u0027d get os-client-config support too with ~/.config/openstack/clouds.yaml","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"}],"setup.cfg":[{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"openstack.dns.v2 \u003d"},{"line_number":90,"context_line":"    blacklist_create \u003d designateclient.v2.cli.blacklists:CreateBlacklistCommand"},{"line_number":91,"context_line":"    blacklist_list \u003d designateclient.v2.cli.blacklists:ListBlacklistsCommand"},{"line_number":92,"context_line":"    blacklist_show \u003d designateclient.v2.cli.blacklists:ShowBlacklistCommand"},{"line_number":93,"context_line":"    blacklist_set \u003d designateclient.v2.cli.blacklists:SetBlacklistCommand"},{"line_number":94,"context_line":"    blacklist_delete \u003d designateclient.v2.cli.blacklists:DeleteBlacklistCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":16,"id":"7a016987_26311d0c","line":91,"updated":"2015-05-27 19:27:50.000000000","message":"I wonder about calling this \u0027blacklist\u0027 too. If any other service implements a blacklist, that could be weird. \n\nMaybe a \"zone blacklist\"? Maybe it\u0027s not a big deal. I understand that it kind of sucks to make the CLI different from the API resource.","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"openstack.dns.v2 \u003d"},{"line_number":90,"context_line":"    blacklist_create \u003d designateclient.v2.cli.blacklists:CreateBlacklistCommand"},{"line_number":91,"context_line":"    blacklist_list \u003d designateclient.v2.cli.blacklists:ListBlacklistsCommand"},{"line_number":92,"context_line":"    blacklist_show \u003d designateclient.v2.cli.blacklists:ShowBlacklistCommand"},{"line_number":93,"context_line":"    blacklist_set \u003d designateclient.v2.cli.blacklists:SetBlacklistCommand"},{"line_number":94,"context_line":"    blacklist_delete \u003d designateclient.v2.cli.blacklists:DeleteBlacklistCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":16,"id":"3afb71cf_c7992e70","line":91,"in_reply_to":"7a016987_26311d0c","updated":"2015-06-08 10:46:06.000000000","message":"Done","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":106,"context_line":"    zone_delete \u003d designateclient.v2.cli.zones:DeleteZoneCommand"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    zone_export \u003d designateclient.v2.cli.zones:ExportZoneCommand"},{"line_number":109,"context_line":"    zone_import \u003d designateclient.v2.cli.zones:ImportZoneCommand"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    zone_abandon \u003d designateclient.v2.cli.zones:AbandonZoneCommand"},{"line_number":112,"context_line":"    zone_axfr \u003d designateclient.v2.cli.zones:AXFRZoneCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":16,"id":"7a016987_a3b05f8c","line":109,"updated":"2015-05-27 19:27:50.000000000","message":"Import and export don\u0027t seem to be implemented in the bindings.","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":106,"context_line":"    zone_delete \u003d designateclient.v2.cli.zones:DeleteZoneCommand"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    zone_export \u003d designateclient.v2.cli.zones:ExportZoneCommand"},{"line_number":109,"context_line":"    zone_import \u003d designateclient.v2.cli.zones:ImportZoneCommand"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    zone_abandon \u003d designateclient.v2.cli.zones:AbandonZoneCommand"},{"line_number":112,"context_line":"    zone_axfr \u003d designateclient.v2.cli.zones:AXFRZoneCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":16,"id":"3afb71cf_87f9f682","line":109,"in_reply_to":"7a016987_a3b05f8c","updated":"2015-06-08 10:46:06.000000000","message":"Done","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8174,"name":"Tim Simmons","email":"tim.r.simmons@gmail.com","username":"TimSimmons"},"change_message_id":"7244995b8c795445611f52cf661b07c4df664588","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    recordset_set \u003d designateclient.v2.cli.recordsets:SetRecordSetCommand"},{"line_number":126,"context_line":"    recordset_delete \u003d designateclient.v2.cli.recordsets:DeleteRecordSetCommand"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    floatingip_list \u003d designateclient.v2.cli.reverse:ListFloatingIPCommand"},{"line_number":129,"context_line":"    floatingip_show \u003d designateclient.v2.cli.reverse:ShowFloatingIPCommand"},{"line_number":130,"context_line":"    floatingip_set \u003d designateclient.v2.cli.reverse:SetFloatingIPCommand"},{"line_number":131,"context_line":"    floatingip_unset \u003d designateclient.v2.cli.reverse:UnsetFloatingIPCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":16,"id":"7a016987_c684890c","line":128,"updated":"2015-05-27 19:27:50.000000000","message":"naming these floatingip_x makes the openstack client pretty confusing. \n\n # Delete a floating ip from Neutron\n openstack ip floating delete\n # Delete a PTR record from Designate\n openstack floatingip delete \n\n\nMaybe ptr record, or ptr, or rdns? Thoughts?","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":395,"name":"Endre Karlson","email":"endre.karlson@gmail.com","username":"endre-karlson"},"change_message_id":"8193f8d4f00c71d582cd8ae9b49875faad77e77a","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    recordset_set \u003d designateclient.v2.cli.recordsets:SetRecordSetCommand"},{"line_number":126,"context_line":"    recordset_delete \u003d designateclient.v2.cli.recordsets:DeleteRecordSetCommand"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    floatingip_list \u003d designateclient.v2.cli.reverse:ListFloatingIPCommand"},{"line_number":129,"context_line":"    floatingip_show \u003d designateclient.v2.cli.reverse:ShowFloatingIPCommand"},{"line_number":130,"context_line":"    floatingip_set \u003d designateclient.v2.cli.reverse:SetFloatingIPCommand"},{"line_number":131,"context_line":"    floatingip_unset \u003d designateclient.v2.cli.reverse:UnsetFloatingIPCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":16,"id":"3afb71cf_47dc1e12","line":128,"in_reply_to":"7a016987_c684890c","updated":"2015-06-08 10:46:06.000000000","message":"I went with ptr record for now :)","commit_id":"a6c7102ba8188da26ceb03896fd4f3a3fd694246"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"76700b8dfbe747b7c2a1515df4b8492a7d9ed073","unresolved":false,"context_lines":[{"line_number":108,"context_line":"    zone_abandon \u003d designateclient.v2.cli.zones:AbandonZoneCommand"},{"line_number":109,"context_line":"    zone_axfr \u003d designateclient.v2.cli.zones:AXFRZoneCommand"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    zone_transfer_create \u003d designateclient.v2.cli.zones:CreateTransferCommand"},{"line_number":112,"context_line":"    zone_transfer_list \u003d designateclient.v2.cli.zones:ListTransferCommand"},{"line_number":113,"context_line":"    zone_transfer_show \u003d designateclient.v2.cli.zones:ShowTransferCommand"},{"line_number":114,"context_line":"    zone_transfer_set \u003d designateclient.v2.cli.zones:SetTransferCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_2129709e","line":111,"updated":"2015-08-26 20:31:16.000000000","message":"these are zone transfer requests not zone transfers","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":113,"context_line":"    zone_transfer_show \u003d designateclient.v2.cli.zones:ShowTransferCommand"},{"line_number":114,"context_line":"    zone_transfer_set \u003d designateclient.v2.cli.zones:SetTransferCommand"},{"line_number":115,"context_line":"    zone_transfer_delete \u003d designateclient.v2.cli.zones:DeleteTransferCommand"},{"line_number":116,"context_line":"    zone_transfer_accept \u003d designateclient.v2.cli.zones:AcceptCommand"},{"line_number":117,"context_line":"    zone_transfer_accept-show \u003d designateclient.v2.cli.zones:ShowAcceptCommand"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    recordset_create \u003d designateclient.v2.cli.recordsets:CreateRecordSetCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_c449e2b8","line":116,"updated":"2015-08-19 16:55:14.000000000","message":"Maybe the accept could be combined with \u0027transfer set\u0027 with an --accept","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"76700b8dfbe747b7c2a1515df4b8492a7d9ed073","unresolved":false,"context_lines":[{"line_number":113,"context_line":"    zone_transfer_show \u003d designateclient.v2.cli.zones:ShowTransferCommand"},{"line_number":114,"context_line":"    zone_transfer_set \u003d designateclient.v2.cli.zones:SetTransferCommand"},{"line_number":115,"context_line":"    zone_transfer_delete \u003d designateclient.v2.cli.zones:DeleteTransferCommand"},{"line_number":116,"context_line":"    zone_transfer_accept \u003d designateclient.v2.cli.zones:AcceptCommand"},{"line_number":117,"context_line":"    zone_transfer_accept-show \u003d designateclient.v2.cli.zones:ShowAcceptCommand"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    recordset_create \u003d designateclient.v2.cli.recordsets:CreateRecordSetCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_a11b4080","line":116,"in_reply_to":"fa1b9901_c449e2b8","updated":"2015-08-26 20:31:16.000000000","message":"this is a totally separate object - we just need to append _request to the command above.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    zone_transfer_set \u003d designateclient.v2.cli.zones:SetTransferCommand"},{"line_number":115,"context_line":"    zone_transfer_delete \u003d designateclient.v2.cli.zones:DeleteTransferCommand"},{"line_number":116,"context_line":"    zone_transfer_accept \u003d designateclient.v2.cli.zones:AcceptCommand"},{"line_number":117,"context_line":"    zone_transfer_accept-show \u003d designateclient.v2.cli.zones:ShowAcceptCommand"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    recordset_create \u003d designateclient.v2.cli.recordsets:CreateRecordSetCommand"},{"line_number":120,"context_line":"    recordset_list \u003d designateclient.v2.cli.recordsets:ListRecordSetsCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_a41e2eb9","line":117,"updated":"2015-08-19 16:55:14.000000000","message":"Maybe this accept show command could be combined with trans list.  perhaps with a ---accept option on list.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"76700b8dfbe747b7c2a1515df4b8492a7d9ed073","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    zone_transfer_set \u003d designateclient.v2.cli.zones:SetTransferCommand"},{"line_number":115,"context_line":"    zone_transfer_delete \u003d designateclient.v2.cli.zones:DeleteTransferCommand"},{"line_number":116,"context_line":"    zone_transfer_accept \u003d designateclient.v2.cli.zones:AcceptCommand"},{"line_number":117,"context_line":"    zone_transfer_accept-show \u003d designateclient.v2.cli.zones:ShowAcceptCommand"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"    recordset_create \u003d designateclient.v2.cli.recordsets:CreateRecordSetCommand"},{"line_number":120,"context_line":"    recordset_list \u003d designateclient.v2.cli.recordsets:ListRecordSetsCommand"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_6111a85c","line":117,"in_reply_to":"fa1b9901_a41e2eb9","updated":"2015-08-26 20:31:16.000000000","message":"also need to see a zone_transfer_accept_list\n\nshould this be _show instead of -show ?","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"0350aae28edda9cada53d898661a829b0fddc848","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    ptr_record_list \u003d designateclient.v2.cli.reverse:ListFloatingIPCommand"},{"line_number":126,"context_line":"    ptr_record_show \u003d designateclient.v2.cli.reverse:ShowFloatingIPCommand"},{"line_number":127,"context_line":"    ptr_record_set \u003d designateclient.v2.cli.reverse:SetFloatingIPCommand"},{"line_number":128,"context_line":"    ptr_record_unset \u003d designateclient.v2.cli.reverse:UnsetFloatingIPCommand"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"[build_sphinx]"},{"line_number":131,"context_line":"all_files \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_849c4a10","line":128,"updated":"2015-08-19 16:55:14.000000000","message":"I think these are the ones that sounded kind of like they could be create/delete instead of set/unset.","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"},{"author":{"_account_id":8099,"name":"Graham Hayes","email":"gr@ham.ie","username":"graham"},"change_message_id":"76700b8dfbe747b7c2a1515df4b8492a7d9ed073","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    ptr_record_list \u003d designateclient.v2.cli.reverse:ListFloatingIPCommand"},{"line_number":126,"context_line":"    ptr_record_show \u003d designateclient.v2.cli.reverse:ShowFloatingIPCommand"},{"line_number":127,"context_line":"    ptr_record_set \u003d designateclient.v2.cli.reverse:SetFloatingIPCommand"},{"line_number":128,"context_line":"    ptr_record_unset \u003d designateclient.v2.cli.reverse:UnsetFloatingIPCommand"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"[build_sphinx]"},{"line_number":131,"context_line":"all_files \u003d 1"}],"source_content_type":"text/x-ttcn-cfg","patch_set":20,"id":"fa1b9901_ac29f78b","line":128,"in_reply_to":"fa1b9901_849c4a10","updated":"2015-08-26 20:31:16.000000000","message":"They are records that are not owned by the user, so they can just set or unset them, not create / delete them","commit_id":"8aa90f8c28eabe84d7b4e5a991a54fbffe610783"}]}