)]}'
{"ironic_inspector/common/swift.py":[{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"61ecadd26d0a83a40576ff28b120ed5646ea79cc","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            adapter \u003d keystone.get_adapter(\u0027swift\u0027, session\u003dSWIFT_SESSION,"},{"line_number":62,"context_line":"                                           **adapter_opts)"},{"line_number":63,"context_line":"        except Exception as exc:"},{"line_number":64,"context_line":"            raise utils.Error(_(\"Could not create an adapter to connect to \""},{"line_number":65,"context_line":"                                \"the object storage service: %s\") % exc)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        # TODO(pas-ha) reverse-construct SSL-related session options here"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_a37b8689","line":64,"range":{"start_line":64,"start_character":33,"end_line":64,"end_character":74},"updated":"2019-05-31 14:56:16.000000000","message":"It seems get_session can get exception, so do we also need to raise exception in L61?","commit_id":"cebeb820068b3aa7f7a8a145888e226430987868"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"fee826f36194e19a59d0495f038fd76f72057d38","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            adapter \u003d keystone.get_adapter(\u0027swift\u0027, session\u003dSWIFT_SESSION,"},{"line_number":62,"context_line":"                                           **adapter_opts)"},{"line_number":63,"context_line":"        except Exception as exc:"},{"line_number":64,"context_line":"            raise utils.Error(_(\"Could not create an adapter to connect to \""},{"line_number":65,"context_line":"                                \"the object storage service: %s\") % exc)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        # TODO(pas-ha) reverse-construct SSL-related session options here"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_a37e26e1","line":64,"range":{"start_line":64,"start_character":33,"end_line":64,"end_character":74},"in_reply_to":"bfb3d3c7_a37b8689","updated":"2019-05-31 15:08:22.000000000","message":"I\u0027m capturing both get_session and get_adapter under one try block since they are not really different from our perspective.","commit_id":"cebeb820068b3aa7f7a8a145888e226430987868"},{"author":{"_account_id":12860,"name":"Dongcan Ye","email":"yedongcan@yeah.net","username":"yedongcan"},"change_message_id":"edba8500d00a2156baf37629c3f34b2fd0c034b1","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            adapter \u003d keystone.get_adapter(\u0027swift\u0027, session\u003dSWIFT_SESSION,"},{"line_number":62,"context_line":"                                           **adapter_opts)"},{"line_number":63,"context_line":"        except Exception as exc:"},{"line_number":64,"context_line":"            raise utils.Error(_(\"Could not create an adapter to connect to \""},{"line_number":65,"context_line":"                                \"the object storage service: %s\") % exc)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        # TODO(pas-ha) reverse-construct SSL-related session options here"}],"source_content_type":"text/x-python","patch_set":1,"id":"bfb3d3c7_83c9a226","line":64,"range":{"start_line":64,"start_character":33,"end_line":64,"end_character":74},"in_reply_to":"bfb3d3c7_a37e26e1","updated":"2019-05-31 15:10:38.000000000","message":"Thanks for explain.","commit_id":"cebeb820068b3aa7f7a8a145888e226430987868"}]}