)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"5aa56a1beb3d2630148bc87022b5d2da4d0ae133","unresolved":false,"context_lines":[{"line_number":11,"context_line":"isolation. Note the test before it asserts the listed transfers are"},{"line_number":12,"context_line":"greater than or equal to one so it works without tenant isolation."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1277788"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I68b2df2d7792da7c3a9ce001af41cd199ccdbc13"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAUX%2F%2Fxfs%3D","line":14,"updated":"2014-02-10 21:49:55.000000000","message":"I\u0027d still like to know what is causing the leaks in the single threaded case. Can you make this a partial bug.","commit_id":"2d9bdd0ef0be44c9cc68df732239945d498d458f"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0c77b43991f4ff18bb7ad00ff1c878b7bcf58449","unresolved":false,"context_lines":[{"line_number":11,"context_line":"isolation. Note the test before it asserts the listed transfers are"},{"line_number":12,"context_line":"greater than or equal to one so it works without tenant isolation."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1277788"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I68b2df2d7792da7c3a9ce001af41cd199ccdbc13"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAUX%2F%2Fxaw%3D","line":14,"in_reply_to":"AAAAUX%2F%2Fxfs%3D","updated":"2014-02-10 21:59:06.000000000","message":"Done","commit_id":"2d9bdd0ef0be44c9cc68df732239945d498d458f"}],"tempest/api/volume/test_volume_transfers.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"5aa56a1beb3d2630148bc87022b5d2da4d0ae133","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        if CONF.compute.allow_tenant_isolation:"},{"line_number":114,"context_line":"            self.assertEqual(volume[\u0027id\u0027], body[0][\u0027volume_id\u0027])"},{"line_number":115,"context_line":"        else:"},{"line_number":116,"context_line":"            for transfer in body:"},{"line_number":117,"context_line":"                if volume[\u0027id\u0027] \u003d\u003d transfer[\u0027volume_id\u0027]:"},{"line_number":118,"context_line":"                    break"},{"line_number":119,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2FxgU%3D","line":116,"updated":"2014-02-10 21:49:55.000000000","message":"What I meant is that wouldn\u0027t this loop work fine for both the isolated and non-isolated case. Why bother to special case it if works for both cases.","commit_id":"2d9bdd0ef0be44c9cc68df732239945d498d458f"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0c77b43991f4ff18bb7ad00ff1c878b7bcf58449","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        if CONF.compute.allow_tenant_isolation:"},{"line_number":114,"context_line":"            self.assertEqual(volume[\u0027id\u0027], body[0][\u0027volume_id\u0027])"},{"line_number":115,"context_line":"        else:"},{"line_number":116,"context_line":"            for transfer in body:"},{"line_number":117,"context_line":"                if volume[\u0027id\u0027] \u003d\u003d transfer[\u0027volume_id\u0027]:"},{"line_number":118,"context_line":"                    break"},{"line_number":119,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2Fxao%3D","line":116,"in_reply_to":"AAAAUX%2F%2FxgU%3D","updated":"2014-02-10 21:59:06.000000000","message":"Ohhhh, well that\u0027s much simpler - sure!","commit_id":"2d9bdd0ef0be44c9cc68df732239945d498d458f"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"8fb843dbd136e19e25c63b73b51e1c5b1e9bbac4","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        # List all volume transfers (looking for the one we created)"},{"line_number":111,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":112,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":113,"context_line":"        for transfer in body:"},{"line_number":114,"context_line":"            if volume[\u0027id\u0027] \u003d\u003d transfer[\u0027volume_id\u0027]:"},{"line_number":115,"context_line":"                break"},{"line_number":116,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUX%2F%2FwhY%3D","line":113,"updated":"2014-02-11 00:27:05.000000000","message":"I probably would have done this whole block as something like:\n\nself.assertIn(volume[\u0027id\u0027], map(lambda transfer: transfer[\u0027volume_id\u0027], body))\n\nbut whatever works.","commit_id":"1f391f8372cf6ab22f9eeb77314bbfe79e4e6f65"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a4c1bcd405174eb0eeef78c54a1ee67ac4b8ca19","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        # List all volume transfers (looking for the one we created)"},{"line_number":111,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":112,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":113,"context_line":"        for transfer in body:"},{"line_number":114,"context_line":"            if volume[\u0027id\u0027] \u003d\u003d transfer[\u0027volume_id\u0027]:"},{"line_number":115,"context_line":"                break"},{"line_number":116,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUX%2F%2FvPY%3D","line":113,"in_reply_to":"AAAAUX%2F%2FwhY%3D","updated":"2014-02-11 05:30:52.000000000","message":"I knew there was probably a slicker way to do this, but my experience with such tricks isn\u0027t there yet. :)","commit_id":"1f391f8372cf6ab22f9eeb77314bbfe79e4e6f65"}]}