)]}'
{"tempest/cmd/javelin.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"953fbff6411d1f38bf40e5064d75583ad66e6133","unresolved":false,"context_lines":[{"line_number":109,"context_line":"    for tenant in tenants:"},{"line_number":110,"context_line":"        tenant_id \u003d admin.identity.get_tenant_by_name(tenant)[\u0027id\u0027]"},{"line_number":111,"context_line":"        r, body \u003d admin.identity.delete_tenant(tenant_id)"},{"line_number":112,"context_line":"    if r !\u003d 200:"},{"line_number":113,"context_line":"        raise ValueError(\"unable to destroy tenant: [%s] %s\" % (r, body))"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"##############"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_7bd55bb8","line":112,"updated":"2014-07-08 11:21:32.000000000","message":"Indentation issue: you should check the return code for each call to delete_tenant(), not only the last one.","commit_id":"8b7cf6e7f5a1a196e157a5c1b5a34aa8defb72eb"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"953fbff6411d1f38bf40e5064d75583ad66e6133","unresolved":false,"context_lines":[{"line_number":181,"context_line":"    for user in users:"},{"line_number":182,"context_line":"        user_id \u003d admin.identity.get_user_by_name(user[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":183,"context_line":"        r, body \u003d admin.identity.delete_user(user_id)"},{"line_number":184,"context_line":"    if r !\u003d 200:"},{"line_number":185,"context_line":"        raise ValueError(\"unable to destroy user: [%s] %s\" % (r, body))"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_9bd94ff7","line":184,"updated":"2014-07-08 11:21:32.000000000","message":"Ditto.","commit_id":"8b7cf6e7f5a1a196e157a5c1b5a34aa8defb72eb"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"953fbff6411d1f38bf40e5064d75583ad66e6133","unresolved":false,"context_lines":[{"line_number":300,"context_line":"    for obj in objects:"},{"line_number":301,"context_line":"        client \u003d client_for_user(obj[\u0027owner\u0027])"},{"line_number":302,"context_line":"        r, body \u003d client.objects.delete_object(obj[\u0027container\u0027], obj[\u0027name\u0027])"},{"line_number":303,"context_line":"    if r !\u003d 200:"},{"line_number":304,"context_line":"        raise ValueError(\"unable to destroy object: [%s] %s\" % (r, body))"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_bbd613c2","line":303,"updated":"2014-07-08 11:21:32.000000000","message":"Ditto.","commit_id":"8b7cf6e7f5a1a196e157a5c1b5a34aa8defb72eb"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"953fbff6411d1f38bf40e5064d75583ad66e6133","unresolved":false,"context_lines":[{"line_number":355,"context_line":"        client \u003d client_for_user(image[\u0027owner\u0027])"},{"line_number":356,"context_line":"        image_id \u003d _get_image_by_name(client, image[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":357,"context_line":"        r, body \u003d client.images.delete_image(image_id)"},{"line_number":358,"context_line":"    if r !\u003d 200:"},{"line_number":359,"context_line":"        raise ValueError(\"unable to destroy image: [%s] %s\" % (r, body))"},{"line_number":360,"context_line":""},{"line_number":361,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_5be7b7ac","line":358,"updated":"2014-07-08 11:21:32.000000000","message":"Ditto.","commit_id":"8b7cf6e7f5a1a196e157a5c1b5a34aa8defb72eb"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"953fbff6411d1f38bf40e5064d75583ad66e6133","unresolved":false,"context_lines":[{"line_number":398,"context_line":"        client \u003d client_for_user(server[\u0027owner\u0027])"},{"line_number":399,"context_line":"        server_id \u003d _get_server_by_name(client, server[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":400,"context_line":"        r, body \u003d client.servers.delete_server(server_id)"},{"line_number":401,"context_line":"    if r !\u003d 200:"},{"line_number":402,"context_line":"        raise ValueError(\"unable to destroy server: [%s] %s\" % (r, body))"},{"line_number":403,"context_line":""},{"line_number":404,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_9b006f37","line":401,"updated":"2014-07-08 11:21:32.000000000","message":"Ditto.","commit_id":"8b7cf6e7f5a1a196e157a5c1b5a34aa8defb72eb"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"953fbff6411d1f38bf40e5064d75583ad66e6133","unresolved":false,"context_lines":[{"line_number":433,"context_line":"        client \u003d client_for_user(volume[\u0027owner\u0027])"},{"line_number":434,"context_line":"        volume_id \u003d _get_volume_by_name(client, volume[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":435,"context_line":"        r, body \u003d client.volumes.delete_volume(volume_id)"},{"line_number":436,"context_line":"    if r !\u003d 200:"},{"line_number":437,"context_line":"        raise ValueError(\"unable to destroy volume: [%s] %s\" % (r, body))"},{"line_number":438,"context_line":""},{"line_number":439,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_7bec7b90","line":436,"updated":"2014-07-08 11:21:32.000000000","message":"Ditto.","commit_id":"8b7cf6e7f5a1a196e157a5c1b5a34aa8defb72eb"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"91c65e37ccac99857826a1b68c26acff6b877f1a","unresolved":false,"context_lines":[{"line_number":110,"context_line":"    for tenant in tenants:"},{"line_number":111,"context_line":"        tenant_id \u003d admin.identity.get_tenant_by_name(tenant)[\u0027id\u0027]"},{"line_number":112,"context_line":"        r, body \u003d admin.identity.delete_tenant(tenant_id)"},{"line_number":113,"context_line":"        if not (200 \u003e\u003d int(r[\u0027status\u0027]) \u003c 299):"},{"line_number":114,"context_line":"            raise ValueError(\"unable to destroy tenant: [%s] %s\" % (r, body))"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"##############"}],"source_content_type":"text/x-python","patch_set":11,"id":"1abeadc6_b32426c7","line":113,"updated":"2014-08-26 19:35:58.000000000","message":"The tempest identity client already checks this.","commit_id":"2192cfd64f6970007a343215fa26cfdcf1e7a81a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"158620191c18ec83255f06ba4a13a3d3c565e61d","unresolved":false,"context_lines":[{"line_number":110,"context_line":"    for tenant in tenants:"},{"line_number":111,"context_line":"        tenant_id \u003d admin.identity.get_tenant_by_name(tenant)[\u0027id\u0027]"},{"line_number":112,"context_line":"        r, body \u003d admin.identity.delete_tenant(tenant_id)"},{"line_number":113,"context_line":"        if not (200 \u003e\u003d int(r[\u0027status\u0027]) \u003c 299):"},{"line_number":114,"context_line":"            raise ValueError(\"unable to destroy tenant: [%s] %s\" % (r, body))"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"##############"}],"source_content_type":"text/x-python","patch_set":11,"id":"1abeadc6_4b02173f","line":113,"in_reply_to":"1abeadc6_b32426c7","updated":"2014-08-27 13:33:34.000000000","message":"Done","commit_id":"2192cfd64f6970007a343215fa26cfdcf1e7a81a"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"91c65e37ccac99857826a1b68c26acff6b877f1a","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    for user in users:"},{"line_number":183,"context_line":"        user_id \u003d admin.identity.get_user_by_name(user[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":184,"context_line":"        r, body \u003d admin.identity.delete_user(user_id)"},{"line_number":185,"context_line":"        if not (200 \u003e\u003d int(r[\u0027status\u0027]) \u003c 299):"},{"line_number":186,"context_line":"            raise ValueError(\"unable to destroy user: [%s] %s\" % (r, body))"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"1abeadc6_d31fe26e","line":185,"updated":"2014-08-26 19:35:58.000000000","message":"Ditto.","commit_id":"2192cfd64f6970007a343215fa26cfdcf1e7a81a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"158620191c18ec83255f06ba4a13a3d3c565e61d","unresolved":false,"context_lines":[{"line_number":182,"context_line":"    for user in users:"},{"line_number":183,"context_line":"        user_id \u003d admin.identity.get_user_by_name(user[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":184,"context_line":"        r, body \u003d admin.identity.delete_user(user_id)"},{"line_number":185,"context_line":"        if not (200 \u003e\u003d int(r[\u0027status\u0027]) \u003c 299):"},{"line_number":186,"context_line":"            raise ValueError(\"unable to destroy user: [%s] %s\" % (r, body))"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"1abeadc6_2ba31b50","line":185,"in_reply_to":"1abeadc6_d31fe26e","updated":"2014-08-27 13:33:34.000000000","message":"Done","commit_id":"2192cfd64f6970007a343215fa26cfdcf1e7a81a"},{"author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"change_message_id":"91c65e37ccac99857826a1b68c26acff6b877f1a","unresolved":false,"context_lines":[{"line_number":481,"context_line":"        client \u003d client_for_user(volume[\u0027owner\u0027])"},{"line_number":482,"context_line":"        volume_id \u003d _get_volume_by_name(client, volume[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":483,"context_line":"        r, body \u003d client.volumes.delete_volume(volume_id)"},{"line_number":484,"context_line":"        if not (200 \u003e\u003d int(r[\u0027status\u0027]) \u003c 299):"},{"line_number":485,"context_line":"            raise ValueError(\"unable to destroy volume: [%s] %s\" % (r, body))"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"1abeadc6_531bd27f","line":484,"updated":"2014-08-26 19:35:58.000000000","message":"The tempest volume client already checks this.","commit_id":"2192cfd64f6970007a343215fa26cfdcf1e7a81a"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"158620191c18ec83255f06ba4a13a3d3c565e61d","unresolved":false,"context_lines":[{"line_number":481,"context_line":"        client \u003d client_for_user(volume[\u0027owner\u0027])"},{"line_number":482,"context_line":"        volume_id \u003d _get_volume_by_name(client, volume[\u0027name\u0027])[\u0027id\u0027]"},{"line_number":483,"context_line":"        r, body \u003d client.volumes.delete_volume(volume_id)"},{"line_number":484,"context_line":"        if not (200 \u003e\u003d int(r[\u0027status\u0027]) \u003c 299):"},{"line_number":485,"context_line":"            raise ValueError(\"unable to destroy volume: [%s] %s\" % (r, body))"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"1abeadc6_4bb0578b","line":484,"in_reply_to":"1abeadc6_531bd27f","updated":"2014-08-27 13:33:34.000000000","message":"Done","commit_id":"2192cfd64f6970007a343215fa26cfdcf1e7a81a"}]}