)]}'
{"octaviaclient/api/load_balancer_v2.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":405,"context_line":"            A dict of the created l7rule\u0027s settings"},{"line_number":406,"context_line":"        \"\"\""},{"line_number":407,"context_line":"        url \u003d const.BASE_L7RULE_URL.format(policy_uuid\u003dl7policy_id)"},{"line_number":408,"context_line":"        rule \u003d self.create(url, method\u003d\u0027POST\u0027, **kwargs)"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        return rule"},{"line_number":411,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_8d2c514b","line":408,"range":{"start_line":408,"start_character":32,"end_line":408,"end_character":45},"updated":"2017-06-06 19:26:30.000000000","message":"Post is default and this param is not on the load balancer create.  We should be consistent and either remove this one or add it to load balancer","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":405,"context_line":"            A dict of the created l7rule\u0027s settings"},{"line_number":406,"context_line":"        \"\"\""},{"line_number":407,"context_line":"        url \u003d const.BASE_L7RULE_URL.format(policy_uuid\u003dl7policy_id)"},{"line_number":408,"context_line":"        rule \u003d self.create(url, method\u003d\u0027POST\u0027, **kwargs)"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"        return rule"},{"line_number":411,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_4e0cef93","line":408,"range":{"start_line":408,"start_character":32,"end_line":408,"end_character":45},"in_reply_to":"df140735_8d2c514b","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"}],"octaviaclient/osc/v2/constants.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":147,"context_line":"L7RULE_ROWS \u003d ("},{"line_number":148,"context_line":"    \u0027Created At\u0027,"},{"line_number":149,"context_line":"    \u0027Compare Type\u0027,"},{"line_number":150,"context_line":"    \u0027Provisioning Status\u0027,"},{"line_number":151,"context_line":"    \u0027Invert\u0027,"},{"line_number":152,"context_line":"    \u0027Admin State Up\u0027,"},{"line_number":153,"context_line":"    \u0027Updated At\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_ed1ced14","line":150,"range":{"start_line":150,"start_character":5,"end_line":150,"end_character":24},"updated":"2017-06-06 19:26:30.000000000","message":"Same question here about these being pretty names vs. the API names.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":147,"context_line":"L7RULE_ROWS \u003d ("},{"line_number":148,"context_line":"    \u0027Created At\u0027,"},{"line_number":149,"context_line":"    \u0027Compare Type\u0027,"},{"line_number":150,"context_line":"    \u0027Provisioning Status\u0027,"},{"line_number":151,"context_line":"    \u0027Invert\u0027,"},{"line_number":152,"context_line":"    \u0027Admin State Up\u0027,"},{"line_number":153,"context_line":"    \u0027Updated At\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_ae4a6ba9","line":150,"range":{"start_line":150,"start_character":5,"end_line":150,"end_character":24},"in_reply_to":"df140735_ed1ced14","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":165,"context_line":"    \u0027Compare Type\u0027,"},{"line_number":166,"context_line":"    \u0027Type\u0027,"},{"line_number":167,"context_line":"    \u0027Key\u0027,"},{"line_number":168,"context_line":"    \u0027Value\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_0d93a15e","line":168,"range":{"start_line":168,"start_character":5,"end_line":168,"end_character":11},"updated":"2017-06-06 19:26:30.000000000","message":"invert and admin_state_up are on the old client here.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":165,"context_line":"    \u0027Compare Type\u0027,"},{"line_number":166,"context_line":"    \u0027Type\u0027,"},{"line_number":167,"context_line":"    \u0027Key\u0027,"},{"line_number":168,"context_line":"    \u0027Value\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_8ec5670d","line":168,"range":{"start_line":168,"start_character":5,"end_line":168,"end_character":11},"in_reply_to":"df140735_0d93a15e","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"}],"octaviaclient/osc/v2/l7rule.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":35,"context_line":"            \u0027--compare-type\u0027,"},{"line_number":36,"context_line":"            metavar\u003d\u0027\u003ctype\u003e\u0027,"},{"line_number":37,"context_line":"            required\u003dTrue,"},{"line_number":38,"context_line":"            choices\u003d[\u0027REGEX\u0027, \u0027EQUAL_TO\u0027, \u0027CONTAINS\u0027, \u0027ENDS_WITH\u0027,"},{"line_number":39,"context_line":"                     \u0027STARTS_WITH\u0027],"},{"line_number":40,"context_line":"            help\u003d\"Compare type for the l7rule\""},{"line_number":41,"context_line":"        )"},{"line_number":42,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_8db691dc","line":39,"range":{"start_line":38,"start_character":21,"end_line":39,"end_character":34},"updated":"2017-06-06 19:26:30.000000000","message":"This list is used in multiple places, it should be put into a constant","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":56,"context_line":"            metavar\u003d\u0027\u003ckey\u003e\u0027,"},{"line_number":57,"context_line":"            help\u003d\"Key for the l7rule\""},{"line_number":58,"context_line":"        )"},{"line_number":59,"context_line":"        parser.add_argument("},{"line_number":60,"context_line":"            \u0027--project\u0027,"},{"line_number":61,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":62,"context_line":"            help\u003d\"Project for the l7rule (name or ID)\""},{"line_number":63,"context_line":"        )"},{"line_number":64,"context_line":"        parser.add_argument("},{"line_number":65,"context_line":"            \u0027--type\u0027,"},{"line_number":66,"context_line":"            metavar\u003d\u0027\u003ctype\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_edc62d4a","line":63,"range":{"start_line":59,"start_character":8,"end_line":63,"end_character":9},"updated":"2017-06-06 19:26:30.000000000","message":"Remove, should inherit","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":56,"context_line":"            metavar\u003d\u0027\u003ckey\u003e\u0027,"},{"line_number":57,"context_line":"            help\u003d\"Key for the l7rule\""},{"line_number":58,"context_line":"        )"},{"line_number":59,"context_line":"        parser.add_argument("},{"line_number":60,"context_line":"            \u0027--project\u0027,"},{"line_number":61,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":62,"context_line":"            help\u003d\"Project for the l7rule (name or ID)\""},{"line_number":63,"context_line":"        )"},{"line_number":64,"context_line":"        parser.add_argument("},{"line_number":65,"context_line":"            \u0027--type\u0027,"},{"line_number":66,"context_line":"            metavar\u003d\u0027\u003ctype\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_ee83c3a8","line":63,"range":{"start_line":59,"start_character":8,"end_line":63,"end_character":9},"in_reply_to":"df140735_edc62d4a","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            \u0027--type\u0027,"},{"line_number":66,"context_line":"            metavar\u003d\u0027\u003ctype\u003e\u0027,"},{"line_number":67,"context_line":"            required\u003dTrue,"},{"line_number":68,"context_line":"            choices\u003d[\u0027FILE_TYPE\u0027, \u0027PATH\u0027, \u0027COOKIE\u0027, \u0027HOST_NAME\u0027, \u0027HEADER\u0027],"},{"line_number":69,"context_line":"            help\u003d\"Type for the l7rule\""},{"line_number":70,"context_line":"        )"},{"line_number":71,"context_line":"        admin_group \u003d parser.add_mutually_exclusive_group()"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_4db079de","line":68,"range":{"start_line":68,"start_character":21,"end_line":68,"end_character":73},"updated":"2017-06-06 19:26:30.000000000","message":"This list is used in multiple places, it should be put into a constant","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":65,"context_line":"            \u0027--type\u0027,"},{"line_number":66,"context_line":"            metavar\u003d\u0027\u003ctype\u003e\u0027,"},{"line_number":67,"context_line":"            required\u003dTrue,"},{"line_number":68,"context_line":"            choices\u003d[\u0027FILE_TYPE\u0027, \u0027PATH\u0027, \u0027COOKIE\u0027, \u0027HOST_NAME\u0027, \u0027HEADER\u0027],"},{"line_number":69,"context_line":"            help\u003d\"Type for the l7rule\""},{"line_number":70,"context_line":"        )"},{"line_number":71,"context_line":"        admin_group \u003d parser.add_mutually_exclusive_group()"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_ce90fff0","line":68,"range":{"start_line":68,"start_character":21,"end_line":68,"end_character":73},"in_reply_to":"df140735_4db079de","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        parser \u003d super(DeleteL7Rule, self).get_parser(prog_name)"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027l7policy\u0027,"},{"line_number":112,"context_line":"            metavar\u003d\"\u003cpolicy\u003e\","},{"line_number":113,"context_line":"            help\u003d\"l7policy to delete rule from (name or ID)\""},{"line_number":114,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_2d030576","line":111,"range":{"start_line":111,"start_character":12,"end_line":111,"end_character":23},"updated":"2017-06-06 19:26:30.000000000","message":"We are changing these to mandatory --l7policy options.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":108,"context_line":"        parser \u003d super(DeleteL7Rule, self).get_parser(prog_name)"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"        parser.add_argument("},{"line_number":111,"context_line":"            \u0027l7policy\u0027,"},{"line_number":112,"context_line":"            metavar\u003d\"\u003cpolicy\u003e\","},{"line_number":113,"context_line":"            help\u003d\"l7policy to delete rule from (name or ID)\""},{"line_number":114,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_ce7ebfae","line":111,"range":{"start_line":111,"start_character":12,"end_line":111,"end_character":23},"in_reply_to":"df140735_2d030576","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":128,"context_line":"            l7policy_id\u003dattrs[\u0027l7policy_id\u0027]"},{"line_number":129,"context_line":"        )"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"        return response"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"class ListL7Rule(lister.Lister):"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_48b067de","line":131,"range":{"start_line":131,"start_character":8,"end_line":131,"end_character":23},"updated":"2017-06-06 19:26:30.000000000","message":"This command is returning \"\u003cResponse [204]\u003e\" when it should not return any output.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":128,"context_line":"            l7policy_id\u003dattrs[\u0027l7policy_id\u0027]"},{"line_number":129,"context_line":"        )"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"        return response"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"class ListL7Rule(lister.Lister):"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_ee5aa34e","line":131,"range":{"start_line":131,"start_character":8,"end_line":131,"end_character":23},"in_reply_to":"df140735_48b067de","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":138,"context_line":"        parser \u003d super(ListL7Rule, self).get_parser(prog_name)"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"        parser.add_argument("},{"line_number":141,"context_line":"            \u0027l7policy\u0027,"},{"line_number":142,"context_line":"            metavar\u003d\u0027\u003cl7policy\u003e\u0027,"},{"line_number":143,"context_line":"            help\u003d\u0027l7policy to list rules for (name or ID)\u0027"},{"line_number":144,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_2d38a53b","line":141,"range":{"start_line":141,"start_character":12,"end_line":141,"end_character":23},"updated":"2017-06-06 19:26:30.000000000","message":"We are changing these to mandatory --l7policy options.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":138,"context_line":"        parser \u003d super(ListL7Rule, self).get_parser(prog_name)"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"        parser.add_argument("},{"line_number":141,"context_line":"            \u0027l7policy\u0027,"},{"line_number":142,"context_line":"            metavar\u003d\u0027\u003cl7policy\u003e\u0027,"},{"line_number":143,"context_line":"            help\u003d\u0027l7policy to list rules for (name or ID)\u0027"},{"line_number":144,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_822bfb80","line":141,"range":{"start_line":141,"start_character":12,"end_line":141,"end_character":23},"in_reply_to":"df140735_2d38a53b","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        parser \u003d super(ShowL7Rule, self).get_parser(prog_name)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        parser.add_argument("},{"line_number":168,"context_line":"            \u0027l7policy\u0027,"},{"line_number":169,"context_line":"            metavar\u003d\"\u003cpolicy\u003e\","},{"line_number":170,"context_line":"            help\u003d\"l7policy to show rule from (name or ID)\""},{"line_number":171,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_cdf12998","line":168,"range":{"start_line":168,"start_character":12,"end_line":168,"end_character":23},"updated":"2017-06-06 19:26:30.000000000","message":"We are changing these to mandatory --l7policy options.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        parser \u003d super(ShowL7Rule, self).get_parser(prog_name)"},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"        parser.add_argument("},{"line_number":168,"context_line":"            \u0027l7policy\u0027,"},{"line_number":169,"context_line":"            metavar\u003d\"\u003cpolicy\u003e\","},{"line_number":170,"context_line":"            help\u003d\"l7policy to show rule from (name or ID)\""},{"line_number":171,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_c217b345","line":168,"range":{"start_line":168,"start_character":12,"end_line":168,"end_character":23},"in_reply_to":"df140735_cdf12998","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":197,"context_line":"        parser \u003d super(SetL7Rule, self).get_parser(prog_name)"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"        parser.add_argument("},{"line_number":200,"context_line":"            \u0027l7policy\u0027,"},{"line_number":201,"context_line":"            metavar\u003d\u0027\u003cpolicy\u003e\u0027,"},{"line_number":202,"context_line":"            help\u003d\"L7policy to update rule on (name or ID)\""},{"line_number":203,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_6d17bda5","line":200,"range":{"start_line":200,"start_character":12,"end_line":200,"end_character":23},"updated":"2017-06-06 19:26:30.000000000","message":"We are changing these to mandatory --l7policy options.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":197,"context_line":"        parser \u003d super(SetL7Rule, self).get_parser(prog_name)"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"        parser.add_argument("},{"line_number":200,"context_line":"            \u0027l7policy\u0027,"},{"line_number":201,"context_line":"            metavar\u003d\u0027\u003cpolicy\u003e\u0027,"},{"line_number":202,"context_line":"            help\u003d\"L7policy to update rule on (name or ID)\""},{"line_number":203,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_e21ab76d","line":200,"range":{"start_line":200,"start_character":12,"end_line":200,"end_character":23},"in_reply_to":"df140735_6d17bda5","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":266,"context_line":"            json\u003dbody"},{"line_number":267,"context_line":"        )"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        return response"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_888d5f2c","line":269,"range":{"start_line":269,"start_character":8,"end_line":269,"end_character":23},"updated":"2017-06-06 19:26:30.000000000","message":"This returns raw json output, but it shouldn\u0027t return anything.","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":266,"context_line":"            json\u003dbody"},{"line_number":267,"context_line":"        )"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"        return response"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_020ecb31","line":269,"range":{"start_line":269,"start_character":8,"end_line":269,"end_character":23},"in_reply_to":"df140735_888d5f2c","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"eb1931e399f2a9701000312b36910c9ef9ef9b77","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        )"},{"line_number":112,"context_line":"        parser.add_argument("},{"line_number":113,"context_line":"            \u0027l7rule\u0027,"},{"line_number":114,"context_line":"            metavar\u003d\"\u003crule_id\u003e\","},{"line_number":115,"context_line":"            help\u003d\"l7rule to delete\""},{"line_number":116,"context_line":"        )"},{"line_number":117,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"bf091321_55835b05","line":114,"updated":"2017-06-08 01:26:58.000000000","message":"So, i don\u0027t really know for sure, but ... should this be \u003crule\u003e or is \u003crule_id\u003e ok here?","commit_id":"f9929dc37f417d17b24b4006737aca6b27e6b59c"}],"octaviaclient/osc/v2/utils.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"151a57af3745690d40ba07130c7a78aa3581ff71","unresolved":false,"context_lines":[{"line_number":312,"context_line":"        ),"},{"line_number":313,"context_line":"        \u0027value\u0027: (\u0027value\u0027, str),"},{"line_number":314,"context_line":"        \u0027key\u0027: (\u0027key\u0027, str),"},{"line_number":315,"context_line":"        \u0027type\u0027: (\u0027type\u0027, str.upper),"},{"line_number":316,"context_line":"        \u0027compare_type\u0027: (\u0027compare_type\u0027, str.upper),"},{"line_number":317,"context_line":"        \u0027enable\u0027: (\u0027admin_state_up\u0027, lambda x: True),"},{"line_number":318,"context_line":"        \u0027disable\u0027: (\u0027admin_state_up\u0027, lambda x: False)"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_cd9009f1","line":315,"range":{"start_line":315,"start_character":25,"end_line":315,"end_character":34},"updated":"2017-06-06 19:26:30.000000000","message":"This fails on python2.7:\n\ndescriptor \u0027upper\u0027 requires a \u0027str\u0027 object but received a \u0027unicode\u0027\nTraceback (most recent call last):\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/cliff/app.py\", line 400, in run_subcommand\n    result \u003d cmd.run(parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/osc_lib/command/command.py\", line 41, in run\n    return super(Command, self).run(parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/cliff/display.py\", line 112, in run\n    column_names, data \u003d self.take_action(parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/octaviaclient/osc/v2/l7rule.py\", line 92, in take_action\n    parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/octaviaclient/osc/v2/utils.py\", line 323, in get_l7rule_attrs\n    attrs \u003d _map_attrs(_attrs, attr_map)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/octaviaclient/osc/v2/utils.py\", line 403, in _map_attrs\n    mapped_attrs[attr_map[k][0]] \u003d attr_map[k][1](v)\nTypeError: descriptor \u0027upper\u0027 requires a \u0027str\u0027 object but received a \u0027unicode\u0027\nclean_up CreateL7Rule: descriptor \u0027upper\u0027 requires a \u0027str\u0027 object but received a \u0027unicode\u0027\nTraceback (most recent call last):\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/osc_lib/shell.py\", line 135, in run\n    ret_val \u003d super(OpenStackShell, self).run(argv)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/cliff/app.py\", line 279, in run\n    result \u003d self.run_subcommand(remainder)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/osc_lib/shell.py\", line 180, in run_subcommand\n    ret_value \u003d super(OpenStackShell, self).run_subcommand(argv)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/cliff/app.py\", line 400, in run_subcommand\n    result \u003d cmd.run(parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/osc_lib/command/command.py\", line 41, in run\n    return super(Command, self).run(parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/cliff/display.py\", line 112, in run\n    column_names, data \u003d self.take_action(parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/octaviaclient/osc/v2/l7rule.py\", line 92, in take_action\n    parsed_args)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/octaviaclient/osc/v2/utils.py\", line 323, in get_l7rule_attrs\n    attrs \u003d _map_attrs(_attrs, attr_map)\n  File \"/home/michjohn/.venvs/octaviaclient/local/lib/python2.7/site-packages/octaviaclient/osc/v2/utils.py\", line 403, in _map_attrs\n    mapped_attrs[attr_map[k][0]] \u003d attr_map[k][1](v)\nTypeError: descriptor \u0027upper\u0027 requires a \u0027str\u0027 object but received a \u0027unicode\u0027","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"959c2cf8761cf85a23b47f56dd1141b74b8ca790","unresolved":false,"context_lines":[{"line_number":312,"context_line":"        ),"},{"line_number":313,"context_line":"        \u0027value\u0027: (\u0027value\u0027, str),"},{"line_number":314,"context_line":"        \u0027key\u0027: (\u0027key\u0027, str),"},{"line_number":315,"context_line":"        \u0027type\u0027: (\u0027type\u0027, str.upper),"},{"line_number":316,"context_line":"        \u0027compare_type\u0027: (\u0027compare_type\u0027, str.upper),"},{"line_number":317,"context_line":"        \u0027enable\u0027: (\u0027admin_state_up\u0027, lambda x: True),"},{"line_number":318,"context_line":"        \u0027disable\u0027: (\u0027admin_state_up\u0027, lambda x: False)"}],"source_content_type":"text/x-python","patch_set":6,"id":"df140735_223aefcc","line":315,"range":{"start_line":315,"start_character":25,"end_line":315,"end_character":34},"in_reply_to":"df140735_cd9009f1","updated":"2017-06-06 20:59:05.000000000","message":"Done","commit_id":"8ed984886c9512666a97ea4347ba2610d8e22137"}]}