)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"change_message_id":"774e437a5792ed444d22127725112ccc05c2ee2c","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[fix] Openstack exporter update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This ps updates the logic to clear the token when it"},{"line_number":10,"context_line":"receives error from the keystone."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I0c4bcb17aa420e31591405b6000a2db2821e2428"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"68bab6d0_84a2446f","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":31},"updated":"2021-03-16 16:22:57.000000000","message":"I\u0027d reset only for authorization error, 401.","commit_id":"19f790bb9ec3c1c64c963dd92ea5286384162cad"}],"prometheus-openstack-exporter/exporter/check_os_api.py":[{"author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"change_message_id":"72dfa7c688d4f6a0789ff37f9e6b2c2c0f9989b2","unresolved":true,"context_lines":[{"line_number":96,"context_line":"                        name, status_code, check[\u0027expect\u0027])"},{"line_number":97,"context_line":"                )"},{"line_number":98,"context_line":"                status \u003d self.FAIL"},{"line_number":99,"context_line":"                flag \u003d 1"},{"line_number":100,"context_line":"            else:"},{"line_number":101,"context_line":"                status \u003d self.OK"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"c757f7f1_cb587fb4","line":99,"range":{"start_line":99,"start_character":16,"end_line":99,"end_character":21},"updated":"2021-03-16 13:32:40.000000000","message":"I would make it a bool and rename is something like reset_token or clear_token or something like that.","commit_id":"a23ee62f7d8a87e81fb2c02e0e0344612f669add"},{"author":{"_account_id":18256,"name":"Hemachandra Reddy","email":"hemachandra.reddy@att.com","username":"hr858f"},"change_message_id":"774e437a5792ed444d22127725112ccc05c2ee2c","unresolved":true,"context_lines":[{"line_number":96,"context_line":"                        name, status_code, check[\u0027expect\u0027])"},{"line_number":97,"context_line":"                )"},{"line_number":98,"context_line":"                status \u003d self.FAIL"},{"line_number":99,"context_line":"                reset_token \u003d True"},{"line_number":100,"context_line":"            else:"},{"line_number":101,"context_line":"                status \u003d self.OK"},{"line_number":102,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"8244e729_573b34e4","line":99,"range":{"start_line":99,"start_character":15,"end_line":99,"end_character":34},"updated":"2021-03-16 16:22:57.000000000","message":"How about reset only for error code 401, and stop getting any more tokens after four retries. Anymore retries would lock the user up. Please note, \"placement\" expects \"401\".","commit_id":"19f790bb9ec3c1c64c963dd92ea5286384162cad"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"aa1c2cb759c313147f5d67b214332b08bbba04ec","unresolved":true,"context_lines":[{"line_number":96,"context_line":"                        name, status_code, check[\u0027expect\u0027])"},{"line_number":97,"context_line":"                )"},{"line_number":98,"context_line":"                status \u003d self.FAIL"},{"line_number":99,"context_line":"                if status_code is 401 and name is not \u0027placement\u0027:"},{"line_number":100,"context_line":"                    self.osclient.clear_token()"},{"line_number":101,"context_line":"                    logger.info("},{"line_number":102,"context_line":"                        \"Received 401 authorization error. Resetting the token.\")"}],"source_content_type":"text/x-python","patch_set":8,"id":"88230c99_c60f8923","line":99,"range":{"start_line":99,"start_character":47,"end_line":99,"end_character":53},"updated":"2021-03-30 07:06:02.000000000","message":"!\u003d we are doing value comparison, not checking if they point to the same obj in memory.","commit_id":"eed01d98e60daa4c151bb5d0c3f8a94c83c877b5"},{"author":{"_account_id":30582,"name":"RADHIKA PAI","email":"rp592h@att.com","username":"rp592h"},"change_message_id":"c136ad8cd0a88c4864df83661a434feba0d422a1","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                        name, status_code, check[\u0027expect\u0027])"},{"line_number":97,"context_line":"                )"},{"line_number":98,"context_line":"                status \u003d self.FAIL"},{"line_number":99,"context_line":"                if status_code is 401 and name is not \u0027placement\u0027:"},{"line_number":100,"context_line":"                    self.osclient.clear_token()"},{"line_number":101,"context_line":"                    logger.info("},{"line_number":102,"context_line":"                        \"Received 401 authorization error. Resetting the token.\")"}],"source_content_type":"text/x-python","patch_set":8,"id":"bb13a843_8e371280","line":99,"range":{"start_line":99,"start_character":47,"end_line":99,"end_character":53},"in_reply_to":"88230c99_c60f8923","updated":"2021-03-31 15:47:04.000000000","message":"Done","commit_id":"eed01d98e60daa4c151bb5d0c3f8a94c83c877b5"}]}