)]}'
{"cinder/tests/test_rbd.py":[{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"498763b6ecc1dce25270f498eb65d4effbac8bf3","unresolved":false,"context_lines":[{"line_number":602,"context_line":""},{"line_number":603,"context_line":"        # See tests.image.fake for image types."},{"line_number":604,"context_line":"        if raw:"},{"line_number":605,"context_line":"            expected_calls \u003d [\u0027clone_image\u0027]"},{"line_number":606,"context_line":"            image_id \u003d \u0027155d900f-4e14-4e4c-a73d-069cbf4541e6\u0027"},{"line_number":607,"context_line":"        else:"},{"line_number":608,"context_line":"            expected_calls \u003d [\u0027clone_image\u0027, \u0027create_volume\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F9MOw%3D","line":605,"updated":"2013-12-14 21:25:14.000000000","message":"expected_calls appears to never be used.","commit_id":"cf78dcb4707c233cb89ddf5bdc2dfffe66a8ddee"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"498763b6ecc1dce25270f498eb65d4effbac8bf3","unresolved":false,"context_lines":[{"line_number":664,"context_line":"        self._create_volume_from_image(\u0027available\u0027, raw\u003dFalse)"},{"line_number":665,"context_line":""},{"line_number":666,"context_line":"    def test_create_vol_from_image_status_error(self):"},{"line_number":667,"context_line":"        \"\"\"Clone raw image with failure then verify volume is in error"},{"line_number":668,"context_line":"        state."},{"line_number":669,"context_line":"        \"\"\""},{"line_number":670,"context_line":"        self._create_volume_from_image(\u0027error\u0027, raw\u003dTrue, clone_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F9MO8%3D","line":667,"updated":"2013-12-14 21:25:14.000000000","message":"This should be a single line or fixed to a multi-line doc string format: http://docs.openstack.org/developer/hacking/#docstrings","commit_id":"cf78dcb4707c233cb89ddf5bdc2dfffe66a8ddee"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"498763b6ecc1dce25270f498eb65d4effbac8bf3","unresolved":false,"context_lines":[{"line_number":674,"context_line":""},{"line_number":675,"context_line":"        # Test uncloneable case(s)"},{"line_number":676,"context_line":"        with mock.patch.object(driver, \u0027_is_cloneable\u0027,"},{"line_number":677,"context_line":"                               lambda *args: False) as mock_is_cloneable:"},{"line_number":678,"context_line":"            image_loc \u003d (mock.Mock(), mock.Mock())"},{"line_number":679,"context_line":"            actual \u003d driver.clone_image(mock.Mock(), image_loc,"},{"line_number":680,"context_line":"                                        mock.Mock(), {})"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F9MO0%3D","line":677,"updated":"2013-12-14 21:25:14.000000000","message":"mock_is_cloneable is never used.","commit_id":"cf78dcb4707c233cb89ddf5bdc2dfffe66a8ddee"},{"author":{"_account_id":170,"name":"Mike Perez","email":"thingee@gmail.com","username":"thingee"},"change_message_id":"498763b6ecc1dce25270f498eb65d4effbac8bf3","unresolved":false,"context_lines":[{"line_number":682,"context_line":""},{"line_number":683,"context_line":"        self.assertEqual(({}, False),"},{"line_number":684,"context_line":"                         driver.clone_image(object(), None, None, {}))"},{"line_number":685,"context_line":""},{"line_number":686,"context_line":"        # Test success case(s)"},{"line_number":687,"context_line":"        expected \u003d ({\u0027provider_location\u0027: None}, True)"},{"line_number":688,"context_line":"        mpo \u003d mock.patch.object"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F9MOY%3D","line":685,"updated":"2013-12-14 21:25:14.000000000","message":"It looks like 686 to 699 is the same as test case test_clone_success. We should just make this test case test for failure.","commit_id":"cf78dcb4707c233cb89ddf5bdc2dfffe66a8ddee"}],"cinder/volume/drivers/lvm.py":[{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"ef1ef987bc83396959e71d6e27823e0acf571dbe","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"class LVMVolumeDriver(driver.VolumeDriver):"},{"line_number":61,"context_line":"    \"\"\"Executes commands relating to Volumes.\"\"\""},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    VERSION \u003d \u00272.0.0\u0027"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    def __init__(self, vg_obj\u003dNone, *args, **kwargs):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F98m8%3D","line":62,"updated":"2013-12-04 06:03:16.000000000","message":"Minor version bump?","commit_id":"db6bb92e7cde78d034fa7e7bac7e64c3061ec81d"}],"cinder/volume/flows/create_volume/__init__.py":[{"author":{"_account_id":7689,"name":"Bill Owen","email":"billowen@us.ibm.com","username":"billowen"},"change_message_id":"4660a4bb680987947e881af1bba204be4b306e5a","unresolved":false,"context_lines":[{"line_number":1441,"context_line":"        # dict containing provider_location for cloned volume"},{"line_number":1442,"context_line":"        # and clone status."},{"line_number":1443,"context_line":"        model_update, cloned \u003d self.driver.clone_image("},{"line_number":1444,"context_line":"            volume_ref, image_location, image_id, image_meta)"},{"line_number":1445,"context_line":"        if not cloned:"},{"line_number":1446,"context_line":"            # TODO(harlowja): what needs to be rolled back in the clone if this"},{"line_number":1447,"context_line":"            # volume create fails?? Likely this should be a subflow or broken"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F9988%3D","line":1444,"updated":"2013-12-03 22:22:45.000000000","message":"Can driver determine format of source image from image_location instead of using image_meta?  In gpfs driver we call image_utils.qemu_img_info on image_location.","commit_id":"f147be77991581b9cf83f90fe26c2d8e795cec10"}]}