)]}'
{"openstackclient/identity/v3/region.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2012-2013 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_ac000d02","line":1,"updated":"2014-11-06 18:58:36.000000000","message":"Remove this line; only Foundation employees can add a Foundation copyright to new code.  Do the same for all new files.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def get_parser(self, prog_name):"},{"line_number":34,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":35,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":36,"context_line":"        parser.add_argument("},{"line_number":37,"context_line":"            \u0027--id\u0027,"},{"line_number":38,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_81595a5c","line":35,"updated":"2014-11-06 18:58:36.000000000","message":"What does it mean to create a region with no arguments?  Is it useful?  Just because the REST API doesn\u0027t require it doesn\u0027t mean it is a correct action.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"a680185c06900728a829a16a949fb881cf853e63","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    def get_parser(self, prog_name):"},{"line_number":34,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":35,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":36,"context_line":"        parser.add_argument("},{"line_number":37,"context_line":"            \u0027--id\u0027,"},{"line_number":38,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_dfffba65","line":35,"in_reply_to":"7a8c0949_81595a5c","updated":"2014-11-07 08:44:58.000000000","message":"If you create region with no arguments,  you can get a region that contains an autogenerated ID.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":35,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":36,"context_line":"        parser.add_argument("},{"line_number":37,"context_line":"            \u0027--id\u0027,"},{"line_number":38,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"},{"line_number":39,"context_line":"            help\u003d\u0027New region id\u0027,"},{"line_number":40,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_c132820f","line":37,"updated":"2014-11-06 18:58:36.000000000","message":"Is ID a UUID or a string or a numeric id?  My gut feeling is we want this to be a required argument, but if not, it shouldn\u0027t be named id.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"a680185c06900728a829a16a949fb881cf853e63","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":35,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":36,"context_line":"        parser.add_argument("},{"line_number":37,"context_line":"            \u0027--id\u0027,"},{"line_number":38,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"},{"line_number":39,"context_line":"            help\u003d\u0027New region id\u0027,"},{"line_number":40,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_ff02b66d","line":37,"in_reply_to":"7a8c0949_c132820f","updated":"2014-11-07 08:44:58.000000000","message":"The ID can either a string or a UUID. That is say, region supports user-specified ID.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":44,"context_line":"            help\u003d\u0027New region url\u0027,"},{"line_number":45,"context_line":"        )"},{"line_number":46,"context_line":"        parser.add_argument("},{"line_number":47,"context_line":"            \u0027--parent_region_id\u0027,"},{"line_number":48,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"},{"line_number":49,"context_line":"            help\u003d\u0027The parent region of new region\u0027,"},{"line_number":50,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_2140a655","line":47,"updated":"2014-11-06 18:58:36.000000000","message":"This should match the conclusions reached for --id.\n\nAlso, use \u0027-\u0027; not \u0027_\u0027 inside the option name.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":101,"context_line":"    def get_parser(self, prog_name):"},{"line_number":102,"context_line":"        parser \u003d super(ListRegion, self).get_parser(prog_name)"},{"line_number":103,"context_line":"        parser.add_argument("},{"line_number":104,"context_line":"            \u0027--parent_region_id\u0027,"},{"line_number":105,"context_line":"            metavar\u003d\u0027\u003cparent-region-id\u003e\u0027,"},{"line_number":106,"context_line":"            help\u003d\u0027The parent region of new region\u0027,"},{"line_number":107,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_2117c652","line":104,"updated":"2014-11-06 18:58:36.000000000","message":"ditto from the create option","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":103,"context_line":"        parser.add_argument("},{"line_number":104,"context_line":"            \u0027--parent_region_id\u0027,"},{"line_number":105,"context_line":"            metavar\u003d\u0027\u003cparent-region-id\u003e\u0027,"},{"line_number":106,"context_line":"            help\u003d\u0027The parent region of new region\u0027,"},{"line_number":107,"context_line":"        )"},{"line_number":108,"context_line":"        return parser"},{"line_number":109,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_6108ee2d","line":106,"updated":"2014-11-06 18:58:36.000000000","message":"Update the help for a list action","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":137,"context_line":"            metavar\u003d\u0027\u003cregion\u003e\u0027,"},{"line_number":138,"context_line":"            help\u003d\u0027ID of region to change\u0027,"},{"line_number":139,"context_line":"        )"},{"line_number":140,"context_line":"        parser.add_argument("},{"line_number":141,"context_line":"            \u0027--parent_region_id\u0027,"},{"line_number":142,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"},{"line_number":143,"context_line":"            help\u003d\u0027New parent region of the region\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_c119a289","line":140,"updated":"2014-11-06 18:58:36.000000000","message":"I\u0027d like to see the create and set options in the same order.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":138,"context_line":"            help\u003d\u0027ID of region to change\u0027,"},{"line_number":139,"context_line":"        )"},{"line_number":140,"context_line":"        parser.add_argument("},{"line_number":141,"context_line":"            \u0027--parent_region_id\u0027,"},{"line_number":142,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"},{"line_number":143,"context_line":"            help\u003d\u0027New parent region of the region\u0027,"},{"line_number":144,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_81307a02","line":141,"updated":"2014-11-06 18:58:36.000000000","message":"ditto from the create option","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"8dbb17bf5223d0471b58c86e182cf6189b034bdf","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def get_parser(self, prog_name):"},{"line_number":32,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":33,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":34,"context_line":"        parser.add_argument("},{"line_number":35,"context_line":"            \u0027--id\u0027,"},{"line_number":36,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_8b87ced6","line":33,"updated":"2014-11-12 20:50:37.000000000","message":"that is weird, but true!","commit_id":"c92567c2f0d7ebd254ffdd51d5016f07344724b7"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"c5dbd9586e871b05cce2d1ed15e01ff6397a36e2","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def get_parser(self, prog_name):"},{"line_number":32,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":33,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":34,"context_line":"        parser.add_argument("},{"line_number":35,"context_line":"            \u0027--id\u0027,"},{"line_number":36,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_16d9fe21","line":33,"in_reply_to":"5a890539_727fceab","updated":"2014-11-14 17:00:19.000000000","message":"One way or the other this makes for an inconsistent command.  Every other region command in this review has the region ID as the first/only positional argument.  Make it optional to auto-generate an ID.\n\nI still don\u0027t see what the use-case is for creating a region with an auto-gen ID, no description, no parent and no URL, but whatever, it isn\u0027t my API.  When the underlying API does odd things, we should have a strong reason to expose that to the user, and I don\u0027t see it here.\n\n\u0027create flavor\u0027 is similar, except that it has an actual name property, as well as an --id option that I think also is poorly designed.  The Region API appears to use ID in the same capacity as flavor\u0027s name with the addition of the auto-generated value.","commit_id":"c92567c2f0d7ebd254ffdd51d5016f07344724b7"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"7184e9c62ed6b774f71eea78926285d2152823fb","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def get_parser(self, prog_name):"},{"line_number":32,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":33,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":34,"context_line":"        parser.add_argument("},{"line_number":35,"context_line":"            \u0027--id\u0027,"},{"line_number":36,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_b78a0819","line":33,"in_reply_to":"5a890539_8b87ced6","updated":"2014-11-13 21:10:35.000000000","message":"Steve, see my comments in patchset 1 about this.  Just because the API lets you doesn\u0027t automatically mean it is a good idea to let the user do this.","commit_id":"c92567c2f0d7ebd254ffdd51d5016f07344724b7"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f4b11c9374f7552e20497b549b35e75a95de0240","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def get_parser(self, prog_name):"},{"line_number":32,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":33,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":34,"context_line":"        parser.add_argument("},{"line_number":35,"context_line":"            \u0027--id\u0027,"},{"line_number":36,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_f2973e38","line":33,"in_reply_to":"5a890539_b78a0819","updated":"2014-11-13 21:44:53.000000000","message":"Steve talked me in to this, I just have one change and that is for the ID to be an optional positional argument to match every other create command we have:\n\n  openstack region create fred","commit_id":"c92567c2f0d7ebd254ffdd51d5016f07344724b7"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"e51824ba9d1ed0b549fd3b2d07597600834e87d9","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    def get_parser(self, prog_name):"},{"line_number":32,"context_line":"        parser \u003d super(CreateRegion, self).get_parser(prog_name)"},{"line_number":33,"context_line":"        # NOTE(wanghong): No parameters are required for creating regions"},{"line_number":34,"context_line":"        parser.add_argument("},{"line_number":35,"context_line":"            \u0027--id\u0027,"},{"line_number":36,"context_line":"            metavar\u003d\u0027\u003cregion-id\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_727fceab","line":33,"in_reply_to":"5a890539_f2973e38","updated":"2014-11-13 21:55:49.000000000","message":"It\u0027s worth noting a few things.\n\nregion IDs can be both auto-generated, or user-defined.\nboth are supported by keystone server APIs.\n\n  PUT /regions/{region_id} is for a user-defined region id (like fred).\n  POST /regions is for an auto generated one.\n\nBy doing your suggestion, we are limiting the user to the user-defined ones. Which arguably, isn\u0027t a bad thing, but would be hard to undo.","commit_id":"c92567c2f0d7ebd254ffdd51d5016f07344724b7"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"8dbb17bf5223d0471b58c86e182cf6189b034bdf","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        parser.add_argument("},{"line_number":45,"context_line":"            \u0027--description\u0027,"},{"line_number":46,"context_line":"            metavar\u003d\u0027\u003cregion-description\u003e\u0027,"},{"line_number":47,"context_line":"            help\u003d\u0027New region description\u0027,"},{"line_number":48,"context_line":"        )"},{"line_number":49,"context_line":"        parser.add_argument("},{"line_number":50,"context_line":"            \u0027--url\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_4b640649","line":47,"updated":"2014-11-12 20:50:37.000000000","message":"can you mark the help strings for translation using _()","commit_id":"c92567c2f0d7ebd254ffdd51d5016f07344724b7"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1bde7e71869407ec244ceafd41bbd130391b63d0","unresolved":false,"context_lines":[{"line_number":39,"context_line":"            help\u003d_(\u0027New region id\u0027),"},{"line_number":40,"context_line":"        )"},{"line_number":41,"context_line":"        parser.add_argument("},{"line_number":42,"context_line":"            \u0027--parent-region-id\u0027,"},{"line_number":43,"context_line":"            metavar\u003d\u0027\u003cparent-region-id\u003e\u0027,"},{"line_number":44,"context_line":"            help\u003d_(\u0027The parent region of new region\u0027),"},{"line_number":45,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_62507b0b","line":42,"updated":"2014-11-18 04:48:20.000000000","message":"Remove -id\n\nI\u0027d like to remove all occurrences of ID here, but will settle for just getting it out of the option name.\n\nAlso in list and set","commit_id":"58485c0bb5484d12cde017210a4e9bd01a4c394e"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1d3b0fa39c873c147953cb2d23fbd956a3dfd4f4","unresolved":false,"context_lines":[{"line_number":39,"context_line":"            help\u003d_(\u0027New region id\u0027),"},{"line_number":40,"context_line":"        )"},{"line_number":41,"context_line":"        parser.add_argument("},{"line_number":42,"context_line":"            \u0027--parent-region-id\u0027,"},{"line_number":43,"context_line":"            metavar\u003d\u0027\u003cparent-region-id\u003e\u0027,"},{"line_number":44,"context_line":"            help\u003d_(\u0027The parent region of new region\u0027),"},{"line_number":45,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_45b8e910","line":42,"in_reply_to":"5a890539_62507b0b","updated":"2014-11-18 05:49:52.000000000","message":"i think i caught them all","commit_id":"58485c0bb5484d12cde017210a4e9bd01a4c394e"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1bde7e71869407ec244ceafd41bbd130391b63d0","unresolved":false,"context_lines":[{"line_number":178,"context_line":"class ShowRegion(show.ShowOne):"},{"line_number":179,"context_line":"    \"\"\"Show region command\"\"\""},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    log \u003d logging.getLogger(__name__ + \u0027.ShowRegion\u0027)"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"    def get_parser(self, prog_name):"},{"line_number":184,"context_line":"        parser \u003d super(ShowRegion, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_228f5376","line":181,"updated":"2014-11-18 04:48:20.000000000","message":"s/command//","commit_id":"58485c0bb5484d12cde017210a4e9bd01a4c394e"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1d3b0fa39c873c147953cb2d23fbd956a3dfd4f4","unresolved":false,"context_lines":[{"line_number":178,"context_line":"class ShowRegion(show.ShowOne):"},{"line_number":179,"context_line":"    \"\"\"Show region command\"\"\""},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    log \u003d logging.getLogger(__name__ + \u0027.ShowRegion\u0027)"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"    def get_parser(self, prog_name):"},{"line_number":184,"context_line":"        parser \u003d super(ShowRegion, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_853d7191","line":181,"in_reply_to":"5a890539_228f5376","updated":"2014-11-18 05:49:52.000000000","message":"Done","commit_id":"58485c0bb5484d12cde017210a4e9bd01a4c394e"}],"openstackclient/tests/identity/v3/test_region.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0b720d95b9eb6e2819dd9666651cf10a6f3e6111","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        )"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        collist \u003d (\u0027description\u0027, \u0027id\u0027, \u0027parent_region_id\u0027, \u0027url\u0027)"},{"line_number":67,"context_line":"        self.assertEqual(columns, collist)"},{"line_number":68,"context_line":"        datalist \u003d ("},{"line_number":69,"context_line":"            identity_fakes.region_description,"},{"line_number":70,"context_line":"            identity_fakes.region_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_cc2ab97d","line":67,"updated":"2014-11-06 18:58:36.000000000","message":"Could you please swap these values in all of the assertEqual() calls so the expected value is first?  We still have this backward in a lot of places but I\u0027d like to not create new ones.","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"a680185c06900728a829a16a949fb881cf853e63","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        )"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        collist \u003d (\u0027description\u0027, \u0027id\u0027, \u0027parent_region_id\u0027, \u0027url\u0027)"},{"line_number":67,"context_line":"        self.assertEqual(columns, collist)"},{"line_number":68,"context_line":"        datalist \u003d ("},{"line_number":69,"context_line":"            identity_fakes.region_description,"},{"line_number":70,"context_line":"            identity_fakes.region_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a8c0949_bf8cae36","line":67,"in_reply_to":"7a8c0949_cc2ab97d","updated":"2014-11-07 08:44:58.000000000","message":"Done","commit_id":"e75709010fa53d9b73be0952930795ad9ba075bb"}],"setup.cfg":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"1bde7e71869407ec244ceafd41bbd130391b63d0","unresolved":false,"context_lines":[{"line_number":247,"context_line":"    region_delete \u003d openstackclient.identity.v3.region:DeleteRegion"},{"line_number":248,"context_line":"    region_list \u003d openstackclient.identity.v3.region:ListRegion"},{"line_number":249,"context_line":"    region_show \u003d openstackclient.identity.v3.region:ShowRegion"},{"line_number":250,"context_line":"    region_set \u003d openstackclient.identity.v3.region:SetRegion"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    request_token_authorize \u003d openstackclient.identity.v3.token:AuthorizeRequestToken"},{"line_number":253,"context_line":"    request_token_create \u003d openstackclient.identity.v3.token:CreateRequestToken"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"5a890539_c2f5afed","line":250,"updated":"2014-11-18 04:48:20.000000000","message":"sort set/show","commit_id":"58485c0bb5484d12cde017210a4e9bd01a4c394e"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"1d3b0fa39c873c147953cb2d23fbd956a3dfd4f4","unresolved":false,"context_lines":[{"line_number":247,"context_line":"    region_delete \u003d openstackclient.identity.v3.region:DeleteRegion"},{"line_number":248,"context_line":"    region_list \u003d openstackclient.identity.v3.region:ListRegion"},{"line_number":249,"context_line":"    region_show \u003d openstackclient.identity.v3.region:ShowRegion"},{"line_number":250,"context_line":"    region_set \u003d openstackclient.identity.v3.region:SetRegion"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"    request_token_authorize \u003d openstackclient.identity.v3.token:AuthorizeRequestToken"},{"line_number":253,"context_line":"    request_token_create \u003d openstackclient.identity.v3.token:CreateRequestToken"}],"source_content_type":"text/x-ttcn-cfg","patch_set":3,"id":"5a890539_a540ad07","line":250,"in_reply_to":"5a890539_c2f5afed","updated":"2014-11-18 05:49:52.000000000","message":"Done","commit_id":"58485c0bb5484d12cde017210a4e9bd01a4c394e"}]}