)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f70ea67197b0a7f8e94dd87d15b84671e31b2144","unresolved":true,"context_lines":[{"line_number":15,"context_line":"intentionally used by developer which bandit"},{"line_number":16,"context_line":"wasn\u0027t happy about."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Depends-On: I27d1204ec7dafd3b578d1261c3fd2e371ae405fb"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I171c90a281c7b62c2601131293f9f00a926641e2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"e47ef4a1_0eaf9e4d","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":53},"updated":"2024-04-09 18:49:27.000000000","message":"you still need this","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":36758,"name":"Elvis Kobi Acheampong","email":"kobi.elvis790@yahoo.com","username":"acheampongelvis"},"change_message_id":"d0ac44cb29a6398015c802c17cff23ecd1ae150f","unresolved":false,"context_lines":[{"line_number":15,"context_line":"intentionally used by developer which bandit"},{"line_number":16,"context_line":"wasn\u0027t happy about."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Depends-On: I27d1204ec7dafd3b578d1261c3fd2e371ae405fb"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I171c90a281c7b62c2601131293f9f00a926641e2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9594ca8f_b2bb7778","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":53},"in_reply_to":"e47ef4a1_0eaf9e4d","updated":"2024-04-14 15:52:17.000000000","message":"Done","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d479b9685c47c27ad715c4d6da10314aa0b7368e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d7149b3b_1208ce42","updated":"2024-04-04 20:08:13.000000000","message":"Thanks Elvis, some comments inline","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"79f450125d9963329ca015cfbeb41d7bf5ab9532","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9aa3df9f_c0ca0f0a","updated":"2024-04-18 21:00:51.000000000","message":"You abandoned the wrong patch.. Please un-abandon this, and squash the next commit into this commit","commit_id":"b17866e916442196c3c0b968c46a7ac2ed0ac4e8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0ff211a4db1461516679f48e4a062af9c5858ee0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b7f74f1e_490dfbb1","updated":"2024-04-30 20:50:39.000000000","message":"Thanks Elvis; LGTM","commit_id":"05c9446bd96e393c739bdb4235b74a1b5a127847"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"8ab525b59a4cd88bad70ff2b62d397dad2d9314f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"81246c65_425b5a29","updated":"2024-05-03 20:39:18.000000000","message":"We\u0027ve approved similar things; lets get this moving..","commit_id":"05c9446bd96e393c739bdb4235b74a1b5a127847"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"0b0c10cdfce16d8de421034a041f6d4f9216f8b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4d0acdf5_5d3f816f","updated":"2024-05-08 21:10:23.000000000","message":"needs follow up patch to address in line comment","commit_id":"05c9446bd96e393c739bdb4235b74a1b5a127847"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b4ac70b4bb661fe042512cd0201f5b8d8baf7e56","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4cd5b3f2_613abd53","updated":"2024-05-07 16:28:36.000000000","message":"recheck\n\nbug/2064673","commit_id":"05c9446bd96e393c739bdb4235b74a1b5a127847"}],"manila/share/drivers/qnap/api.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d479b9685c47c27ad715c4d6da10314aa0b7368e","unresolved":true,"context_lines":[{"line_number":21,"context_line":"import re"},{"line_number":22,"context_line":"import ssl"},{"line_number":23,"context_line":"from urllib import parse as urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from defusedxml.ElementTree import fromstring"},{"line_number":26,"context_line":"from oslo_log import log as logging"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7fa4f452_f5c4d00a","line":24,"updated":"2024-04-04 20:08:13.000000000","message":"The imports here are messed up; can you please fix them. \n\nImports are broken into three groups by convention: https://docs.openstack.org/hacking/latest/user/hacking.html#imports","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f70ea67197b0a7f8e94dd87d15b84671e31b2144","unresolved":false,"context_lines":[{"line_number":21,"context_line":"import re"},{"line_number":22,"context_line":"import ssl"},{"line_number":23,"context_line":"from urllib import parse as urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from defusedxml.ElementTree import fromstring"},{"line_number":26,"context_line":"from oslo_log import log as logging"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1f96da24_c370603e","line":24,"in_reply_to":"271000da_9e4b0df4","updated":"2024-04-09 18:49:27.000000000","message":"My bad; this import order conforms to our guidelines.","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":36758,"name":"Elvis Kobi Acheampong","email":"kobi.elvis790@yahoo.com","username":"acheampongelvis"},"change_message_id":"75f15012ce4773df9698fa5407f349ce7c4be938","unresolved":true,"context_lines":[{"line_number":21,"context_line":"import re"},{"line_number":22,"context_line":"import ssl"},{"line_number":23,"context_line":"from urllib import parse as urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from defusedxml.ElementTree import fromstring"},{"line_number":26,"context_line":"from oslo_log import log as logging"},{"line_number":27,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"271000da_9e4b0df4","line":24,"in_reply_to":"7fa4f452_f5c4d00a","updated":"2024-04-09 18:05:31.000000000","message":"I think it follows the OpenStack Style Guidelines. Can you elaborate more on what is wrong?","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"d479b9685c47c27ad715c4d6da10314aa0b7368e","unresolved":true,"context_lines":[{"line_number":22,"context_line":"import ssl"},{"line_number":23,"context_line":"from urllib import parse as urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from defusedxml.ElementTree import fromstring"},{"line_number":26,"context_line":"from oslo_log import log as logging"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"from manila import exception"}],"source_content_type":"text/x-python","patch_set":4,"id":"1ba2c631_3d62b134","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":45},"updated":"2024-04-04 20:08:13.000000000","message":"we have a convention to not import objects; we import modules and use the objects from them.. please do:\n\n```\nfrom defusedxml import ElementTree as ET\n```\n\nand all these lines of code using \"ET.fromstring\" needn\u0027t be changed\n\nalthough, I would rather prefer:\n\n```\nfrom defusedxml import ElementTree as etree\n```\n\nand modifying the code to use:\n\n```\netree.fromstring(..)\n```","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":36758,"name":"Elvis Kobi Acheampong","email":"kobi.elvis790@yahoo.com","username":"acheampongelvis"},"change_message_id":"75f15012ce4773df9698fa5407f349ce7c4be938","unresolved":false,"context_lines":[{"line_number":22,"context_line":"import ssl"},{"line_number":23,"context_line":"from urllib import parse as urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from defusedxml.ElementTree import fromstring"},{"line_number":26,"context_line":"from oslo_log import log as logging"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"from manila import exception"}],"source_content_type":"text/x-python","patch_set":4,"id":"b78ce9e8_01ca1a6c","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":45},"in_reply_to":"1ba2c631_3d62b134","updated":"2024-04-09 18:05:31.000000000","message":"Done","commit_id":"4e377e556b3d97912a9a3caad9a3da522335dabe"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f70ea67197b0a7f8e94dd87d15b84671e31b2144","unresolved":true,"context_lines":[{"line_number":79,"context_line":"    def _prepare_connection(self, isSSL, ip, port):"},{"line_number":80,"context_line":"        if isSSL:"},{"line_number":81,"context_line":"            if hasattr(ssl, \u0027_create_unverified_context\u0027):"},{"line_number":82,"context_line":"                context \u003d ssl._create_unverified_context() # nosec"},{"line_number":83,"context_line":"                connection \u003d http_client.HTTPSConnection(ip,"},{"line_number":84,"context_line":"                                                         port\u003dport,"},{"line_number":85,"context_line":"                                                         context\u003dcontext)"}],"source_content_type":"text/x-python","patch_set":5,"id":"8c798c10_50a7038c","line":82,"range":{"start_line":82,"start_character":58,"end_line":82,"end_character":66},"updated":"2024-04-09 18:49:27.000000000","message":"two spaces before # nosec:\n```\ncontext \u003d ssl._create_unverified_context()  # nosec\n```","commit_id":"b17866e916442196c3c0b968c46a7ac2ed0ac4e8"},{"author":{"_account_id":36758,"name":"Elvis Kobi Acheampong","email":"kobi.elvis790@yahoo.com","username":"acheampongelvis"},"change_message_id":"d0ac44cb29a6398015c802c17cff23ecd1ae150f","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    def _prepare_connection(self, isSSL, ip, port):"},{"line_number":80,"context_line":"        if isSSL:"},{"line_number":81,"context_line":"            if hasattr(ssl, \u0027_create_unverified_context\u0027):"},{"line_number":82,"context_line":"                context \u003d ssl._create_unverified_context() # nosec"},{"line_number":83,"context_line":"                connection \u003d http_client.HTTPSConnection(ip,"},{"line_number":84,"context_line":"                                                         port\u003dport,"},{"line_number":85,"context_line":"                                                         context\u003dcontext)"}],"source_content_type":"text/x-python","patch_set":5,"id":"aab8a9b5_05cbc3cc","line":82,"range":{"start_line":82,"start_character":58,"end_line":82,"end_character":66},"in_reply_to":"8c798c10_50a7038c","updated":"2024-04-14 15:52:17.000000000","message":"Done","commit_id":"b17866e916442196c3c0b968c46a7ac2ed0ac4e8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9415fbfddfd69f9973477cb951b0b4430f9adafa","unresolved":true,"context_lines":[{"line_number":79,"context_line":"    def _prepare_connection(self, isSSL, ip, port):"},{"line_number":80,"context_line":"        if isSSL:"},{"line_number":81,"context_line":"            if hasattr(ssl, \u0027_create_unverified_context\u0027):"},{"line_number":82,"context_line":"                context \u003d ssl._create_unverified_context() # nosec"},{"line_number":83,"context_line":"                connection \u003d http_client.HTTPSConnection(ip,"},{"line_number":84,"context_line":"                                                         port\u003dport,"},{"line_number":85,"context_line":"                                                         context\u003dcontext)"}],"source_content_type":"text/x-python","patch_set":5,"id":"e4c317a3_d14820a8","line":82,"range":{"start_line":82,"start_character":58,"end_line":82,"end_character":66},"in_reply_to":"aab8a9b5_05cbc3cc","updated":"2024-04-30 17:20:10.000000000","message":"Please use the exact bandit error code here; for instance:\n\n```\ncontext \u003d ssl._create_unverified_context()  # nosec B501\n```","commit_id":"b17866e916442196c3c0b968c46a7ac2ed0ac4e8"},{"author":{"_account_id":36758,"name":"Elvis Kobi Acheampong","email":"kobi.elvis790@yahoo.com","username":"acheampongelvis"},"change_message_id":"0906b4b09a259a4ead9ac9eaf4cc1a2c75958ca3","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    def _prepare_connection(self, isSSL, ip, port):"},{"line_number":80,"context_line":"        if isSSL:"},{"line_number":81,"context_line":"            if hasattr(ssl, \u0027_create_unverified_context\u0027):"},{"line_number":82,"context_line":"                context \u003d ssl._create_unverified_context() # nosec"},{"line_number":83,"context_line":"                connection \u003d http_client.HTTPSConnection(ip,"},{"line_number":84,"context_line":"                                                         port\u003dport,"},{"line_number":85,"context_line":"                                                         context\u003dcontext)"}],"source_content_type":"text/x-python","patch_set":5,"id":"8f7d8ecb_d59ddb65","line":82,"range":{"start_line":82,"start_character":58,"end_line":82,"end_character":66},"in_reply_to":"e4c317a3_d14820a8","updated":"2024-04-30 17:46:11.000000000","message":"Done","commit_id":"b17866e916442196c3c0b968c46a7ac2ed0ac4e8"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"0b0c10cdfce16d8de421034a041f6d4f9216f8b7","unresolved":true,"context_lines":[{"line_number":79,"context_line":"    def _prepare_connection(self, isSSL, ip, port):"},{"line_number":80,"context_line":"        if isSSL:"},{"line_number":81,"context_line":"            if hasattr(ssl, \u0027_create_unverified_context\u0027):"},{"line_number":82,"context_line":"                context \u003d ssl._create_unverified_context()  # nosec B314"},{"line_number":83,"context_line":"                connection \u003d http_client.HTTPSConnection(ip,"},{"line_number":84,"context_line":"                                                         port\u003dport,"},{"line_number":85,"context_line":"                                                         context\u003dcontext)"}],"source_content_type":"text/x-python","patch_set":6,"id":"4b1bd512_ae399e8f","line":82,"range":{"start_line":82,"start_character":68,"end_line":82,"end_character":72},"updated":"2024-05-08 21:10:23.000000000","message":"should be B323","commit_id":"05c9446bd96e393c739bdb4235b74a1b5a127847"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"c478f989b7463ab121ad4ed1af7a757f51245ac5","unresolved":false,"context_lines":[{"line_number":79,"context_line":"    def _prepare_connection(self, isSSL, ip, port):"},{"line_number":80,"context_line":"        if isSSL:"},{"line_number":81,"context_line":"            if hasattr(ssl, \u0027_create_unverified_context\u0027):"},{"line_number":82,"context_line":"                context \u003d ssl._create_unverified_context()  # nosec B314"},{"line_number":83,"context_line":"                connection \u003d http_client.HTTPSConnection(ip,"},{"line_number":84,"context_line":"                                                         port\u003dport,"},{"line_number":85,"context_line":"                                                         context\u003dcontext)"}],"source_content_type":"text/x-python","patch_set":6,"id":"4ee65126_4fcb46a1","line":82,"range":{"start_line":82,"start_character":68,"end_line":82,"end_character":72},"in_reply_to":"4b1bd512_ae399e8f","updated":"2024-05-13 14:27:36.000000000","message":"resolved: https://review.opendev.org/c/openstack/manila/+/918779","commit_id":"05c9446bd96e393c739bdb4235b74a1b5a127847"}]}