)]}'
{"tempest/api/compute/admin/test_live_migration.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"3e3301ee61f67956f3b40fe20e80ef4110fe1d1e","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        self.assertEqual(target_host, self._get_host_for_server(server_id))"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"class LiveBlockMigrationTestV234JSON(LiveBlockMigrationTestJSON):"},{"line_number":233,"context_line":"    min_microversion \u003d \u00272.34\u0027"},{"line_number":234,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_68711e44","line":232,"updated":"2016-07-06 14:49:03.000000000","message":"Is this done because Tempest only tests based on min_microversion? Given that there are no changes to the underlying tests it would be nicer if the same test class could check multiple microversions, but perhaps there\u0027s no facility for that.","commit_id":"e2489b2e7d4d0fb292647401f8c4cca80eb76717"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"5a2c16500732c9a192b4ea53b52597ccc5c3be92","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        self.assertEqual(target_host, self._get_host_for_server(server_id))"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"class LiveBlockMigrationTestV234JSON(LiveBlockMigrationTestJSON):"},{"line_number":233,"context_line":"    min_microversion \u003d \u00272.34\u0027"},{"line_number":234,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_6008dc28","line":232,"in_reply_to":"1aa78d24_474eb6c1","updated":"2016-07-07 13:09:54.000000000","message":"Cool. Thanks for the info.","commit_id":"e2489b2e7d4d0fb292647401f8c4cca80eb76717"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d0ecaf0108c8e8490818896968fab8ea1dccebde","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        self.assertEqual(target_host, self._get_host_for_server(server_id))"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"class LiveBlockMigrationTestV234JSON(LiveBlockMigrationTestJSON):"},{"line_number":233,"context_line":"    min_microversion \u003d \u00272.34\u0027"},{"line_number":234,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_474eb6c1","line":232,"in_reply_to":"1aa78d24_68711e44","updated":"2016-07-07 00:31:34.000000000","message":"Yea, microversion are defined on class level so single test class support single microversion to be requested on Nova based on [1].\nIn case same tests needs to run, yea we need to define the new class. But in case of superclass, there might be some redundant test cases which are not related to changes in that microversion which we should avoid by fixing the inheritance. \n\n\n1 - http://docs.openstack.org/developer/tempest/library/api_microversion_testing.html#tempest.lib.common.api_version_utils.select_request_microversion","commit_id":"e2489b2e7d4d0fb292647401f8c4cca80eb76717"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f2429de411a7088b49ca316d60091daa1d2d8924","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"class LiveBlockMigrationTestV225JSON(LiveBlockMigrationTestJSON):"},{"line_number":184,"context_line":"    min_microversion \u003d \u00272.25\u0027"},{"line_number":185,"context_line":"    max_microversion \u003d \u00272.33\u0027"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"    @test.idempotent_id(\u002726670215-0e4f-4eb4-a920-86e38aa25930\u0027)"},{"line_number":188,"context_line":"    def test_live_block_migration_auto(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_dd42ae3d","line":185,"updated":"2016-07-11 21:20:06.000000000","message":"I think you have to set this in the 2.34 class for 2.34 else you get a 406:\n\nhttp://logs.openstack.org/56/338256/4/experimental/gate-tempest-dsvm-multinode-live-migration/479eb1a/console.html#_2016-07-11_21_07_24_356981","commit_id":"353712a0e90111cf1152fe23297ada9bc3a712ef"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"6b1b6f4b248e6c26bf80a855967187db3d7a325f","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"class LiveBlockMigrationTestV234JSON(LiveBlockMigrationTestJSON):"},{"line_number":233,"context_line":"    min_microversion \u003d \u00272.34\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"1aa78d24_3d7ad288","line":233,"updated":"2016-07-11 21:20:53.000000000","message":"Add this in this test:\n\nmax_microversion \u003d \u0027latest\u0027","commit_id":"353712a0e90111cf1152fe23297ada9bc3a712ef"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ddc15267a2c2503b2392e03a09c4af49865a9f7d","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"class LiveBlockMigrationTestV225JSON(LiveBlockMigrationTestJSON):"},{"line_number":184,"context_line":"    min_microversion \u003d \u00272.25\u0027"},{"line_number":185,"context_line":"    max_microversion \u003d \u00272.33\u0027"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":"    @test.idempotent_id(\u002726670215-0e4f-4eb4-a920-86e38aa25930\u0027)"},{"line_number":188,"context_line":"    def test_live_block_migration_auto(self):"}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_0cf7dc8b","line":185,"range":{"start_line":185,"start_character":0,"end_line":185,"end_character":29},"updated":"2016-07-12 06:20:15.000000000","message":"as you are not inheriting 2.34 tests from this class, you should not cap this with max_microversion","commit_id":"8f6407bca5ccbfd5f484aff26429615e64a21b6e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ddc15267a2c2503b2392e03a09c4af49865a9f7d","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        self.assertEqual(target_host, self._get_host_for_server(server_id))"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"class LiveBlockMigrationTestV234JSON(LiveBlockMigrationTestJSON):"},{"line_number":233,"context_line":"    min_microversion \u003d \u00272.34\u0027"},{"line_number":234,"context_line":"    max_microversion \u003d \u0027latest\u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_ece410e1","line":234,"range":{"start_line":232,"start_character":0,"end_line":234,"end_character":31},"updated":"2016-07-12 06:20:15.000000000","message":"I think for 2.34 we should check the async call for live migration request and tracking the same via instance action.\n\nBut this will just run the tests with version 2.34 without those verification.","commit_id":"8f6407bca5ccbfd5f484aff26429615e64a21b6e"}]}