)]}'
{"nova/tests/functional/regressions/test_bug_1732947.py":[{"author":{"_account_id":8768,"name":"Chris Friesen","email":"chris.friesen@windriver.com","username":"cbf123"},"change_message_id":"33e9bb5f473099c41cab78c266dabf576583fa28","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        self.useFixture(nova_fixtures.CinderFixture(self))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def test_volume_backed_rebuild_same_image(self):"},{"line_number":37,"context_line":"        \"\"\"Rebuilds a server with a different image which will run it through"},{"line_number":38,"context_line":"        the scheduler to validate the image is still OK with the compute host"},{"line_number":39,"context_line":"        that the instance is running on."},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1f485f77_f333cea9","line":37,"range":{"start_line":37,"start_character":36,"end_line":37,"end_character":45},"updated":"2017-11-17 22:02:16.000000000","message":"should this be \"same\" to match the title of the testcase?","commit_id":"25354d5826cc3b0bb6166369fb7506bdf7f179a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"bb9912a6ed7bb061c0fda26772417e8bc9cb0892","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        self.useFixture(nova_fixtures.CinderFixture(self))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def test_volume_backed_rebuild_same_image(self):"},{"line_number":37,"context_line":"        \"\"\"Rebuilds a server with a different image which will run it through"},{"line_number":38,"context_line":"        the scheduler to validate the image is still OK with the compute host"},{"line_number":39,"context_line":"        that the instance is running on."},{"line_number":40,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1f485f77_13a42ab9","line":37,"range":{"start_line":37,"start_character":36,"end_line":37,"end_character":45},"in_reply_to":"1f485f77_f333cea9","updated":"2017-11-17 22:05:47.000000000","message":"Yes, copy/paste error.","commit_id":"25354d5826cc3b0bb6166369fb7506bdf7f179a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2f9fe93f69021f982083c5d93db75d24f2ae9b23","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        # in the server response."},{"line_number":58,"context_line":"        self.assertEqual(\u0027\u0027, server[\u0027image\u0027])"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        # Disable the single compute service that the instance is running on."},{"line_number":61,"context_line":"        # Before the bug is fixed, this will make the rebuild fail."},{"line_number":62,"context_line":"        self.api_fixture.admin_api.api_put(\u0027os-services/disable\u0027,"},{"line_number":63,"context_line":"                                           {\u0027host\u0027: self.compute.host,"},{"line_number":64,"context_line":"                                            \u0027binary\u0027: \u0027nova-compute\u0027})"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"        # Now rebuild the server with the same image that was used to create"},{"line_number":67,"context_line":"        # our fake volume."}],"source_content_type":"text/x-python","patch_set":2,"id":"ff82abbf_b180e7f6","line":64,"range":{"start_line":60,"start_character":8,"end_line":64,"end_character":70},"updated":"2017-11-28 15:08:06.000000000","message":"Actually this might not work now that https://review.openstack.org/#/c/521186/ is merged.","commit_id":"24bcdcfdb9c6d11bb0db319dae687a103fa811a2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"926e8cd101df808f78bbdef10a1cc116c1b9d852","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        enabled_filters \u003d CONF.filter_scheduler.enabled_filters"},{"line_number":43,"context_line":"        enabled_filters.append(\u0027IsolatedHostsFilter\u0027)"},{"line_number":44,"context_line":"        self.flags(enabled_filters\u003denabled_filters, group\u003d\u0027filter_scheduler\u0027)"},{"line_number":45,"context_line":"        return self.start_service(\u0027scheduler\u0027)"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def test_volume_backed_rebuild_same_image(self):"},{"line_number":48,"context_line":"        # First create our server as normal."}],"source_content_type":"text/x-python","patch_set":4,"id":"ff82abbf_e971660e","line":45,"updated":"2017-11-30 15:23:49.000000000","message":"Note to reviewers... if you\u0027re wondering where this method is called, it\u0027s overriding the base class\u0027s method:\n\nhttps://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L92","commit_id":"284f113443d1d49d7f8846613162a89768790896"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"926e8cd101df808f78bbdef10a1cc116c1b9d852","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        # should actually come back to the API and result in a 500 error."},{"line_number":86,"context_line":"        # Normally the user would get a 202 response because nova-api RPC casts"},{"line_number":87,"context_line":"        # to nova-conductor which RPC calls the scheduler which raises the"},{"line_number":88,"context_line":"        # NoValidHost."},{"line_number":89,"context_line":"        self.api.api_post(\u0027/servers/%s/action\u0027 % server[\u0027id\u0027],"},{"line_number":90,"context_line":"                          rebuild_req_body, check_response_status\u003d[500])"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff82abbf_496bb2f0","line":88,"updated":"2017-11-30 15:23:49.000000000","message":"did you want to put a note in here about what the expected result of this should be once bug 1732947 is addressed?","commit_id":"284f113443d1d49d7f8846613162a89768790896"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"57eb910c6d9713623a61d5d466dfcec7396ac817","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        # should actually come back to the API and result in a 500 error."},{"line_number":86,"context_line":"        # Normally the user would get a 202 response because nova-api RPC casts"},{"line_number":87,"context_line":"        # to nova-conductor which RPC calls the scheduler which raises the"},{"line_number":88,"context_line":"        # NoValidHost."},{"line_number":89,"context_line":"        self.api.api_post(\u0027/servers/%s/action\u0027 % server[\u0027id\u0027],"},{"line_number":90,"context_line":"                          rebuild_req_body, check_response_status\u003d[500])"}],"source_content_type":"text/x-python","patch_set":4,"id":"df87a7cf_4dd86a52","line":88,"in_reply_to":"ff82abbf_496bb2f0","updated":"2017-12-04 21:50:05.000000000","message":"I will if I have to update this change for some reason.","commit_id":"284f113443d1d49d7f8846613162a89768790896"}]}