)]}'
{"tempest/common/isolated_creds.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"ba0574bf0ac6198293b3a55d97ef451ce1b92a9f","unresolved":false,"context_lines":[{"line_number":262,"context_line":"                    body[\u0027subnet\u0027][\u0027cidr\u0027] \u003d str(subnet_cidr)"},{"line_number":263,"context_line":"                    resp_body \u003d self.network_admin_client.create_subnet(body)"},{"line_number":264,"context_line":"                break"},{"line_number":265,"context_line":"            except n_exc.BadRequest as e:"},{"line_number":266,"context_line":"                if \u0027overlaps with another subnet\u0027 not in str(e):"},{"line_number":267,"context_line":"                    raise"},{"line_number":268,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_e03514f8","line":265,"updated":"2014-08-25 19:38:58.000000000","message":"So this fixes it for the official client case, but breaks it for the tempest client case. I think we need to handle both exceptions here.","commit_id":"b09bbb09c707353a4389998ab59f498455fb1ddd"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"441474f1080248031b4d95f9bab210518a707916","unresolved":false,"context_lines":[{"line_number":262,"context_line":"                    body[\u0027subnet\u0027][\u0027cidr\u0027] \u003d str(subnet_cidr)"},{"line_number":263,"context_line":"                    resp_body \u003d self.network_admin_client.create_subnet(body)"},{"line_number":264,"context_line":"                break"},{"line_number":265,"context_line":"            except n_exc.BadRequest as e:"},{"line_number":266,"context_line":"                if \u0027overlaps with another subnet\u0027 not in str(e):"},{"line_number":267,"context_line":"                    raise"},{"line_number":268,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_4b942944","line":265,"in_reply_to":"1abeadc6_e03514f8","updated":"2014-08-25 19:50:35.000000000","message":"Oh okay. I didn\u0027t realize that was for the tempest client. I just ran into this locally for the Big Switch CI system.","commit_id":"b09bbb09c707353a4389998ab59f498455fb1ddd"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"9ef5d2814ed48f25809e01042f9f6d07c1d488b5","unresolved":false,"context_lines":[{"line_number":262,"context_line":"                    body[\u0027subnet\u0027][\u0027cidr\u0027] \u003d str(subnet_cidr)"},{"line_number":263,"context_line":"                    resp_body \u003d self.network_admin_client.create_subnet(body)"},{"line_number":264,"context_line":"                break"},{"line_number":265,"context_line":"            except (n_exc.BadRequest, exceptions.BadRequest) as e:"},{"line_number":266,"context_line":"                if \u0027overlaps with another subnet\u0027 not in str(e):"},{"line_number":267,"context_line":"                    raise"},{"line_number":268,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1abeadc6_53d67228","line":265,"updated":"2014-08-26 19:16:26.000000000","message":"If the existing exception is wrong, why is it being added to rather than replaced?","commit_id":"c10a9cf9f84667578dc76c3dd83737ed3672f747"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"1cfbe86e2b969e60e871ae3559e439d1b9dbdfd9","unresolved":false,"context_lines":[{"line_number":262,"context_line":"                    body[\u0027subnet\u0027][\u0027cidr\u0027] \u003d str(subnet_cidr)"},{"line_number":263,"context_line":"                    resp_body \u003d self.network_admin_client.create_subnet(body)"},{"line_number":264,"context_line":"                break"},{"line_number":265,"context_line":"            except (n_exc.BadRequest, exceptions.BadRequest) as e:"},{"line_number":266,"context_line":"                if \u0027overlaps with another subnet\u0027 not in str(e):"},{"line_number":267,"context_line":"                    raise"},{"line_number":268,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1abeadc6_b6b32fe8","line":265,"in_reply_to":"1abeadc6_53d67228","updated":"2014-08-26 22:01:26.000000000","message":"It\u0027s not that the existing exception is wrong. It apparently depends on the client being used. See the comments on my first patch.","commit_id":"c10a9cf9f84667578dc76c3dd83737ed3672f747"}]}