)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"bad50270d3db6ab66c17aaa32752ddb903a7b2b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f8129441_f86865f0","updated":"2022-10-05 01:41:36.000000000","message":"The comparioson function that returns false if A and B are the same but empty is a little worrisome, but acceptable.","commit_id":"06ef324a9b856b9bd8a55714c2e435cc46e2e19e"},{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"c28532ed9837133fd600b7681ea7e47b6bc5d36d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b4a2a878_7ed7bfca","updated":"2022-11-25 21:14:03.000000000","message":"Looks good to me, thank you!","commit_id":"89daaa634e9ae98939d68c6672bdc5e70cefaa20"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"02402217753585b23f08db407dd7da17d79bd48c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"0fa6b699_8c3853f3","updated":"2023-01-30 12:03:19.000000000","message":"looks good to me, thanks for adding this!","commit_id":"e25ee86224f45f9532dde611fa19d0143e2ed765"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"9e6e7a8996f83a14e7bb21a5f62c473e7d89fe34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"cd04e306_08d21aab","updated":"2023-02-20 15:19:19.000000000","message":"Few comments but nothign worth holding for. LGTM.","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":33609,"name":"XuQi","display_name":"Inori","email":"xuq.fnstxz@fujitsu.com","username":"inori"},"change_message_id":"fa29fa75449cca44315c0d481451e60049c52b72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"1cf0a24c_48707acb","updated":"2023-02-17 03:40:27.000000000","message":"Looks good to me.","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"5518ac910bfee0abf566e2536832efcfdd769a4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"c43328f0_c90799e8","updated":"2023-02-22 14:29:11.000000000","message":"Since today is the deadline for feature-freeze, i think Rajat\u0027s comments can be address in a follow-up. \nLGTM","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"515463cbe8e61553c50b89308ccdf4ec9a6bdd0a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"5987d992_c7afeabd","updated":"2023-02-15 13:23:52.000000000","message":"recheck","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ff9044448d96661ac76f0f4203f4e40627d9c4a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"3df5660f_993351d3","updated":"2023-02-16 15:42:55.000000000","message":"recheck - devstack-plugin-nfs-tempest-full failure is unrelated since it\u0027s the NFS driver and we are only touching LVM","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f1568337c7e852028c68c6ad5e4ba87402abe8e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"bf4975ae_328266e7","updated":"2023-02-17 13:55:00.000000000","message":"recheck - lvm-lio-barbican error is due to Bad router request: Router already has a port on subnet","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c43265f07cbf7fc2e27c53156ab451a17edd2ae5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"752b90d9_73fdc30b","updated":"2023-02-17 11:36:25.000000000","message":"recheck - server fault and server unavaiable errors","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"}],"cinder/volume/targets/driver.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"9e6e7a8996f83a14e7bb21a5f62c473e7d89fe34","unresolved":true,"context_lines":[{"line_number":74,"context_line":"        \"\"\"Whether 2 connectors belong to the same host or not."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        This is used for multi attach volumes, to be able to know when there"},{"line_number":77,"context_line":"        are no more attachments on a given host."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        This is the generic implementation, but specific targets may overwrite"},{"line_number":80,"context_line":"        it. For example iSCSI would check the the \"initiator\" key instead, and"}],"source_content_type":"text/x-python","patch_set":10,"id":"b5d712ef_f971bc1a","line":77,"range":{"start_line":77,"start_character":12,"end_line":77,"end_character":31},"updated":"2023-02-20 15:19:19.000000000","message":"no more than 1 attachment?","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"9e6e7a8996f83a14e7bb21a5f62c473e7d89fe34","unresolved":true,"context_lines":[{"line_number":77,"context_line":"        are no more attachments on a given host."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        This is the generic implementation, but specific targets may overwrite"},{"line_number":80,"context_line":"        it. For example iSCSI would check the the \"initiator\" key instead, and"},{"line_number":81,"context_line":"        NVMe-oF would check the \"nqn\" key."},{"line_number":82,"context_line":"        \"\"\""},{"line_number":83,"context_line":"        a_host \u003d A.get(\u0027host\u0027)"}],"source_content_type":"text/x-python","patch_set":10,"id":"eebb13dc_886db751","line":80,"range":{"start_line":80,"start_character":46,"end_line":80,"end_character":49},"updated":"2023-02-20 15:19:19.000000000","message":"nit: repeated \u0027the\u0027","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"9e6e7a8996f83a14e7bb21a5f62c473e7d89fe34","unresolved":true,"context_lines":[{"line_number":81,"context_line":"        NVMe-oF would check the \"nqn\" key."},{"line_number":82,"context_line":"        \"\"\""},{"line_number":83,"context_line":"        a_host \u003d A.get(\u0027host\u0027)"},{"line_number":84,"context_line":"        return a_host and (a_host \u003d\u003d B.get(\u0027host\u0027))"}],"source_content_type":"text/x-python","patch_set":10,"id":"cb24d68b_53650748","line":84,"range":{"start_line":84,"start_character":15,"end_line":84,"end_character":21},"updated":"2023-02-20 15:19:19.000000000","message":"is this done to handle the case where \u0027host\u0027 key doesn\u0027t exist in both connectors and None\u003d\u003dNone would return True?","commit_id":"daa803b8ee1adcadc7d0e797bd04b287a7df5946"}]}