)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"0d1f51a7d5a8b776b9f405e983136d9b8ff163a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a12c25af_a0602747","updated":"2021-10-27 16:06:23.000000000","message":"Looks reasonable to me","commit_id":"04fe235e556fca1d19731eb76a3efb3419859804"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"5f95c6455ae126c84e2fbf3983cbae523188d8b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"998006c8_7357aa35","updated":"2021-10-27 17:01:56.000000000","message":"run Pure Storage CI","commit_id":"04fe235e556fca1d19731eb76a3efb3419859804"},{"author":{"_account_id":12670,"name":"Helen Walsh","email":"helen.walsh@emc.com","username":"walshh2"},"change_message_id":"824ce6bde5a744981ec70d095d4086a22d460e0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c2432b17_4d091c43","updated":"2021-11-03 14:55:44.000000000","message":"LGTM","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"676ca7c1dd135ce9ceff4adf889c4c6f975a908d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"57c33dd9_fdf52fb4","updated":"2021-11-09 02:06:00.000000000","message":"This is a great start to this project!  Two minor things noted inline.","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"5be7a6d263fb92684e1cd83af29c9b7a91a37d28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"8bd1dcd7_0cd8c151","updated":"2022-04-06 14:31:42.000000000","message":"A few wording updates to address.","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"eb07904c3030571f47ac500de0ef876e0f9ad6a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"0d308ffb_0596d3b6","updated":"2022-04-06 14:39:58.000000000","message":"Apart from Brian\u0027s and Jay\u0027s Comments, patch looks good to me","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d53c90ea0f8931440ca99877afd54f00e7e620d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d7a60585_da438065","updated":"2022-04-06 14:41:01.000000000","message":"Content looks good, just two formatting suggestions.","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"58cdf3344f543226de2b1ce8f4613802fc4a053a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1280a255_9eb9eeca","updated":"2022-04-07 19:51:59.000000000","message":"Read through again, and except for what Rajat noted, LGTM.","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d324cfd506d823049100e477043f4c32521ba110","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6b8aef6f_13e14bf9","updated":"2022-04-08 10:13:01.000000000","message":"Sorry about that","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0137e0b84baef3d30b089ee35de6f178e1f5c0f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d896a20e_c9273568","updated":"2022-04-07 10:59:38.000000000","message":"The comments mentioned on the last PS were not completely addressed so marking them inline.","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e776558cf4cc176129862aa6e54d9071d330a622","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"2842a91f_4edb9416","updated":"2022-05-12 21:51:15.000000000","message":"Docs built fine.  Failure was a post_failure in cinder-plugin-ceph-tempest that had nothing to do with this patch.  Ninja-approving a documentation change.","commit_id":"e071ee263201d5ed0689e0755cf06f0a8568d16c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"984da984a3205a9215f567566a42ef8c32ac6ff7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e2ba6ed9_83bace37","updated":"2022-05-14 01:26:44.000000000","message":"So much for luck.  Looks like we need https://review.opendev.org/c/openstack/cinder/+/841782 to merge before rechecking.","commit_id":"e071ee263201d5ed0689e0755cf06f0a8568d16c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"001fb6ded8a7fe210b5a061b45197180d72fcd80","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"d4d32e1c_1701503a","updated":"2022-05-13 13:33:17.000000000","message":"cinder-plugin-ceph-tempest failed 40 tests -- mostly server faults and db connection problems.  I think the fix is increasing the swap size on the machine running the tests.  See https://review.opendev.org/q/topic:increase-swap\n\nMy current assessment is that the swap-size-based job failures happen about 35% of the time using the default swap size, so odds are that if we recheck, this job may pass.","commit_id":"e071ee263201d5ed0689e0755cf06f0a8568d16c"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4924a6389fa919a5881326de5a34b69b5693355e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"439b253f_a89d21b9","updated":"2022-08-10 10:45:29.000000000","message":"recheck","commit_id":"e071ee263201d5ed0689e0755cf06f0a8568d16c"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"bdde14facf880f732a2b79db5fd1250591417fd3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"a20b6356_6e3329e6","updated":"2022-05-13 13:34:37.000000000","message":"recheck","commit_id":"e071ee263201d5ed0689e0755cf06f0a8568d16c"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"0cef1f0dec8625987bc871b86a215aa6afb80b66","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"b9fb974a_93a64152","updated":"2022-08-11 09:05:45.000000000","message":"recheck - py39 failure on test_backup_google which is completely unrelated to this patch","commit_id":"e071ee263201d5ed0689e0755cf06f0a8568d16c"}],"cinder/interface/volume_driver.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"676ca7c1dd135ce9ceff4adf889c4c6f975a908d","unresolved":true,"context_lines":[{"line_number":239,"context_line":"                       dictionary compatibility interface it provides."},{"line_number":240,"context_line":"        :param image_location: Tuple with (``direct_url``, ``locations``) from"},{"line_number":241,"context_line":"                               the `image metadata fields."},{"line_number":242,"context_line":"                               \u003chttps://docs.openstack.org/api-ref/image/v2/index.html?expanded\u003dshow-image-detail#show-image-detail\u003e`_"},{"line_number":243,"context_line":"                               ``direct_url``, when present, is a string whose"},{"line_number":244,"context_line":"                               format depends on the image service\u0027s external"},{"line_number":245,"context_line":"                               storage in use."}],"source_content_type":"text/x-python","patch_set":3,"id":"05987e8e_081e8d23","line":242,"range":{"start_line":242,"start_character":113,"end_line":242,"end_character":131},"updated":"2021-11-09 02:06:00.000000000","message":"I think the anchor part should be just #show-image (same below)","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f5e8b92af23de250f10c753b08ca5070580d44de","unresolved":true,"context_lines":[{"line_number":239,"context_line":"                       dictionary compatibility interface it provides."},{"line_number":240,"context_line":"        :param image_location: Tuple with (``direct_url``, ``locations``) from"},{"line_number":241,"context_line":"                               the `image metadata fields."},{"line_number":242,"context_line":"                               \u003chttps://docs.openstack.org/api-ref/image/v2/index.html?expanded\u003dshow-image-detail#show-image-detail\u003e`_"},{"line_number":243,"context_line":"                               ``direct_url``, when present, is a string whose"},{"line_number":244,"context_line":"                               format depends on the image service\u0027s external"},{"line_number":245,"context_line":"                               storage in use."}],"source_content_type":"text/x-python","patch_set":3,"id":"372976c9_88fa7922","line":242,"range":{"start_line":242,"start_character":113,"end_line":242,"end_character":131},"in_reply_to":"05987e8e_081e8d23","updated":"2021-12-02 18:33:06.000000000","message":"I did this on purpose, because in my opinion since we are talking about the fields that\u0027s what we should point to, and the driver doesn\u0027t have to care/know about the method cinder has to call from the glance API to get this information.","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d67d1222d6830dd4253a3030fd941323efd865d9","unresolved":false,"context_lines":[{"line_number":239,"context_line":"                       dictionary compatibility interface it provides."},{"line_number":240,"context_line":"        :param image_location: Tuple with (``direct_url``, ``locations``) from"},{"line_number":241,"context_line":"                               the `image metadata fields."},{"line_number":242,"context_line":"                               \u003chttps://docs.openstack.org/api-ref/image/v2/index.html?expanded\u003dshow-image-detail#show-image-detail\u003e`_"},{"line_number":243,"context_line":"                               ``direct_url``, when present, is a string whose"},{"line_number":244,"context_line":"                               format depends on the image service\u0027s external"},{"line_number":245,"context_line":"                               storage in use."}],"source_content_type":"text/x-python","patch_set":3,"id":"1e441ef6_4a5ca91f","line":242,"range":{"start_line":242,"start_character":113,"end_line":242,"end_character":131},"in_reply_to":"3514cdab_0ad9e27b","updated":"2022-04-06 15:04:05.000000000","message":"Ack","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d53c90ea0f8931440ca99877afd54f00e7e620d8","unresolved":true,"context_lines":[{"line_number":239,"context_line":"                       dictionary compatibility interface it provides."},{"line_number":240,"context_line":"        :param image_location: Tuple with (``direct_url``, ``locations``) from"},{"line_number":241,"context_line":"                               the `image metadata fields."},{"line_number":242,"context_line":"                               \u003chttps://docs.openstack.org/api-ref/image/v2/index.html?expanded\u003dshow-image-detail#show-image-detail\u003e`_"},{"line_number":243,"context_line":"                               ``direct_url``, when present, is a string whose"},{"line_number":244,"context_line":"                               format depends on the image service\u0027s external"},{"line_number":245,"context_line":"                               storage in use."}],"source_content_type":"text/x-python","patch_set":3,"id":"3514cdab_0ad9e27b","line":242,"range":{"start_line":242,"start_character":113,"end_line":242,"end_character":131},"in_reply_to":"372976c9_88fa7922","updated":"2022-04-06 14:41:01.000000000","message":"OK, makes sense to me.","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"5be7a6d263fb92684e1cd83af29c9b7a91a37d28","unresolved":true,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        - When volumes are locally accessible and accessing them that way is"},{"line_number":232,"context_line":"          more efficient than going through the remote connection mechanism."},{"line_number":233,"context_line":"          For example in the GPFS if the cloning feature doesn\u0027t work it will"},{"line_number":234,"context_line":"          copy the file without using os-brick to connect to the volume."},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"        :param context: Security/policy info for the request."}],"source_content_type":"text/x-python","patch_set":6,"id":"15b4966f_dce69ae3","line":233,"range":{"start_line":233,"start_character":25,"end_line":233,"end_character":33},"updated":"2022-04-06 14:31:42.000000000","message":"the GPFS driver","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d67d1222d6830dd4253a3030fd941323efd865d9","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        - When volumes are locally accessible and accessing them that way is"},{"line_number":232,"context_line":"          more efficient than going through the remote connection mechanism."},{"line_number":233,"context_line":"          For example in the GPFS if the cloning feature doesn\u0027t work it will"},{"line_number":234,"context_line":"          copy the file without using os-brick to connect to the volume."},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"        :param context: Security/policy info for the request."}],"source_content_type":"text/x-python","patch_set":6,"id":"21272ec4_2961b38e","line":233,"range":{"start_line":233,"start_character":25,"end_line":233,"end_character":33},"in_reply_to":"15b4966f_dce69ae3","updated":"2022-04-06 15:04:05.000000000","message":"Done","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"}],"cinder/volume/driver.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"676ca7c1dd135ce9ceff4adf889c4c6f975a908d","unresolved":true,"context_lines":[{"line_number":1165,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":1166,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":1167,"context_line":""},{"line_number":1168,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":1169,"context_line":"        \"\"\""},{"line_number":1170,"context_line":"        return None, False"},{"line_number":1171,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"b47c861a_cf539a50","line":1168,"range":{"start_line":1168,"start_character":8,"end_line":1168,"end_character":69},"updated":"2021-11-09 02:06:00.000000000","message":"How about:\n\n  Refer to\n  :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`\n  for additional info.","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f5e8b92af23de250f10c753b08ca5070580d44de","unresolved":false,"context_lines":[{"line_number":1165,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":1166,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":1167,"context_line":""},{"line_number":1168,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":1169,"context_line":"        \"\"\""},{"line_number":1170,"context_line":"        return None, False"},{"line_number":1171,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"70861398_9e85963b","line":1168,"range":{"start_line":1168,"start_character":8,"end_line":1168,"end_character":69},"in_reply_to":"b47c861a_cf539a50","updated":"2021-12-02 18:33:06.000000000","message":"Done","commit_id":"0f3fcbc6e19c88e6c714c3239a596c79a15c4152"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"fd15e5811d328f138c2ca6f455ad5658eda107bf","unresolved":true,"context_lines":[{"line_number":1158,"context_line":""},{"line_number":1159,"context_line":"        Refer to"},{"line_number":1160,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":1161,"context_line":"        for addition info."},{"line_number":1162,"context_line":"        \"\"\""},{"line_number":1163,"context_line":"        return None, False"},{"line_number":1164,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5308ba41_a65bbaee","line":1161,"range":{"start_line":1161,"start_character":12,"end_line":1161,"end_character":25},"updated":"2022-04-06 14:26:10.000000000","message":"Might as well say \"additional information\" (same at line 1995).","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d67d1222d6830dd4253a3030fd941323efd865d9","unresolved":false,"context_lines":[{"line_number":1158,"context_line":""},{"line_number":1159,"context_line":"        Refer to"},{"line_number":1160,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":1161,"context_line":"        for addition info."},{"line_number":1162,"context_line":"        \"\"\""},{"line_number":1163,"context_line":"        return None, False"},{"line_number":1164,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"2a23b853_65d6c06a","line":1161,"range":{"start_line":1161,"start_character":12,"end_line":1161,"end_character":25},"in_reply_to":"5308ba41_a65bbaee","updated":"2022-04-06 15:04:05.000000000","message":"Done","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d53c90ea0f8931440ca99877afd54f00e7e620d8","unresolved":true,"context_lines":[{"line_number":1992,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":1993,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":1994,"context_line":""},{"line_number":1995,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":1996,"context_line":"        \"\"\""},{"line_number":1997,"context_line":"        return None, False"},{"line_number":1998,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"60535c45_4da1d0c4","line":1995,"range":{"start_line":1995,"start_character":17,"end_line":1995,"end_character":50},"updated":"2022-04-06 14:41:01.000000000","message":"If you use\n\n  :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`\n\nthen the docs generate a hyperlink directly to the additional info.","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d67d1222d6830dd4253a3030fd941323efd865d9","unresolved":false,"context_lines":[{"line_number":1992,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":1993,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":1994,"context_line":""},{"line_number":1995,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":1996,"context_line":"        \"\"\""},{"line_number":1997,"context_line":"        return None, False"},{"line_number":1998,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"703c7dcb_f687d13c","line":1995,"range":{"start_line":1995,"start_character":17,"end_line":1995,"end_character":50},"in_reply_to":"60535c45_4da1d0c4","updated":"2022-04-06 15:04:05.000000000","message":"Nice!","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"d53c90ea0f8931440ca99877afd54f00e7e620d8","unresolved":true,"context_lines":[{"line_number":2540,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":2541,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":2542,"context_line":""},{"line_number":2543,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":2544,"context_line":"        \"\"\""},{"line_number":2545,"context_line":"        return None, False"},{"line_number":2546,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9abe7584_f8715641","line":2543,"updated":"2022-04-06 14:41:01.000000000","message":"Also, it would be good to use\n\n  :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`\n\ninstead of the path.","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"5be7a6d263fb92684e1cd83af29c9b7a91a37d28","unresolved":true,"context_lines":[{"line_number":2540,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":2541,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":2542,"context_line":""},{"line_number":2543,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":2544,"context_line":"        \"\"\""},{"line_number":2545,"context_line":"        return None, False"},{"line_number":2546,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"686f6217_382b28b5","line":2543,"range":{"start_line":2543,"start_character":55,"end_line":2543,"end_character":69},"updated":"2022-04-06 14:31:42.000000000","message":"same comment here","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d67d1222d6830dd4253a3030fd941323efd865d9","unresolved":false,"context_lines":[{"line_number":2540,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":2541,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":2542,"context_line":""},{"line_number":2543,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":2544,"context_line":"        \"\"\""},{"line_number":2545,"context_line":"        return None, False"},{"line_number":2546,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"8cb62d1b_00c319b4","line":2543,"range":{"start_line":2543,"start_character":55,"end_line":2543,"end_character":69},"in_reply_to":"686f6217_382b28b5","updated":"2022-04-06 15:04:05.000000000","message":"Done","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d67d1222d6830dd4253a3030fd941323efd865d9","unresolved":false,"context_lines":[{"line_number":2540,"context_line":"                    image_location, image_meta, image_service):"},{"line_number":2541,"context_line":"        \"\"\"Create a volume efficiently from an existing image."},{"line_number":2542,"context_line":""},{"line_number":2543,"context_line":"        Refer to cinder/interface/volume_driver.py for addition info."},{"line_number":2544,"context_line":"        \"\"\""},{"line_number":2545,"context_line":"        return None, False"},{"line_number":2546,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"72898432_aedd6f43","line":2543,"in_reply_to":"9abe7584_f8715641","updated":"2022-04-06 15:04:05.000000000","message":"Done","commit_id":"08b75f26e6484c4461ea1e1be0cf081687665e25"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0137e0b84baef3d30b089ee35de6f178e1f5c0f2","unresolved":true,"context_lines":[{"line_number":1158,"context_line":""},{"line_number":1159,"context_line":"        Refer to"},{"line_number":1160,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":1161,"context_line":"        for addition information."},{"line_number":1162,"context_line":"        \"\"\""},{"line_number":1163,"context_line":"        return None, False"},{"line_number":1164,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"2db1baf6_1ba006f0","line":1161,"range":{"start_line":1161,"start_character":12,"end_line":1161,"end_character":20},"updated":"2022-04-07 10:59:38.000000000","message":"nit: additional","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d324cfd506d823049100e477043f4c32521ba110","unresolved":false,"context_lines":[{"line_number":1158,"context_line":""},{"line_number":1159,"context_line":"        Refer to"},{"line_number":1160,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":1161,"context_line":"        for addition information."},{"line_number":1162,"context_line":"        \"\"\""},{"line_number":1163,"context_line":"        return None, False"},{"line_number":1164,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"9dac638b_2d426af9","line":1161,"range":{"start_line":1161,"start_character":12,"end_line":1161,"end_character":20},"in_reply_to":"2db1baf6_1ba006f0","updated":"2022-04-08 10:13:01.000000000","message":"/me facepalms","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0137e0b84baef3d30b089ee35de6f178e1f5c0f2","unresolved":true,"context_lines":[{"line_number":1994,"context_line":""},{"line_number":1995,"context_line":"        Refer to"},{"line_number":1996,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":1997,"context_line":"        for addition info."},{"line_number":1998,"context_line":"        \"\"\""},{"line_number":1999,"context_line":"        return None, False"},{"line_number":2000,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"3d036b4f_5c1f3e1b","line":1997,"range":{"start_line":1997,"start_character":12,"end_line":1997,"end_character":25},"updated":"2022-04-07 10:59:38.000000000","message":"nit: additional information","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d324cfd506d823049100e477043f4c32521ba110","unresolved":false,"context_lines":[{"line_number":1994,"context_line":""},{"line_number":1995,"context_line":"        Refer to"},{"line_number":1996,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":1997,"context_line":"        for addition info."},{"line_number":1998,"context_line":"        \"\"\""},{"line_number":1999,"context_line":"        return None, False"},{"line_number":2000,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"7ad65d13_6e8dba88","line":1997,"range":{"start_line":1997,"start_character":12,"end_line":1997,"end_character":25},"in_reply_to":"3d036b4f_5c1f3e1b","updated":"2022-04-08 10:13:01.000000000","message":"Done","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0137e0b84baef3d30b089ee35de6f178e1f5c0f2","unresolved":true,"context_lines":[{"line_number":2544,"context_line":""},{"line_number":2545,"context_line":"        Refer to"},{"line_number":2546,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":2547,"context_line":"        for addition information."},{"line_number":2548,"context_line":"        \"\"\""},{"line_number":2549,"context_line":"        return None, False"},{"line_number":2550,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"1ce76673_600c0305","line":2547,"range":{"start_line":2547,"start_character":12,"end_line":2547,"end_character":20},"updated":"2022-04-07 10:59:38.000000000","message":"nit: additional","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"d324cfd506d823049100e477043f4c32521ba110","unresolved":false,"context_lines":[{"line_number":2544,"context_line":""},{"line_number":2545,"context_line":"        Refer to"},{"line_number":2546,"context_line":"        :obj:`cinder.interface.volume_driver.VolumeDriverCore.clone_image`"},{"line_number":2547,"context_line":"        for addition information."},{"line_number":2548,"context_line":"        \"\"\""},{"line_number":2549,"context_line":"        return None, False"},{"line_number":2550,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"95c32c0f_b12df48f","line":2547,"range":{"start_line":2547,"start_character":12,"end_line":2547,"end_character":20},"in_reply_to":"1ce76673_600c0305","updated":"2022-04-08 10:13:01.000000000","message":"Done","commit_id":"16a5fce1dbc98ef36dab635f347f93cfe5d605bb"}]}