)]}'
{"tempest/api/volume/test_volume_transfers.py":[{"author":{"_account_id":5371,"name":"Mathew Odden","email":"locke105@gmail.com","username":"locke105"},"change_message_id":"b426aafb0c6ad3831a64c2e9fcf7ae2eb71f5e8a","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(body, matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F%2BPo%3D","line":92,"updated":"2014-02-08 03:22:34.000000000","message":"This looks weird... but I guess it works.","commit_id":"18eaf2b676eb9a312eeae39d02e4c616bd1f73d7"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"2e417dfa90beb3da3f22d9a3d2ba3c5df410c5c2","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(body, matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F6I8%3D","line":92,"in_reply_to":"AAAAUX%2F%2F%2BOw%3D","updated":"2014-02-10 01:51:52.000000000","message":"@Matt, As this test require tenant isolation, how about to fix it using \"force_tenant_isolation\"?\nfor example:\nclass ListServersNegativeV3Test(base.BaseV3ComputeTest):\n    _interface \u003d \u0027json\u0027\n    force_tenant_isolation \u003d True\n\n    @classmethod\n    def setUpClass(cls):\n...snip...","commit_id":"18eaf2b676eb9a312eeae39d02e4c616bd1f73d7"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"353ca226afc49f9ea6257b28e047f53e01eeaaae","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(body, matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F%2BOw%3D","line":92,"in_reply_to":"AAAAUX%2F%2F%2BPo%3D","updated":"2014-02-08 03:32:45.000000000","message":"Yeah, see the details here:\n\nhttps://bugs.launchpad.net/tempest/+bug/1277788","commit_id":"18eaf2b676eb9a312eeae39d02e4c616bd1f73d7"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e3e66612d5a7516fc8b2be4961529abb87b90fd5","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(body, matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F5GY%3D","line":92,"in_reply_to":"AAAAUX%2F%2F5TE%3D","updated":"2014-02-10 05:36:50.000000000","message":"You\u0027re right, it should be len(body), I had GreaterThan confused with HasLength which is a different matcher.  The fact this passed scares me, reminds me of c programming where everything flies in a condition.","commit_id":"18eaf2b676eb9a312eeae39d02e4c616bd1f73d7"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"3e950700c0b3e5df1697b5abf45711720cb2b08b","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(body, matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F5TE%3D","line":92,"in_reply_to":"AAAAUX%2F%2F6I8%3D","updated":"2014-02-10 05:07:16.000000000","message":"This test class does not require tenant isolation, it just needs 2 tenants. Even if it was the case that it was a requirement for this test class it would be outside the scope of this patch.\n\nAlso don\u0027t you need to put body in len() here? I\u0027m pretty sure what you have here will always return True. (well as long as body isn\u0027t None, which I don\u0027t think the request method can return without raising an exception first)","commit_id":"18eaf2b676eb9a312eeae39d02e4c616bd1f73d7"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"cba79ad8f300d878bd5dd96bb6b9c175be12a0f9","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(body, matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUX%2F%2F5SA%3D","line":92,"in_reply_to":"AAAAUX%2F%2F6I8%3D","updated":"2014-02-10 05:01:52.000000000","message":"Zhi Kun, the point of this patch is to make the test work on python 2.6, not to fix bug 1277788.  That\u0027s a separate patch.","commit_id":"18eaf2b676eb9a312eeae39d02e4c616bd1f73d7"},{"author":{"_account_id":5371,"name":"Mathew Odden","email":"locke105@gmail.com","username":"locke105"},"change_message_id":"28665fa73cb76d3b6e71619353d0bf5579d0984c","unresolved":false,"context_lines":[{"line_number":89,"context_line":"        # or equal to 1"},{"line_number":90,"context_line":"        resp, body \u003d self.client.list_volume_transfers()"},{"line_number":91,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":92,"context_line":"        self.assertThat(len(body), matchers.GreaterThan(0))"},{"line_number":93,"context_line":""},{"line_number":94,"context_line":"        # Accept a volume transfer by alt_tenant"},{"line_number":95,"context_line":"        resp, body \u003d self.alt_client.accept_volume_transfer(transfer_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUX%2F%2F0%2Bg%3D","line":92,"updated":"2014-02-10 15:17:32.000000000","message":"This makes more sense to my brain now :)","commit_id":"1c42d6bdbff33a21053c53a89b9766d3002e7670"}]}