)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"6b6bfa568992e94b8def801021e0c6ae41729675","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Gorka Eguileor \u003cgeguileo@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-05-24 15:13:23 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing exted_target driver method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The base target driver class is missing the extennd_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0a52fc72_ccf3c64e","line":7,"range":{"start_line":7,"start_character":12,"end_line":7,"end_character":24},"updated":"2022-05-27 15:10:56.000000000","message":"nit extend_target","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"7683f6911ed2563bd37a3adbeeaa382580f0c95b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Gorka Eguileor \u003cgeguileo@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-05-24 15:13:23 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing exted_target driver method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The base target driver class is missing the extennd_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2e50f695_c3b36f95","line":7,"range":{"start_line":7,"start_character":12,"end_line":7,"end_character":24},"in_reply_to":"0a52fc72_ccf3c64e","updated":"2022-06-10 14:30:39.000000000","message":"Done","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"6b6bfa568992e94b8def801021e0c6ae41729675","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing exted_target driver method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The base target driver class is missing the extennd_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since it was missing there some drivers, such as the nvmeof base class,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b9a3705e_edee4821","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":58},"updated":"2022-05-27 15:10:56.000000000","message":"nit extend_target -- the missing \u0027n\u0027 on the previous nit is found here haha","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"29ede41bf0a468f7214820702bbab103fb47b7df","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing exted_target driver method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The base target driver class is missing the extennd_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since it was missing there some drivers, such as the nvmeof base class,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2b3b9422_ff065307","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":58},"updated":"2022-05-27 15:09:37.000000000","message":"nit: extend_target","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"7683f6911ed2563bd37a3adbeeaa382580f0c95b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing exted_target driver method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The base target driver class is missing the extennd_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since it was missing there some drivers, such as the nvmeof base class,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"126c9759_4a8710bb","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":58},"in_reply_to":"2b3b9422_ff065307","updated":"2022-06-10 14:30:39.000000000","message":"Done","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"7683f6911ed2563bd37a3adbeeaa382580f0c95b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add missing exted_target driver method"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The base target driver class is missing the extennd_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since it was missing there some drivers, such as the nvmeof base class,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"06710af6_58fe1fff","line":9,"range":{"start_line":9,"start_character":44,"end_line":9,"end_character":58},"in_reply_to":"b9a3705e_edee4821","updated":"2022-06-10 14:30:39.000000000","message":"rofl","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4ccfea7b0315841a80288e795c2824ce11d1739d","unresolved":true,"context_lines":[{"line_number":9,"context_line":"The base target driver class is missing the extend_target method which"},{"line_number":10,"context_line":"is called from the LVM driver."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Since it was missing there some drivers, such as the nvmeof base class,"},{"line_number":13,"context_line":"also missed it when they were implemented."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This patch adds a base implementation that does nothing, since this is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"41c52698_2ffbb62f","line":12,"range":{"start_line":12,"start_character":26,"end_line":12,"end_character":27},"updated":"2023-02-21 09:59:02.000000000","message":"nit: comma","commit_id":"9337bc8b8de633fe1407e46fe22a73eff552290b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"0e04aadb17687feb9ec73a1ca4aea2e0e2a984c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"90ef551a_bca443a5","updated":"2022-05-27 15:02:13.000000000","message":"Most drives will not use it, but as Gorka said, it is a good exercise, and I think it is good. Zuul passed.","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"752d27bf0a4467f4eaf01a3e57258db5da344d69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"801edd29_4316c34d","updated":"2022-05-27 15:18:15.000000000","message":"OK, I\u0027ll be the bad guy here since no one else seems to want to do it.  Change looks fine, but it would be good to have the name of the method spelled correctly in the subject line of the commit message so there\u0027s no confusion for people who use \u0027git log --oneline\u0027.\n\nLuckily, this change looks like it\u0027s the last one in the chain, so fixing this won\u0027t make you have to rebase the rest of the patches.  (Otherwise, i would feel very bad.)","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"cb24dcc55ed3974f0cfd9f0a5f842f77d70b061f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"68384000_dd7f550c","updated":"2022-05-30 02:52:39.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"5783f93c55cb671b6521821ced6722e69700a262","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a3f7bd48_5cb6070a","updated":"2022-05-30 07:11:26.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"42cc4064483e6dda5eb3d7f052575db9ff198a65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"30d83b18_fdeef9a7","updated":"2022-07-11 09:33:23.000000000","message":"Looks good","commit_id":"1d86b032e1209334675c3c58589345ce23b42bf8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b2b045f2aac05f715fef04591c6d18fa1180a340","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f42c5de_8e617138","updated":"2022-07-15 15:20:32.000000000","message":"Looks good to me!","commit_id":"1d86b032e1209334675c3c58589345ce23b42bf8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"54523f98df52287be297849fd4b1ee09e6cbcaab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b9a7a2f2_fde2066c","updated":"2022-07-01 16:20:35.000000000","message":"looks good to me. Code is quite easy to read this time.","commit_id":"1d86b032e1209334675c3c58589345ce23b42bf8"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"940615e61f3c356937fdffcd5e990962631d38c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dc230b1b_2fca3fe4","updated":"2022-07-15 15:57:45.000000000","message":"patch updated and lgtm!","commit_id":"1d86b032e1209334675c3c58589345ce23b42bf8"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"ce1d8c21efa6dd3b912493ddcb3f60eeec54c8f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f9dfb84d_0ae76dac","updated":"2022-06-15 09:46:46.000000000","message":"recheck : Any failure is unrelated, since this patch doesn\u0027t have any real impact on how Cinder works.","commit_id":"1d86b032e1209334675c3c58589345ce23b42bf8"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c31bcb8f276104b9e6493966db6e7deace285340","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ae2a4f47_b644db28","updated":"2023-02-16 15:46:40.000000000","message":"recheck - cinder-plugin-ceph-tempest is unrelated to this change","commit_id":"9337bc8b8de633fe1407e46fe22a73eff552290b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4ccfea7b0315841a80288e795c2824ce11d1739d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"43b89592_040c1cd7","updated":"2023-02-21 09:59:02.000000000","message":"recheck grenade failure looks unrelated","commit_id":"9337bc8b8de633fe1407e46fe22a73eff552290b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a1f6a79e7dc97e91786ff1d5ff3653656477279d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e8f028d1_968b5e44","updated":"2023-02-22 12:47:51.000000000","message":"LGTM","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"2d032c1137c52dabae24e08ebd572b8cc531b069","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"317a7045_fa3ddd75","updated":"2023-02-27 13:51:18.000000000","message":"The mirror in the iad3 zone is still not working, and it has just been disabled for the time being by https://review.opendev.org/c/openstack/project-config/+/875488/","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"531169036eeac4ac4a6285e0fa41b6e3a74dbc42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d034f9b7_d250277b","updated":"2023-02-22 21:59:25.000000000","message":"recheck","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"cc48befa26f19c877abf69e1907e0ad1885ab3e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f1665ad4_de546243","updated":"2023-02-27 13:51:46.000000000","message":"recheck\n\nThe mirror in the iad3 zone is still not working, and it has just been disabled for the time being by https://review.opendev.org/c/openstack/project-config/+/875488/","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"c7367ecdeca25ba313a1c581bf75daca588eec20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c8e86064_c69c2b1d","updated":"2023-02-27 10:09:03.000000000","message":"recheck\n\ninfra issue: 3 jobs failed due to failures which reaching a specific mirror (mirror.iad3.inmotion.opendev.org),","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f33d1965e006d3caab54d2360b39607c69c8283f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"200a5840_40ed59de","updated":"2023-02-22 09:36:54.000000000","message":"recheck - slow-py3 job failure is on a network test","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f30096403462415491ca2ed1294c4bdc55d2d7fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1729d363_2e171c0c","updated":"2023-02-25 08:16:44.000000000","message":"recheck cinder-plugin-ceph-tempest post failure","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c26167830b802669908ed064ede31b7592dfdbf1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0243a583_bc93cb7e","updated":"2023-02-27 05:48:03.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican post failure and tempest-slow-py3 timed out","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"735a5a5d242f64c286e34f0a14b1f7407a9b3f91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"8a80e54c_da2cda57","updated":"2023-02-23 14:18:56.000000000","message":"recheck cinder-tempest-plugin-lvm-lio-barbican timeout while running the tests, it\u0027s passed on this patch previously, so I\u0027m assuming it\u0027s not a real problem","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"78917442aa8e58df87a3be35f5b31e94aef10183","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0e922c78_714215d2","updated":"2023-02-26 21:18:58.000000000","message":"recheck grenade - tempest.api.compute.security_groups.test_security_group_rules.SecurityGroupRulesTestJSON.test_security_group_rules_list failed because of a read timeout","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"8df9285595eda4763ca5ac748923e476f80c60d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c871ca36_4f33b1bb","updated":"2023-02-23 03:52:56.000000000","message":"recheck grenade failure","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"3def7d5c5bec13ce154541e7f2004a4b8607efec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"828aa1f2_c453b31f","updated":"2023-02-22 12:46:07.000000000","message":"recheck the fix is merged now so this should pass","commit_id":"09613c97060144cd436ffc079c1374710c501ac2"}],"cinder/volume/targets/driver.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"29ede41bf0a468f7214820702bbab103fb47b7df","unresolved":true,"context_lines":[{"line_number":87,"context_line":"        return a_host and (a_host \u003d\u003d B.get(\u0027host\u0027))"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def extend_target(self, volume):"},{"line_number":90,"context_line":"        \"\"\"Reinitializes a target after the LV has been extended."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        Most drivers don\u0027t need to do anything, but in other cases this may"},{"line_number":93,"context_line":"        cause IO disruption."}],"source_content_type":"text/x-python","patch_set":1,"id":"753e4b02_0f799aaf","line":90,"range":{"start_line":90,"start_character":44,"end_line":90,"end_character":46},"updated":"2022-05-27 15:09:37.000000000","message":"usually I see LV used in context of LVM. I see this is the same statement as the iSCSI driver[1] but should we modify it to make it more generalize since the nvmeof target also inherits from this class.\n\n[1] https://github.com/openstack/cinder/blob/master/cinder/volume/targets/iscsi.py#L323-L326","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"7683f6911ed2563bd37a3adbeeaa382580f0c95b","unresolved":false,"context_lines":[{"line_number":87,"context_line":"        return a_host and (a_host \u003d\u003d B.get(\u0027host\u0027))"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    def extend_target(self, volume):"},{"line_number":90,"context_line":"        \"\"\"Reinitializes a target after the LV has been extended."},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        Most drivers don\u0027t need to do anything, but in other cases this may"},{"line_number":93,"context_line":"        cause IO disruption."}],"source_content_type":"text/x-python","patch_set":1,"id":"fd6ab4cc_5e4ff178","line":90,"range":{"start_line":90,"start_character":44,"end_line":90,"end_character":46},"in_reply_to":"753e4b02_0f799aaf","updated":"2022-06-10 14:30:39.000000000","message":"OK","commit_id":"ad0597c30bb41ad8f36121d9f80af5e7dccad379"}]}