)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"6f6f395f14d478c6b9e72e62d90237798e6c9bc2","unresolved":true,"context_lines":[{"line_number":16,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d4, connect\u003d4, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde1cfb3a0\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":17,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d3, connect\u003d3, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcddbf223a0\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":18,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d2, connect\u003d2, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde0443340\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":19,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d1, connect\u003d1, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde0144a00\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":20,"context_line":"```"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes-Bug: #1971542"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f7b7fbd_2be40183","line":19,"updated":"2022-07-06 12:49:12.000000000","message":"I can see the Retry here, but it seems to me that the problem was not fixed, I\u0027m still seeing the \"Temporary failure in name resolution\". \n\nCould you please explain how the Retry fixed the problem? \n\nFeel free to correct me if this is some misunderstanding from my side.","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"3db0c732554c9d5893699a66cabb12c5e1825b09","unresolved":true,"context_lines":[{"line_number":16,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d4, connect\u003d4, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde1cfb3a0\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":17,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d3, connect\u003d3, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcddbf223a0\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":18,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d2, connect\u003d2, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde0443340\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":19,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d1, connect\u003d1, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde0144a00\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":20,"context_line":"```"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes-Bug: #1971542"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"58518953_dea7a447","line":19,"in_reply_to":"1f7b7fbd_2be40183","updated":"2022-07-06 14:41:34.000000000","message":"I think he could improve the commit message, if I got correctly. Indeed, the commit message makes we think that the idea is to fix the log message, but the idea is adding that the code is working as expected, that\u0027s it, the driver is now retrying.\n\nMaybe something like:\n\n[NetApp] Fix lack of retry HTTP requests\n\nThe driver HTTP requester is not handling connection error with some retries for avoiding temporary network failure in name resolution. This patch adds a custom `HTTPAdapter` with 5 retries, according to urrlib3 documentation [1]. Also, the connection retry will be visible in the log [2].\n\n[1] https://urllib3.readthedocs.io/en/latest/reference/urllib3.util.html#module-urllib3.util.retry\n[2] https://paste.opendev.org/show/bIcwjk91d4vy5Hnxxs4Z/\n\nCloses-Bug: #1971542\nChange-Id: Ic9ff8208f10df9dbed09717d6b218f6293d2338a","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"3a47fd7809fba67c7fafd96650d5bcb79a1337fe","unresolved":false,"context_lines":[{"line_number":16,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d4, connect\u003d4, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde1cfb3a0\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":17,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d3, connect\u003d3, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcddbf223a0\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":18,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d2, connect\u003d2, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde0443340\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":19,"context_line":"WARNING urllib3.connectionpool [req-d92bd8d6-f05f-404c-8ded-087d29c9bf9f] Retrying (Retry(total\u003d1, connect\u003d1, read\u003d2, redirect\u003dNone, status\u003dNone)) after connection broken by \u0027NewConnectionError(\u0027\u003curllib3.connection.HTTPSConnection object at 0x7fcde0144a00\u003e: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution\u0027)\u0027: /servlets/netapp.servlets.admin.XMLrequest_filer"},{"line_number":20,"context_line":"```"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Closes-Bug: #1971542"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9413d146_477b0c50","line":19,"in_reply_to":"58518953_dea7a447","updated":"2022-08-23 09:16:05.000000000","message":"Done","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"6f6f395f14d478c6b9e72e62d90237798e6c9bc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"449dedff_e1d14104","updated":"2022-07-06 12:49:12.000000000","message":"Hi Maurice, I added one comment inline.","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"3db0c732554c9d5893699a66cabb12c5e1825b09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"09a249ce_809ee5d6","updated":"2022-07-06 14:41:34.000000000","message":"More one thing, the commit message could be improved a bit to be more self explainable.","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"1abd4d69d885b809b26b0ff35a6f4807e7cedfdb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"730ad0d2_90e3d3c2","updated":"2022-07-06 14:41:52.000000000","message":"Should it have a release note ?","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"190fcb3c9ff477b17433bf9159ca137c8f72b36e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a32635f7_3a7d8972","updated":"2022-07-06 14:27:11.000000000","message":"Thank you Maurice for providing this improvement to the NetApp driver. Indeed, that is one miss on the HTTP driver implementation, we should retry for things like network errors (connection). I\u0027m just wondering if we should retry on \"read\" errors, may it have any side effect, since the data is already on the server side ?!","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"03bc5c83ab7b6be8f27b276c57a1085b8293b718","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cde6d2f7_a5f58267","updated":"2022-07-06 15:09:55.000000000","message":"The same topic about retries/timeout..","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"360b72f1b6e34296bc5815e951b644eb4847c160","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"77dbbfaf_0374b50f","updated":"2022-09-14 17:49:14.000000000","message":"LGTM; thanks Maurice and Kiran","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"f0ce0496ce158ba776aecef64323a8bf3a3a60d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4f35d49a_b4803826","updated":"2022-09-15 19:44:26.000000000","message":"Thank you for the fix! Code looks good","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"659c178c6aca0603f0af54757e2381f3158080f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3973b158_cd8f7dbf","updated":"2022-08-29 17:51:15.000000000","message":"There\u0027s a minor issue with the imports; and the NetApp-CI hasn\u0027t run here.. ","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":33648,"name":"Nahim Alves de Souza","email":"nahimsouza@outlook.com","username":"nahimsouza"},"change_message_id":"8b32799d086bcc2a0b51b0cd38f2ae051f41d46b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1c2d5ea3_1739ca26","updated":"2022-09-14 12:50:59.000000000","message":"run-NetApp CI","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c33425d648bae7543ac67fee5dce4db2ca85e415","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ee31bb24_fdc6e779","updated":"2022-09-14 15:56:54.000000000","message":"run-NetApp CI","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"784502d295fa2b9569afa3ef83ba52869e5c7abb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1e6695eb_3f8786a0","updated":"2022-09-13 15:13:09.000000000","message":"run-NetApp-CI","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ab672ef8b7e141ea5272760d2c2340e0075e480e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c21f1326_ca1cbede","updated":"2022-09-12 19:04:50.000000000","message":"run-NetApp-CI","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"b5986c081293454ec688e8a73e908c66ce84f62f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"10bab013_d4bace44","updated":"2022-08-29 11:28:00.000000000","message":"thanks, LGTM!","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"}],"manila/share/drivers/netapp/dataontap/client/api.py":[{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"190fcb3c9ff477b17433bf9159ca137c8f72b36e","unresolved":true,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        self._session \u003d requests.Session()"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"        max_retries \u003d Retry(total\u003d5, connect\u003d5, read\u003d2, backoff_factor\u003d1)"},{"line_number":234,"context_line":"        adapter \u003d HTTPAdapter(max_retries\u003dmax_retries)"},{"line_number":235,"context_line":"        self._session.mount(\u0027%s://\u0027 % self._protocol, adapter)"},{"line_number":236,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"92a843c9_21aaddf8","line":233,"range":{"start_line":233,"start_character":48,"end_line":233,"end_character":54},"updated":"2022-07-06 14:27:11.000000000","message":"Why do we want to retry on read erros ?! If I got correctly, the problem is during the connection (DNS lookups failure) It may have some side effect. Is it safe ?","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"823f950185d9995313eeaaaa2bbcae537ade7f8b","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        self._session \u003d requests.Session()"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"        max_retries \u003d Retry(total\u003d5, connect\u003d5, read\u003d2, backoff_factor\u003d1)"},{"line_number":234,"context_line":"        adapter \u003d HTTPAdapter(max_retries\u003dmax_retries)"},{"line_number":235,"context_line":"        self._session.mount(\u0027%s://\u0027 % self._protocol, adapter)"},{"line_number":236,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"392ae920_35721695","line":233,"range":{"start_line":233,"start_character":48,"end_line":233,"end_character":54},"in_reply_to":"92a843c9_21aaddf8","updated":"2022-09-15 19:50:56.000000000","message":"Marking this as resolved :)","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":31721,"name":"Felipe Rodrigues","email":"felipefuty01@gmail.com","username":"felipefutty"},"change_message_id":"03bc5c83ab7b6be8f27b276c57a1085b8293b718","unresolved":true,"context_lines":[{"line_number":337,"context_line":"        try:"},{"line_number":338,"context_line":"            if hasattr(self, \u0027_timeout\u0027):"},{"line_number":339,"context_line":"                response \u003d self._session.post("},{"line_number":340,"context_line":"                    self._get_url(), data\u003drequest_d, timeout\u003dself._timeout)"},{"line_number":341,"context_line":"            else:"},{"line_number":342,"context_line":"                response \u003d self._session.post("},{"line_number":343,"context_line":"                    self._get_url(), data\u003drequest_d)"}],"source_content_type":"text/x-python","patch_set":1,"id":"70ef7d1c_d4eb1a5d","line":340,"range":{"start_line":340,"start_character":53,"end_line":340,"end_character":73},"updated":"2022-07-06 15:09:55.000000000","message":"Another possible issue on connection NetApp driver:\n\nAlthough the `self._timeout` is passed down, it is never set before. Since requests default timeout is to wait infinitely[1], it might be a problem hanging the thread forever depending on the server behavior.\n\nWDYT ?\n\n[1] https://requests.readthedocs.io/en/latest/user/advanced/#timeouts","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"3a47fd7809fba67c7fafd96650d5bcb79a1337fe","unresolved":false,"context_lines":[{"line_number":337,"context_line":"        try:"},{"line_number":338,"context_line":"            if hasattr(self, \u0027_timeout\u0027):"},{"line_number":339,"context_line":"                response \u003d self._session.post("},{"line_number":340,"context_line":"                    self._get_url(), data\u003drequest_d, timeout\u003dself._timeout)"},{"line_number":341,"context_line":"            else:"},{"line_number":342,"context_line":"                response \u003d self._session.post("},{"line_number":343,"context_line":"                    self._get_url(), data\u003drequest_d)"}],"source_content_type":"text/x-python","patch_set":1,"id":"f9d0e42b_4fb30384","line":340,"range":{"start_line":340,"start_character":53,"end_line":340,"end_character":73},"in_reply_to":"70ef7d1c_d4eb1a5d","updated":"2022-08-23 09:16:05.000000000","message":"Done","commit_id":"4b9ca52cd47610650bd9d65027237157aa2ecfe4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"659c178c6aca0603f0af54757e2381f3158080f4","unresolved":true,"context_lines":[{"line_number":25,"context_line":"from oslo_log import log"},{"line_number":26,"context_line":"from oslo_serialization import jsonutils"},{"line_number":27,"context_line":"import requests"},{"line_number":28,"context_line":"from requests.adapters import HTTPAdapter"},{"line_number":29,"context_line":"from requests import auth"},{"line_number":30,"context_line":"from requests.packages.urllib3.util.retry import Retry"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"5738b784_f3ce3031","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":41},"updated":"2022-08-29 17:51:15.000000000","message":"https://docs.openstack.org/hacking/latest/user/hacking.html#imports\n\n\n from requests import adapters\n \n  ..\n \n adapters.HTTPAdapter","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"14cdc7a4a369c16cb64af44f99032077721269ee","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from oslo_log import log"},{"line_number":26,"context_line":"from oslo_serialization import jsonutils"},{"line_number":27,"context_line":"import requests"},{"line_number":28,"context_line":"from requests.adapters import HTTPAdapter"},{"line_number":29,"context_line":"from requests import auth"},{"line_number":30,"context_line":"from requests.packages.urllib3.util.retry import Retry"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ff063496_f1687cdc","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":41},"in_reply_to":"5738b784_f3ce3031","updated":"2022-08-30 09:11:27.000000000","message":"Done","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"659c178c6aca0603f0af54757e2381f3158080f4","unresolved":true,"context_lines":[{"line_number":27,"context_line":"import requests"},{"line_number":28,"context_line":"from requests.adapters import HTTPAdapter"},{"line_number":29,"context_line":"from requests import auth"},{"line_number":30,"context_line":"from requests.packages.urllib3.util.retry import Retry"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"from manila import exception"},{"line_number":33,"context_line":"from manila.i18n import _"}],"source_content_type":"text/x-python","patch_set":3,"id":"38cee018_fa5283f3","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":54},"updated":"2022-08-29 17:51:15.000000000","message":"same comment as above","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"14cdc7a4a369c16cb64af44f99032077721269ee","unresolved":false,"context_lines":[{"line_number":27,"context_line":"import requests"},{"line_number":28,"context_line":"from requests.adapters import HTTPAdapter"},{"line_number":29,"context_line":"from requests import auth"},{"line_number":30,"context_line":"from requests.packages.urllib3.util.retry import Retry"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"from manila import exception"},{"line_number":33,"context_line":"from manila.i18n import _"}],"source_content_type":"text/x-python","patch_set":3,"id":"e283ceb7_a339ea53","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":54},"in_reply_to":"38cee018_fa5283f3","updated":"2022-08-30 09:11:27.000000000","message":"Done","commit_id":"d3310b0b387727aee17ff1dfe5679c24f8a7c878"}]}