)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"0b8be1cebd07c6f8f04dea694344d16638e647ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"224f0a6f_b1c86f07","updated":"2023-07-12 15:50:49.000000000","message":"Looks good - we have something very similar under test. Some small tweaks needed in the unit tests now that timeout is actually passed.","commit_id":"7bd384305fb091aeefb464d83fd5c05b33ec6cf2"},{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"40fe59e15e8eb3a496100473ebd3e93d22713125","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"07829bf1_0132b99d","updated":"2023-07-14 08:08:06.000000000","message":"Looks good to me!","commit_id":"2ef10f5916a6d657820adad84a7aefe73680a023"}],"sushy/connector.py":[{"author":{"_account_id":17669,"name":"Doug Szumski","email":"doug@stackhpc.com","username":"DougSzumski"},"change_message_id":"0b8be1cebd07c6f8f04dea694344d16638e647ce","unresolved":true,"context_lines":[{"line_number":132,"context_line":"        :raises: HTTPError"},{"line_number":133,"context_line":"        \"\"\""},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"        if extra_session_req_kwargs and timeout in extra_session_req_kwargs:"},{"line_number":136,"context_line":"            request_timeout \u003d extra_session_req_kwargs.pop(\u0027timeout\u0027)"},{"line_number":137,"context_line":"        else:"},{"line_number":138,"context_line":"            request_timeout \u003d timeout"}],"source_content_type":"text/x-python","patch_set":2,"id":"59e2623d_d74dc052","line":135,"updated":"2023-07-12 15:50:49.000000000","message":"I suppose that this is just being defensive, or are there known cases when timeout is passed via extra_session_req_kwargs?","commit_id":"7bd384305fb091aeefb464d83fd5c05b33ec6cf2"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"cc275d84d031554feb880703a5f1c51c1d30e4a2","unresolved":true,"context_lines":[{"line_number":132,"context_line":"        :raises: HTTPError"},{"line_number":133,"context_line":"        \"\"\""},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"        if extra_session_req_kwargs and timeout in extra_session_req_kwargs:"},{"line_number":136,"context_line":"            request_timeout \u003d extra_session_req_kwargs.pop(\u0027timeout\u0027)"},{"line_number":137,"context_line":"        else:"},{"line_number":138,"context_line":"            request_timeout \u003d timeout"}],"source_content_type":"text/x-python","patch_set":2,"id":"5c901b70_2f0f630c","line":135,"in_reply_to":"59e2623d_d74dc052","updated":"2023-07-12 20:57:14.000000000","message":"Sushy is a public library, not just intended for OpenStack use. I want to defend against users having their API changed -- this will help make this change more safe to backport.","commit_id":"7bd384305fb091aeefb464d83fd5c05b33ec6cf2"}]}