)]}'
{"nova/tests/unit/compute/test_compute.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9bfaa3a704b4bc64af8ee67a0bf27c53cb9d4fe1","unresolved":false,"context_lines":[{"line_number":645,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"        # Following methods will be called"},{"line_number":648,"context_line":"        # Note - time called two more times from Log"},{"line_number":649,"context_line":"        mock_last.return_value \u003d (0, 0)"},{"line_number":650,"context_line":"        mock_time.side_effect \u003d (10, 20, 21)"},{"line_number":651,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_698252ca","line":648,"range":{"start_line":648,"start_character":8,"end_line":648,"end_character":52},"updated":"2016-07-08 10:18:51.000000000","message":"nit: this comment makes less sense now that it\u0027s not between two calls","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"78d808c1453c367c27da1677c5e4031a5041c709","unresolved":false,"context_lines":[{"line_number":645,"context_line":"        ctxt \u003d context.get_admin_context()"},{"line_number":646,"context_line":""},{"line_number":647,"context_line":"        # Following methods will be called"},{"line_number":648,"context_line":"        # Note - time called two more times from Log"},{"line_number":649,"context_line":"        mock_last.return_value \u003d (0, 0)"},{"line_number":650,"context_line":"        mock_time.side_effect \u003d (10, 20, 21)"},{"line_number":651,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_6c4ca0c8","line":648,"range":{"start_line":648,"start_character":8,"end_line":648,"end_character":52},"in_reply_to":"1aa78d24_698252ca","updated":"2016-07-12 07:18:43.000000000","message":"Done","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"e3afbb4088d1a3415657f2a517d63fba46bc7815","unresolved":false,"context_lines":[{"line_number":661,"context_line":"        mock_get_counter.assert_called_once_with([])"},{"line_number":662,"context_line":"        mock_last.assert_called_once_with()"},{"line_number":663,"context_line":"        mock_get_host.assert_called_once_with(ctxt, \u0027fake-mini\u0027,"},{"line_number":664,"context_line":"                                              use_slave\u003dTrue)"},{"line_number":665,"context_line":""},{"line_number":666,"context_line":"    @mock.patch.object(objects.InstanceList, \u0027get_by_host\u0027)"},{"line_number":667,"context_line":"    @mock.patch.object(objects.BlockDeviceMappingList,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_7dc956ac","line":664,"updated":"2016-07-08 16:34:07.000000000","message":"Nit: not assert time is called multiple times any more.","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"78d808c1453c367c27da1677c5e4031a5041c709","unresolved":false,"context_lines":[{"line_number":661,"context_line":"        mock_get_counter.assert_called_once_with([])"},{"line_number":662,"context_line":"        mock_last.assert_called_once_with()"},{"line_number":663,"context_line":"        mock_get_host.assert_called_once_with(ctxt, \u0027fake-mini\u0027,"},{"line_number":664,"context_line":"                                              use_slave\u003dTrue)"},{"line_number":665,"context_line":""},{"line_number":666,"context_line":"    @mock.patch.object(objects.InstanceList, \u0027get_by_host\u0027)"},{"line_number":667,"context_line":"    @mock.patch.object(objects.BlockDeviceMappingList,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_4c49e4b6","line":664,"in_reply_to":"1aa78d24_7dc956ac","updated":"2016-07-12 07:18:43.000000000","message":"Done","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"9bfaa3a704b4bc64af8ee67a0bf27c53cb9d4fe1","unresolved":false,"context_lines":[{"line_number":689,"context_line":"    @mock.patch.object(utils, \u0027last_completed_audit_period\u0027)"},{"line_number":690,"context_line":"    @mock.patch.object(compute_manager.ComputeManager, \u0027_get_host_volume_bdms\u0027)"},{"line_number":691,"context_line":"    def test_poll_volume_usage_disabled(self, mock_get, mock_last):"},{"line_number":692,"context_line":"        # None of the mocks should be called."},{"line_number":693,"context_line":"        ctxt \u003d \u0027MockContext\u0027"},{"line_number":694,"context_line":""},{"line_number":695,"context_line":"        self.flags(volume_usage_poll_interval\u003d0)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_49a31650","line":692,"range":{"start_line":692,"start_character":8,"end_line":692,"end_character":45},"updated":"2016-07-08 10:18:51.000000000","message":"nit: this should also be removed","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"78d808c1453c367c27da1677c5e4031a5041c709","unresolved":false,"context_lines":[{"line_number":689,"context_line":"    @mock.patch.object(utils, \u0027last_completed_audit_period\u0027)"},{"line_number":690,"context_line":"    @mock.patch.object(compute_manager.ComputeManager, \u0027_get_host_volume_bdms\u0027)"},{"line_number":691,"context_line":"    def test_poll_volume_usage_disabled(self, mock_get, mock_last):"},{"line_number":692,"context_line":"        # None of the mocks should be called."},{"line_number":693,"context_line":"        ctxt \u003d \u0027MockContext\u0027"},{"line_number":694,"context_line":""},{"line_number":695,"context_line":"        self.flags(volume_usage_poll_interval\u003d0)"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_acc84823","line":692,"range":{"start_line":692,"start_character":8,"end_line":692,"end_character":45},"in_reply_to":"1aa78d24_49a31650","updated":"2016-07-12 07:18:43.000000000","message":"Done","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"e3afbb4088d1a3415657f2a517d63fba46bc7815","unresolved":false,"context_lines":[{"line_number":709,"context_line":"        self.flags(volume_usage_poll_interval\u003d10)"},{"line_number":710,"context_line":"        self.compute._poll_volume_usage(ctxt)"},{"line_number":711,"context_line":""},{"line_number":712,"context_line":"        mock_get_bdms.assert_called_once_with(ctxt, use_slave\u003dTrue)"},{"line_number":713,"context_line":""},{"line_number":714,"context_line":"    @mock.patch.object(compute_manager.ComputeManager, \u0027_get_host_volume_bdms\u0027)"},{"line_number":715,"context_line":"    @mock.patch.object(compute_manager.ComputeManager,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_5d13ba0d","line":712,"updated":"2016-07-08 16:34:07.000000000","message":"probably should assert mock_get_usage is not called.","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"},{"author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"change_message_id":"78d808c1453c367c27da1677c5e4031a5041c709","unresolved":false,"context_lines":[{"line_number":709,"context_line":"        self.flags(volume_usage_poll_interval\u003d10)"},{"line_number":710,"context_line":"        self.compute._poll_volume_usage(ctxt)"},{"line_number":711,"context_line":""},{"line_number":712,"context_line":"        mock_get_bdms.assert_called_once_with(ctxt, use_slave\u003dTrue)"},{"line_number":713,"context_line":""},{"line_number":714,"context_line":"    @mock.patch.object(compute_manager.ComputeManager, \u0027_get_host_volume_bdms\u0027)"},{"line_number":715,"context_line":"    @mock.patch.object(compute_manager.ComputeManager,"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_8ccd8c33","line":712,"in_reply_to":"1aa78d24_5d13ba0d","updated":"2016-07-12 07:18:43.000000000","message":"Done","commit_id":"6b1363add3b7b5d3a2e63c757d062cc8303e22c2"}]}