)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"fb8717893220902fdd8b03832f2873665e3e1a97","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Reproduce bug 1724172 in the functional test env"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To reproducing bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3f4b6375_f8be6063","line":9,"range":{"start_line":9,"start_character":3,"end_line":9,"end_character":14},"updated":"2017-10-23 17:07:22.000000000","message":"reproduce","commit_id":"683426b4cb0fdaa45f26aeec7a14daf2f279300f"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"374c52946e00b0053cf8df5d2296a02e8b627238","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Reproduce bug 1724172 in the functional test env"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To reproducing bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f93b717_f64d5472","line":9,"range":{"start_line":9,"start_character":3,"end_line":9,"end_character":14},"in_reply_to":"3f4b6375_f8be6063","updated":"2018-01-29 16:29:13.000000000","message":"Done","commit_id":"683426b4cb0fdaa45f26aeec7a14daf2f279300f"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"39920ccbc3b420efbbc0e48cad7c3e8950d94bde","unresolved":false,"context_lines":[{"line_number":9,"context_line":"To reproduce bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"},{"line_number":13,"context_line":"and those patches revealed another bug in the resource cleanup"},{"line_number":14,"context_line":"of evacuated instances. This patch adds a regression test that"},{"line_number":15,"context_line":"reproduces the bug."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"bfb3d3c7_008eec77","line":12,"range":{"start_line":12,"start_character":21,"end_line":12,"end_character":63},"updated":"2019-05-17 16:13:27.000000000","message":"But are we going to be able to backport the fix with all of these big refactors underneath this?","commit_id":"b5a64af5ab0548a4f571b98e029e287d8dff858f"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"067c66cf65f91704b2cef570acf59d05094939b3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"To reproduce bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"},{"line_number":13,"context_line":"and those patches revealed another bug in the resource cleanup"},{"line_number":14,"context_line":"of evacuated instances. This patch adds a regression test that"},{"line_number":15,"context_line":"reproduces the bug."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"bfb3d3c7_62ad1387","line":12,"range":{"start_line":12,"start_character":21,"end_line":12,"end_character":63},"in_reply_to":"bfb3d3c7_008eec77","updated":"2019-05-23 10:02:05.000000000","message":"I can work on the backport of these. If the refactor is considered not backportable then (is it?) then I have to make  a bugfix without functional reproduce. What do you suggest?","commit_id":"b5a64af5ab0548a4f571b98e029e287d8dff858f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"128f2ca1ea72c97b1aaf52d06f35e02c61cb93b0","unresolved":false,"context_lines":[{"line_number":9,"context_line":"To reproduce bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"},{"line_number":13,"context_line":"and those patches revealed another bug in the resource cleanup"},{"line_number":14,"context_line":"of evacuated instances. This patch adds a regression test that"},{"line_number":15,"context_line":"reproduces the bug."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"bfb3d3c7_51009f01","line":12,"range":{"start_line":12,"start_character":21,"end_line":12,"end_character":63},"in_reply_to":"bfb3d3c7_2a2586ab","updated":"2019-05-28 09:41:38.000000000","message":"Yes I would be okay with this, thanks gibi!","commit_id":"b5a64af5ab0548a4f571b98e029e287d8dff858f"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"031f3fb16013b197b25ab927b8848eaa359b3d21","unresolved":false,"context_lines":[{"line_number":9,"context_line":"To reproduce bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"},{"line_number":13,"context_line":"and those patches revealed another bug in the resource cleanup"},{"line_number":14,"context_line":"of evacuated instances. This patch adds a regression test that"},{"line_number":15,"context_line":"reproduces the bug."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"bfb3d3c7_2a2586ab","line":12,"range":{"start_line":12,"start_character":21,"end_line":12,"end_character":63},"in_reply_to":"bfb3d3c7_3818aad1","updated":"2019-05-27 07:53:56.000000000","message":"If I remove the I2cf2fcbaebc706f897ce5dfbff47d32117064f9c from this series then I have to go back and figure out how to make the usage of the set_nodes() global right in https://review.opendev.org/#/c/512552/10/nova/test.py@471 which is not trivial. Would it be acceptable to fix the bug 1724172 with only unit test coverage, then backport it to stable and then on master only add the whole test-refactor and functional test for the case patches?","commit_id":"b5a64af5ab0548a4f571b98e029e287d8dff858f"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a0ef0876a019399dc7518cead98551979be4566b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"To reproduce bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"},{"line_number":13,"context_line":"and those patches revealed another bug in the resource cleanup"},{"line_number":14,"context_line":"of evacuated instances. This patch adds a regression test that"},{"line_number":15,"context_line":"reproduces the bug."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"bfb3d3c7_0c8f6975","line":12,"range":{"start_line":12,"start_character":21,"end_line":12,"end_character":63},"in_reply_to":"bfb3d3c7_51009f01","updated":"2019-05-28 16:04:01.000000000","message":"I made the bugfix in https://review.opendev.org/#/c/512623 backportable as we agreed here.","commit_id":"b5a64af5ab0548a4f571b98e029e287d8dff858f"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"71591e783b60473f5228fac71e24c2612c347a34","unresolved":false,"context_lines":[{"line_number":9,"context_line":"To reproduce bug 1721652 the functional test env was enhanced to"},{"line_number":10,"context_line":"support a more realistic compute service restart. During that code"},{"line_number":11,"context_line":"review a follow up was suggested to further enhance and generalize"},{"line_number":12,"context_line":"the service restart. This work was proposed in previous patches"},{"line_number":13,"context_line":"and those patches revealed another bug in the resource cleanup"},{"line_number":14,"context_line":"of evacuated instances. This patch adds a regression test that"},{"line_number":15,"context_line":"reproduces the bug."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"bfb3d3c7_3818aad1","line":12,"range":{"start_line":12,"start_character":21,"end_line":12,"end_character":63},"in_reply_to":"bfb3d3c7_62ad1387","updated":"2019-05-23 11:58:02.000000000","message":"I2cf2fcbaebc706f897ce5dfbff47d32117064f9c would be a concern on stable. Is reordering the series to land a basic functional regression test first with the bugfix out of the question?","commit_id":"b5a64af5ab0548a4f571b98e029e287d8dff858f"}],"nova/tests/functional/test_servers.py":[{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"961eaef29b61d977274b55d1a3f28b573cd89737","unresolved":false,"context_lines":[{"line_number":2535,"context_line":"        source_compute_id \u003d self.admin_api.get_services("},{"line_number":2536,"context_line":"            host\u003dsource_hostname, binary\u003d\u0027nova-compute\u0027)[0][\u0027id\u0027]"},{"line_number":2537,"context_line":""},{"line_number":2538,"context_line":"        self.compute1.stop()"},{"line_number":2539,"context_line":"        # force it down to avoid waiting for the service group to time out"},{"line_number":2540,"context_line":"        self.admin_api.put_service("},{"line_number":2541,"context_line":"            source_compute_id, {\u0027forced_down\u0027: \u0027true\u0027})"},{"line_number":2542,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"5f7c97a3_b5a3259e","line":2539,"range":{"start_line":2538,"start_character":2,"end_line":2539,"end_character":74},"updated":"2018-05-28 11:03:11.000000000","message":"Why do we need this stop at all?","commit_id":"52c4f4c822e37ebbe7ea9a9a61f08f822e7273af"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e4a68b3d7f6ca1455fcaef4d76cb533590446103","unresolved":false,"context_lines":[{"line_number":2535,"context_line":"        source_compute_id \u003d self.admin_api.get_services("},{"line_number":2536,"context_line":"            host\u003dsource_hostname, binary\u003d\u0027nova-compute\u0027)[0][\u0027id\u0027]"},{"line_number":2537,"context_line":""},{"line_number":2538,"context_line":"        self.compute1.stop()"},{"line_number":2539,"context_line":"        # force it down to avoid waiting for the service group to time out"},{"line_number":2540,"context_line":"        self.admin_api.put_service("},{"line_number":2541,"context_line":"            source_compute_id, {\u0027forced_down\u0027: \u0027true\u0027})"},{"line_number":2542,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"5f7c97a3_3e8f1bce","line":2539,"range":{"start_line":2538,"start_character":2,"end_line":2539,"end_character":74},"in_reply_to":"5f7c97a3_b5a3259e","updated":"2018-07-03 12:21:27.000000000","message":"the API contract of force_down is the following \"Whether or not this service was forced down manually by an administrator. \" So to follow that rule I manually stopped the service at least as in the func env there is no way to stop a hypervisor.","commit_id":"52c4f4c822e37ebbe7ea9a9a61f08f822e7273af"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"71591e783b60473f5228fac71e24c2612c347a34","unresolved":false,"context_lines":[{"line_number":2313,"context_line":"        self._test_evacuate(clean_hypervisor\u003dFalse)"},{"line_number":2314,"context_line":""},{"line_number":2315,"context_line":"    def test_evacuate_clean_hypervisor(self):"},{"line_number":2316,"context_line":"        # The _test_evacuate call is inlined here to reproduce bug 1724172."},{"line_number":2317,"context_line":"        # After that bug is solved this whole function body should be replaced"},{"line_number":2318,"context_line":"        # with a simple call:"},{"line_number":2319,"context_line":"        # self._test_evacuate(clean_hypervisor\u003dTrue)"},{"line_number":2320,"context_line":"        source_hostname \u003d self.compute1.host"},{"line_number":2321,"context_line":"        dest_hostname \u003d self.compute2.host"},{"line_number":2322,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_3893ca51","line":2319,"range":{"start_line":2316,"start_character":0,"end_line":2319,"end_character":52},"updated":"2019-05-23 11:58:02.000000000","message":"supernit - docstring?","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"24aea3c6b37628428e5a74e33b2ba4283815e5fc","unresolved":false,"context_lines":[{"line_number":2313,"context_line":"        self._test_evacuate(clean_hypervisor\u003dFalse)"},{"line_number":2314,"context_line":""},{"line_number":2315,"context_line":"    def test_evacuate_clean_hypervisor(self):"},{"line_number":2316,"context_line":"        # The _test_evacuate call is inlined here to reproduce bug 1724172."},{"line_number":2317,"context_line":"        # After that bug is solved this whole function body should be replaced"},{"line_number":2318,"context_line":"        # with a simple call:"},{"line_number":2319,"context_line":"        # self._test_evacuate(clean_hypervisor\u003dTrue)"},{"line_number":2320,"context_line":"        source_hostname \u003d self.compute1.host"},{"line_number":2321,"context_line":"        dest_hostname \u003d self.compute2.host"},{"line_number":2322,"context_line":""}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_49055345","line":2319,"range":{"start_line":2316,"start_character":0,"end_line":2319,"end_character":52},"in_reply_to":"bfb3d3c7_3893ca51","updated":"2019-05-28 16:02:52.000000000","message":"N/A","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"71591e783b60473f5228fac71e24c2612c347a34","unresolved":false,"context_lines":[{"line_number":2332,"context_line":"            source_compute_id, {\u0027forced_down\u0027: \u0027true\u0027})"},{"line_number":2333,"context_line":""},{"line_number":2334,"context_line":"        # evacuate the server"},{"line_number":2335,"context_line":"        post \u003d {\u0027evacuate\u0027: {}}"},{"line_number":2336,"context_line":"        self.api.post_server_action("},{"line_number":2337,"context_line":"            server[\u0027id\u0027], post)"},{"line_number":2338,"context_line":"        expected_params \u003d {\u0027OS-EXT-SRV-ATTR:host\u0027: dest_hostname,"},{"line_number":2339,"context_line":"                           \u0027status\u0027: \u0027ACTIVE\u0027}"},{"line_number":2340,"context_line":"        server \u003d self._wait_for_server_parameter(self.api, server,"}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_d884ee0e","line":2337,"range":{"start_line":2335,"start_character":0,"end_line":2337,"end_character":31},"updated":"2019-05-23 11:58:02.000000000","message":"nit - copied from the original test but couldn\u0027t we make this one line?","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"24aea3c6b37628428e5a74e33b2ba4283815e5fc","unresolved":false,"context_lines":[{"line_number":2332,"context_line":"            source_compute_id, {\u0027forced_down\u0027: \u0027true\u0027})"},{"line_number":2333,"context_line":""},{"line_number":2334,"context_line":"        # evacuate the server"},{"line_number":2335,"context_line":"        post \u003d {\u0027evacuate\u0027: {}}"},{"line_number":2336,"context_line":"        self.api.post_server_action("},{"line_number":2337,"context_line":"            server[\u0027id\u0027], post)"},{"line_number":2338,"context_line":"        expected_params \u003d {\u0027OS-EXT-SRV-ATTR:host\u0027: dest_hostname,"},{"line_number":2339,"context_line":"                           \u0027status\u0027: \u0027ACTIVE\u0027}"},{"line_number":2340,"context_line":"        server \u003d self._wait_for_server_parameter(self.api, server,"}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_899cab0e","line":2337,"range":{"start_line":2335,"start_character":0,"end_line":2337,"end_character":31},"in_reply_to":"bfb3d3c7_d884ee0e","updated":"2019-05-28 16:02:52.000000000","message":"N/A","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"71591e783b60473f5228fac71e24c2612c347a34","unresolved":false,"context_lines":[{"line_number":2346,"context_line":"        dest_rp_uuid \u003d self._get_provider_uuid_by_host(dest_hostname)"},{"line_number":2347,"context_line":""},{"line_number":2348,"context_line":"        self.assertFlavorMatchesUsage(source_rp_uuid, self.flavor1)"},{"line_number":2349,"context_line":""},{"line_number":2350,"context_line":"        self.assertFlavorMatchesUsage(dest_rp_uuid, self.flavor1)"},{"line_number":2351,"context_line":""},{"line_number":2352,"context_line":"        self._check_allocation_during_evacuate("}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_f887720e","line":2349,"updated":"2019-05-23 11:58:02.000000000","message":"nit - whitespace","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"24aea3c6b37628428e5a74e33b2ba4283815e5fc","unresolved":false,"context_lines":[{"line_number":2346,"context_line":"        dest_rp_uuid \u003d self._get_provider_uuid_by_host(dest_hostname)"},{"line_number":2347,"context_line":""},{"line_number":2348,"context_line":"        self.assertFlavorMatchesUsage(source_rp_uuid, self.flavor1)"},{"line_number":2349,"context_line":""},{"line_number":2350,"context_line":"        self.assertFlavorMatchesUsage(dest_rp_uuid, self.flavor1)"},{"line_number":2351,"context_line":""},{"line_number":2352,"context_line":"        self._check_allocation_during_evacuate("}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_a9a16f48","line":2349,"in_reply_to":"bfb3d3c7_f887720e","updated":"2019-05-28 16:02:52.000000000","message":"N/A","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"71591e783b60473f5228fac71e24c2612c347a34","unresolved":false,"context_lines":[{"line_number":2362,"context_line":"        # This is bug 1724172 as the allocation is not removed from the source"},{"line_number":2363,"context_line":"        # host during the startup of the compute service."},{"line_number":2364,"context_line":"        self.assertFlavorMatchesUsage(source_rp_uuid, self.flavor1)"},{"line_number":2365,"context_line":""},{"line_number":2366,"context_line":"        self.assertFlavorMatchesUsage(dest_rp_uuid, self.flavor1)"},{"line_number":2367,"context_line":""},{"line_number":2368,"context_line":"        # This is bug 1724172"}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_98951643","line":2365,"updated":"2019-05-23 11:58:02.000000000","message":"nit - whitespace","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"24aea3c6b37628428e5a74e33b2ba4283815e5fc","unresolved":false,"context_lines":[{"line_number":2362,"context_line":"        # This is bug 1724172 as the allocation is not removed from the source"},{"line_number":2363,"context_line":"        # host during the startup of the compute service."},{"line_number":2364,"context_line":"        self.assertFlavorMatchesUsage(source_rp_uuid, self.flavor1)"},{"line_number":2365,"context_line":""},{"line_number":2366,"context_line":"        self.assertFlavorMatchesUsage(dest_rp_uuid, self.flavor1)"},{"line_number":2367,"context_line":""},{"line_number":2368,"context_line":"        # This is bug 1724172"}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_6987f7b6","line":2365,"in_reply_to":"bfb3d3c7_98951643","updated":"2019-05-28 16:02:52.000000000","message":"N/A","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"71591e783b60473f5228fac71e24c2612c347a34","unresolved":false,"context_lines":[{"line_number":2365,"context_line":""},{"line_number":2366,"context_line":"        self.assertFlavorMatchesUsage(dest_rp_uuid, self.flavor1)"},{"line_number":2367,"context_line":""},{"line_number":2368,"context_line":"        # This is bug 1724172"},{"line_number":2369,"context_line":"        allocations \u003d self._get_allocations_by_server_uuid(server[\u0027id\u0027])"},{"line_number":2370,"context_line":"        self.assertIn(source_rp_uuid, allocations)"},{"line_number":2371,"context_line":"        self.assertIn(dest_rp_uuid, allocations)"}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_38a50ab6","line":2368,"range":{"start_line":2368,"start_character":8,"end_line":2368,"end_character":29},"updated":"2019-05-23 11:58:02.000000000","message":"nit - could you just write the justification once above?","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"24aea3c6b37628428e5a74e33b2ba4283815e5fc","unresolved":false,"context_lines":[{"line_number":2365,"context_line":""},{"line_number":2366,"context_line":"        self.assertFlavorMatchesUsage(dest_rp_uuid, self.flavor1)"},{"line_number":2367,"context_line":""},{"line_number":2368,"context_line":"        # This is bug 1724172"},{"line_number":2369,"context_line":"        allocations \u003d self._get_allocations_by_server_uuid(server[\u0027id\u0027])"},{"line_number":2370,"context_line":"        self.assertIn(source_rp_uuid, allocations)"},{"line_number":2371,"context_line":"        self.assertIn(dest_rp_uuid, allocations)"}],"source_content_type":"text/x-python","patch_set":14,"id":"bfb3d3c7_0990bbec","line":2368,"range":{"start_line":2368,"start_character":8,"end_line":2368,"end_character":29},"in_reply_to":"bfb3d3c7_38a50ab6","updated":"2019-05-28 16:02:52.000000000","message":"N/A","commit_id":"1a56fe8f81b51fc1ee74a5577b6e5018a6936ffa"}]}