)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"9d638dcdbd9ee3211c40f7cf1adb26f751f8be04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6bd17154_c4d372c2","updated":"2024-04-29 08:35:38.000000000","message":"LGTM, tested locally.\nThanks Takashi !","commit_id":"8ded34278a7f656d618b3f93da68fbbf4ae1769c"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"abfb68bb14c3aaba1e77679fcb9e7895fcc57f91","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b537a3d9_6509aad6","updated":"2024-04-30 11:15:22.000000000","message":"Looks good, thank you!","commit_id":"300b8c8262d8dcdca7a8d753c3fb8a22cae02d40"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"861c533c231f98e17035ae3ffd9446b356a24fd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"997d53cc_51124e7e","updated":"2024-04-30 11:15:02.000000000","message":"Thanks!","commit_id":"300b8c8262d8dcdca7a8d753c3fb8a22cae02d40"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"bb75b1c57ca663243f91803a5415fd0fe2ee2346","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5a2edbbe_009c0eac","updated":"2024-04-30 12:38:42.000000000","message":"recheck `urllib.error.URLError: \u003curlopen error [Errno 110] Connection timed out\u003e` is probably an infra problem","commit_id":"300b8c8262d8dcdca7a8d753c3fb8a22cae02d40"}],"glanceclient/tests/utils.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"f1fdf753154fd12954bdb6edd64ffea88746679f","unresolved":true,"context_lines":[{"line_number":113,"context_line":"        self.body \u003d body"},{"line_number":114,"context_line":"        self.reason \u003d reason"},{"line_number":115,"context_line":"        self.version \u003d version"},{"line_number":116,"context_line":"        self.headers \u003d EmailMessage()"},{"line_number":117,"context_line":"        for header_key in headers:"},{"line_number":118,"context_line":"            self.headers[header_key] \u003d headers[header_key]"},{"line_number":119,"context_line":"        self.headers[\u0027x-openstack-request-id\u0027] \u003d \u0027req-1234\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"92d651d3_9259219d","line":116,"range":{"start_line":116,"start_character":23,"end_line":116,"end_character":35},"updated":"2024-04-29 14:18:11.000000000","message":"So it\u0027s probably the right fix, but this deserves a thorough explanation. Why do I suddenly end up with en EmailMessage here? As a reader this feels very confusing and deserves a comment.","commit_id":"8ded34278a7f656d618b3f93da68fbbf4ae1769c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"56db8395085def2ed6e222e489cafea5d50812d2","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        self.body \u003d body"},{"line_number":114,"context_line":"        self.reason \u003d reason"},{"line_number":115,"context_line":"        self.version \u003d version"},{"line_number":116,"context_line":"        self.headers \u003d EmailMessage()"},{"line_number":117,"context_line":"        for header_key in headers:"},{"line_number":118,"context_line":"            self.headers[header_key] \u003d headers[header_key]"},{"line_number":119,"context_line":"        self.headers[\u0027x-openstack-request-id\u0027] \u003d \u0027req-1234\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a57644f_d7fcb7f0","line":116,"range":{"start_line":116,"start_character":23,"end_line":116,"end_character":35},"in_reply_to":"92d651d3_9259219d","updated":"2024-04-29 14:58:18.000000000","message":"I\u0027ve added a comment and I hope it provides the required context.\n\nTo detail it, requests-mock started extracting cookies since 0.7.0[1] and this implementation uses requests library\u0027s implementation which requires http.client.HTTPResponse interface. \n\n[1] https://github.com/jamielennox/requests-mock/commit/fe37c6cc3ae3f838cb048e4fdc11b92d58d3c7bd\n\nIt didn\u0027t cause an immediate problem because the headers dict was always converted to compat._FakeHTTPMessage, which provides the required interface, but that casting was removed in 0.12.0.","commit_id":"8ded34278a7f656d618b3f93da68fbbf4ae1769c"}]}