)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"be5a96f0e0d77b4c6fac0410874669115d86b821","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"20513a00_69a33d8d","updated":"2022-06-29 13:31:46.000000000","message":"Some few inline comments.","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"eb5c1395c215a795e3d50a6e3b2ea2d52da97045","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ecf92454_bfc2350c","updated":"2022-06-29 13:30:06.000000000","message":"Would be nice to have a Related-Bug in the commit message","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1b7e01f9cf5748e07862ff91c7db4a375403f6e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ea73141f_edfe633f","in_reply_to":"ecf92454_bfc2350c","updated":"2022-06-30 15:25:57.000000000","message":"Original change didn\u0027t have a bug https://review.opendev.org/c/openstack/tempest/+/842432\nBut added tempest to the gate failures that affect stable branches and referenced it in the commit message.","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b344a7da07ba9a98e673eea3d10960b0795e45f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9eba1b2e_1350afac","updated":"2022-07-14 08:40:14.000000000","message":"LGTM, thanks","commit_id":"d30577d07ceb184f7cf82b2d2f000a6381fb024d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"40b50807a0e96e7915f72210461f1d27c73a0e9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"14f3b91e_6e98820d","updated":"2022-07-05 14:14:53.000000000","message":"This is a fix that we need in order to solve [1]\n```\n    testtools.matchers._impl.MismatchError: [(\u0027Open Source\u0027, \u0027nfs\u0027, \u0027nfs\u0027)] !\u003d [(\u0027Open Source\u0027, \u0027nfs\u0027, \u0027NFS\u0027)]\n```\n\n[1] https://zuul.opendev.org/t/openstack/build/aa8cf50dc3d2444e8bb1a11e34f8ad11","commit_id":"d30577d07ceb184f7cf82b2d2f000a6381fb024d"}],"tempest/api/volume/admin/test_backends_capabilities.py":[{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"be5a96f0e0d77b4c6fac0410874669115d86b821","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        capabilities \u003d sorted(list(capabilities))"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        # Cinder Bug #1966103: Some drivers were reporting different strings"},{"line_number":51,"context_line":"        # to represent the same storage protocol. For backward compatibility,"},{"line_number":52,"context_line":"        # the scheduler can handle the variants, but to standardize this for"},{"line_number":53,"context_line":"        # operators (who may need to refer to the protocol in volume-type"},{"line_number":54,"context_line":"        # extra-specs), the get-pools and get-capabilities response was changed"}],"source_content_type":"text/x-python","patch_set":1,"id":"46f2ad24_cae59d8f","line":51,"updated":"2022-06-29 13:31:46.000000000","message":"Can you please refer the link of the bug which you trying to resolve here?","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1b7e01f9cf5748e07862ff91c7db4a375403f6e8","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        capabilities \u003d sorted(list(capabilities))"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"        # Cinder Bug #1966103: Some drivers were reporting different strings"},{"line_number":51,"context_line":"        # to represent the same storage protocol. For backward compatibility,"},{"line_number":52,"context_line":"        # the scheduler can handle the variants, but to standardize this for"},{"line_number":53,"context_line":"        # operators (who may need to refer to the protocol in volume-type"},{"line_number":54,"context_line":"        # extra-specs), the get-pools and get-capabilities response was changed"}],"source_content_type":"text/x-python","patch_set":1,"id":"fdf50f60_1a609c44","line":51,"in_reply_to":"46f2ad24_cae59d8f","updated":"2022-06-30 15:25:57.000000000","message":"Added to the commit message.","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"be5a96f0e0d77b4c6fac0410874669115d86b821","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        # to represent the same storage protocol. For backward compatibility,"},{"line_number":52,"context_line":"        # the scheduler can handle the variants, but to standardize this for"},{"line_number":53,"context_line":"        # operators (who may need to refer to the protocol in volume-type"},{"line_number":54,"context_line":"        # extra-specs), the get-pools and get-capabilities response was changed"},{"line_number":55,"context_line":"        # to only report the canonical name for a storage protocol, but these"},{"line_number":56,"context_line":"        # 2 REST API call swere not changed simultaneously, so we may or may"},{"line_number":57,"context_line":"        # not get canonical names, so just convert canonical names."}],"source_content_type":"text/x-python","patch_set":1,"id":"93894e90_ae47cfc0","line":54,"updated":"2022-06-29 13:31:46.000000000","message":"It would be easy to review if any api-ref doc mentions/defines the Rest api response parameters.","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"1b7e01f9cf5748e07862ff91c7db4a375403f6e8","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        # to represent the same storage protocol. For backward compatibility,"},{"line_number":52,"context_line":"        # the scheduler can handle the variants, but to standardize this for"},{"line_number":53,"context_line":"        # operators (who may need to refer to the protocol in volume-type"},{"line_number":54,"context_line":"        # extra-specs), the get-pools and get-capabilities response was changed"},{"line_number":55,"context_line":"        # to only report the canonical name for a storage protocol, but these"},{"line_number":56,"context_line":"        # 2 REST API call swere not changed simultaneously, so we may or may"},{"line_number":57,"context_line":"        # not get canonical names, so just convert canonical names."}],"source_content_type":"text/x-python","patch_set":1,"id":"349f94ff_566af22a","line":54,"in_reply_to":"93894e90_ae47cfc0","updated":"2022-06-30 15:25:57.000000000","message":"Cinder Rest API doesn\u0027t mention that values could be wrong before, but these are the 2 APIs that this test is calling\n\nhttps://docs.openstack.org/api-ref/block-storage/v3/?expanded\u003dshow-all-back-end-capabilities-detail,list-all-back-end-storage-pools-detail#list-all-back-end-storage-pools\n\nhttps://docs.openstack.org/api-ref/block-storage/v3/?expanded\u003dshow-all-back-end-capabilities-detail#show-all-back-end-capabilities\n\nAnd here is the release note for the fix:\n\nhttps://opendev.org/openstack/cinder/src/branch/master/releasenotes/notes/fix-storage_protocol-6baf55e13249463c.yaml","commit_id":"6538be953412c3e4e12cc0f4d0cf8cccd3f153ef"}]}