)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"be23864ac4cf60f5468d5b2e793a28598dbae609","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"24f18424_3b3ffa38","updated":"2022-01-21 14:29:08.000000000","message":"I believe your commit message does not match the openstack community pattern, you could take a moment to take a look on the following link:\nhttps://docs.openstack.org/contributors/common/git.html#commit-messages","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"8acb8bc8e92d7b3bee16b44f07079556a73d17a5","unresolved":true,"context_lines":[{"line_number":7,"context_line":"requests.packages.urllib3.disable_warnings: Module patch check for attributes."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There is a possible installation bug in \u0027requests\u0027 library module for python, that causes the imported library packages not being accessable, when installed using pip, by module path \u0027requests.packages.urllib3\u0027. The check makes sure that library was installed properly."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Signed-off-by: mionsz \u003cmilosz.linkiewicz@intel.com\u003e"},{"line_number":12,"context_line":"Change-Id: Ie38062a20bb289673b1c76c80dcc57ec04b61aa4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"2df97aaa_1a3a657a","line":10,"updated":"2022-01-24 14:25:18.000000000","message":"The real bug here is that the requests library no longer carries a requests.packages.urllib3 library anyway -- we should update the Cinder code to no longer reference that and depend on a sufficiently new version of requests.\n\nhttps://github.com/psf/requests/commit/8e17600ef60de4faf632acb55d15cb3c178de9bb","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28801,"name":"Cisco Cinder CI","email":"cisco-cinder-ci@cisco.com","username":"cisco-cinder-ci"},"change_message_id":"ea339dad6513735272ae88c24b30218f6a9b75b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5b194ae4_9023f38c","updated":"2022-01-18 04:09:17.000000000","message":"cisco-cinder-ci","commit_id":"d367f4f30ffe7e5d1472ecc34bdaa8143c3a9a9e"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"68c8cd5ed5c12ddf8e5033f9aa49c9d05afd0862","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"49d2fb80_57bd4657","updated":"2022-01-21 15:28:33.000000000","message":"Doesn\u0027t https://review.opendev.org/c/openstack/cinder/+/825130 address the same issue?","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"858e136368250181c4a6f9b4bea1fe8fd8ebb2c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"57623153_b33d2e34","updated":"2022-01-20 07:16:09.000000000","message":"LGTM","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"},{"author":{"_account_id":33431,"name":"Fábio Oliveira","email":"fabioaurelio1269@gmail.com","username":"fabiooliveira1"},"change_message_id":"be23864ac4cf60f5468d5b2e793a28598dbae609","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"de8de576_7b5407a7","updated":"2022-01-21 14:29:08.000000000","message":"Thank you for working on this!\nThere are some issues with the commit message.\nI\u0027m also in doubt if this patch needs a bug item on launchpad?","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"31d097f0d7d52610daccda791ac239f2ae80fc2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"90012eeb_0df3cc17","updated":"2022-01-24 13:30:52.000000000","message":"https://review.opendev.org/c/openstack/cinder/+/825130 has landed - can you please check if it fixes the issue? If it does, please abandon this patch and rebase https://review.opendev.org/c/openstack/cinder/+/825012 directly on top of https://review.opendev.org/c/openstack/cinder/+/824839","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"}],"cinder/volume/manager.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0121ee4a77a172cb4719434a119a5d1f73e590f8","unresolved":true,"context_lines":[{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        if self.configuration.suppress_requests_ssl_warnings \\"},{"line_number":298,"context_line":"           and \"packages\" in dir(requests):"},{"line_number":299,"context_line":"            LOG.warning(\"Suppressing requests library SSL Warnings. Success\")"},{"line_number":300,"context_line":"            requests.packages.urllib3.disable_warnings("},{"line_number":301,"context_line":"                requests.packages.urllib3.exceptions.InsecureRequestWarning)"},{"line_number":302,"context_line":"            requests.packages.urllib3.disable_warnings("}],"source_content_type":"text/x-python","patch_set":4,"id":"319cc9df_8087c6b4","line":299,"range":{"start_line":299,"start_character":67,"end_line":299,"end_character":75},"updated":"2022-01-21 15:38:16.000000000","message":"\"Success\" is confusing here -- I think the previous text would be better.","commit_id":"7397cc36f41f9d5d59d4ea6ce5e82a6a20969174"}]}