)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"f1b923c5a5de50ba70a76fc9f0607a1cef0b4982","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"813387b2_9f0fed89","updated":"2022-06-07 12:23:35.000000000","message":"Change looks good. Zuul also passed. Hopefully CI will pass soon.","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"6c2a4772aa91e6e66c4111ad6cb0e2b48fa11d16","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0eb7da5b_1283bbdc","updated":"2022-06-03 15:36:47.000000000","message":"LGTM","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1116b32c4d77d9a721693889c3279a543e4dc17f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fe299236_c9d54a49","updated":"2022-06-17 16:05:55.000000000","message":"Question inline.","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"b7a5e6a551e602c20b312656f3b334aa5cf9b2b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a9dc0c84_8d68508b","updated":"2022-06-03 11:22:09.000000000","message":"This bug fix is with respect to the change:\nhttps://review.opendev.org/c/openstack/cinder/+/817351\n\n\nExisting UTs which is relevant to this code change are mentioned here: \nhttps://github.com/openstack/cinder/blob/master/cinder/tests/unit/volume/drivers/ibm/test_storwize_svc.py#L811","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":34541,"name":"Manish","email":"manish.laddha@ibm.com","username":"manishladdha"},"change_message_id":"67616220de27f8b42ad567c9af38e17b400b187d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fcb1edcc_df52c10c","updated":"2022-06-06 04:17:03.000000000","message":"changes look good","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":34541,"name":"Manish","email":"manish.laddha@ibm.com","username":"manishladdha"},"change_message_id":"c085289ca88b5541760c86415e4af769861756ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"484e1c5a_49e8ec93","updated":"2022-06-06 04:17:21.000000000","message":"run-IBM Storage CI","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"ef07326c066ed74c1aff2bfe9eef479a3126652f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ab946a74_23c6888c","updated":"2022-06-06 10:41:48.000000000","message":"run-IBM Storage CI","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"7bb95bd434929a2eacf3fcced0138ed6449ffb4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e951ed3e_25252e86","updated":"2022-06-06 19:54:29.000000000","message":"run-IBM Storage CI","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4c726350b0c24847a06fc1ee9a2b5f5862d2be2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f8a9dd84_b85e476a","updated":"2022-08-17 12:22:57.000000000","message":"Change meets the bug description, will only make the call for \u003e\u003d 8.5.x","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"74e30203f49d62eb6385de7b74ce685acb6f302a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f245ac9e_4ca2194e","updated":"2022-08-12 05:57:43.000000000","message":"Code Level Fixed","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":34201,"name":"Mounika Sreeram","email":"sreeram.mounika@ibm.com","username":"sreerammounika"},"change_message_id":"bfafb5a745ecee5202bbba492a6fcd3dbe3041ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bc1addb2_627770cf","updated":"2022-08-12 08:11:12.000000000","message":"Code looks good. All the comments are addressed.","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"a286fe64745831af4486d59713862459e0404e79","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"59be5a31_a636305a","updated":"2022-08-19 15:54:16.000000000","message":"IBM Storage CI passed and code looks good to me","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":34541,"name":"Manish","email":"manish.laddha@ibm.com","username":"manishladdha"},"change_message_id":"30f6859e4d3e7a1f410270831112069e26896713","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"576c42e7_ff3bc86b","updated":"2022-08-12 10:44:35.000000000","message":"looks good to me","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":22248,"name":"HPE AlletraMP iSCSI CI","email":"hp.cinder.blr.ci@groups.ext.hpe.com","username":"hp-storage-blr-ci","tags":["SERVICE_USER"]},"change_message_id":"c6e44dd4c4871d4f3424ebcd4b942c14b2e81cd2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"88db1f1c_5f690a73","updated":"2022-08-22 10:46:08.000000000","message":"run-HPE Storage CI","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"9dec3ed1528471f6a0bed027f0748d4cc013f0fd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3bd5d851_02626ff4","updated":"2022-08-12 08:37:31.000000000","message":"run-IBM Storage CI","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"44a3bf9915901ef2ee684c7bc0deebcb1706d2b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"40887570_fe8059f6","updated":"2022-08-12 08:09:04.000000000","message":"run-IBM Storage CI","commit_id":"0df29396bfa4eca306b5e4700defbfa5d5637b24"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"1116b32c4d77d9a721693889c3279a543e4dc17f","unresolved":true,"context_lines":[{"line_number":1160,"context_line":"        # indicates whether the port can be used for host I/O"},{"line_number":1161,"context_line":"        targetportfc_resp \u003d self.ssh.lstargetportfc(current_node_id\u003dnode_id,"},{"line_number":1162,"context_line":"                                                    host_io_permitted\u003dhost_io)"},{"line_number":1163,"context_line":"        if code_level \u003e (8, 4, 2, 0):"},{"line_number":1164,"context_line":"            portset_name \u003d portset if portset else \u0027portset64\u0027"},{"line_number":1165,"context_line":"            port_ids \u003d set()"},{"line_number":1166,"context_line":"            fcportsetmember_resp \u003d self.ssh.lsfcportsetmember()"}],"source_content_type":"text/x-python","patch_set":1,"id":"3433c676_2161a671","line":1163,"updated":"2022-06-17 16:05:55.000000000","message":"This assumes that the call will be available in 8.4.2.1, for example, but your bug report and your commit message say that the functionality wasn\u0027t introduced until the 8.5.x series.  So should this be\n\n  code_level \u003e\u003d (8, 5, 0, 0)\n\n?","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"74e30203f49d62eb6385de7b74ce685acb6f302a","unresolved":false,"context_lines":[{"line_number":1160,"context_line":"        # indicates whether the port can be used for host I/O"},{"line_number":1161,"context_line":"        targetportfc_resp \u003d self.ssh.lstargetportfc(current_node_id\u003dnode_id,"},{"line_number":1162,"context_line":"                                                    host_io_permitted\u003dhost_io)"},{"line_number":1163,"context_line":"        if code_level \u003e (8, 4, 2, 0):"},{"line_number":1164,"context_line":"            portset_name \u003d portset if portset else \u0027portset64\u0027"},{"line_number":1165,"context_line":"            port_ids \u003d set()"},{"line_number":1166,"context_line":"            fcportsetmember_resp \u003d self.ssh.lsfcportsetmember()"}],"source_content_type":"text/x-python","patch_set":1,"id":"140a6a92_9794a9a5","line":1163,"in_reply_to":"3433c676_2161a671","updated":"2022-08-12 05:57:43.000000000","message":"Fixed.","commit_id":"e645f4cfcaa630708b0f4271b1ea4ed63c5943d2"}]}
