)]}'
{"nova/compute/api.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"26e446790162565273aa6fb81da9ce86ad454db0","unresolved":false,"context_lines":[{"line_number":2770,"context_line":"        bdms \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":2771,"context_line":"                context, instance.uuid)"},{"line_number":2772,"context_line":""},{"line_number":2773,"context_line":"        @wrap_instance_event(prefix\u003d\u0027api\u0027)"},{"line_number":2774,"context_line":"        def snapshot_instance(self, context, instance, bdms):"},{"line_number":2775,"context_line":"            mapping \u003d []"},{"line_number":2776,"context_line":"            for bdm in bdms:"}],"source_content_type":"text/x-python","patch_set":4,"id":"df87a7cf_b12d2796","line":2773,"updated":"2017-12-11 23:42:23.000000000","message":"You\u0027re doing this because we never cast to the compute for a volume-backed instance snapshot correct? So we need to create the action and complete the events within this same call.","commit_id":"d0336ee172157d7d0b88e52c75ae7f9e4fd313c7"}],"nova/tests/unit/compute/test_compute_api.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"27c176d0991cc78028fb83b11acb35c502993464","unresolved":false,"context_lines":[{"line_number":2823,"context_line":"                       fake_unquiesce_instance)"},{"line_number":2824,"context_line":"        fake_image.stub_out_image_service(self)"},{"line_number":2825,"context_line":""},{"line_number":2826,"context_line":"        with test.nested("},{"line_number":2827,"context_line":"                mock.patch.object(compute_api.API, \u0027_record_action_start\u0027),"},{"line_number":2828,"context_line":"                mock.patch.object(compute_utils, \u0027EventReporter\u0027)) as ("},{"line_number":2829,"context_line":"            mock_record, mock_event):"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf8cb3f7_e31e360b","line":2826,"range":{"start_line":2826,"start_character":8,"end_line":2826,"end_character":25},"updated":"2017-12-22 17:10:55.000000000","message":"I was pretty sure we\u0027re not supposed to use these any more as it\u0027s a Python \u003c 2.7 thing? You can do:\n\n  with mock.patch.object(...), \\\n          mock.patch.object(...):","commit_id":"d0336ee172157d7d0b88e52c75ae7f9e4fd313c7"},{"author":{"_account_id":7634,"name":"Takashi Natsume","email":"takanattie@gmail.com","username":"natsumet"},"change_message_id":"5a44dd653a2f269105886bad66faab8e32ee1325","unresolved":false,"context_lines":[{"line_number":2823,"context_line":"                       fake_unquiesce_instance)"},{"line_number":2824,"context_line":"        fake_image.stub_out_image_service(self)"},{"line_number":2825,"context_line":""},{"line_number":2826,"context_line":"        with test.nested("},{"line_number":2827,"context_line":"                mock.patch.object(compute_api.API, \u0027_record_action_start\u0027),"},{"line_number":2828,"context_line":"                mock.patch.object(compute_utils, \u0027EventReporter\u0027)) as ("},{"line_number":2829,"context_line":"            mock_record, mock_event):"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf8cb3f7_9e768306","line":2826,"range":{"start_line":2826,"start_character":8,"end_line":2826,"end_character":25},"in_reply_to":"bf8cb3f7_587025a6","updated":"2017-12-26 11:18:44.000000000","message":"\u0027contextlib.nested\u0027 (NOT \u0027test.nested\u0027) is deprecated in Python 2.7.\nThere is a bug report for it.\n\nRemove use of contextlib.nested\nhttps://bugs.launchpad.net/nova/+bug/1428424\n\n\u0027test.nested\u0027 is defined in nova/test.py.\n\nhttps://github.com/openstack/nova/blob/3bc4cb022f7dc782feccf0b6ab939754603392cf/nova/test.py#L78-L84","commit_id":"d0336ee172157d7d0b88e52c75ae7f9e4fd313c7"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"79448697fc8ce85b076b5d586bcb07c92f804ea4","unresolved":false,"context_lines":[{"line_number":2823,"context_line":"                       fake_unquiesce_instance)"},{"line_number":2824,"context_line":"        fake_image.stub_out_image_service(self)"},{"line_number":2825,"context_line":""},{"line_number":2826,"context_line":"        with test.nested("},{"line_number":2827,"context_line":"                mock.patch.object(compute_api.API, \u0027_record_action_start\u0027),"},{"line_number":2828,"context_line":"                mock.patch.object(compute_utils, \u0027EventReporter\u0027)) as ("},{"line_number":2829,"context_line":"            mock_record, mock_event):"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf8cb3f7_587025a6","line":2826,"range":{"start_line":2826,"start_character":8,"end_line":2826,"end_character":25},"in_reply_to":"bf8cb3f7_e31e360b","updated":"2017-12-23 14:19:32.000000000","message":"test.nested is a nova-specific thing, not from the stdlib.\n\nIf this was py27 only, this test would be failing in the py35 job.","commit_id":"d0336ee172157d7d0b88e52c75ae7f9e4fd313c7"}]}