)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"24d927a6c77d7fb9de8970ed9bda31e28cc6102f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9e984c32_7a8de68f","updated":"2022-11-22 18:23:42.000000000","message":"This patch should start on Xena and be backported from there back to Ussuri.\nIt also needs a release note.\n\nIdeally we would add test coverage to catch this issue and show that the patch resolves it. I would not block the patch for the test coverage however.","commit_id":"dda3c31465683f5f64403b75d7d599db17af6e93"},{"author":{"_account_id":32927,"name":"mitya-eremeev-2","display_name":"Mitya Eremeev","email":"mitossvyaz@mail.ru","username":"mitya-eremeev-2"},"change_message_id":"33552763dcbb989c927a0f46026ebb304fd11ee0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"66a08f17_ea933361","in_reply_to":"9e984c32_7a8de68f","updated":"2022-11-28 21:23:13.000000000","message":"I made patch in Xena:\nhttps://review.opendev.org/c/openstack/designate/+/865843\nI\u0027ll added unit test, RN will do later.\nPlease review.\nIf it\u0027s OK I\u0027ll cherry pick it to Ussuri - Wallaby.","commit_id":"dda3c31465683f5f64403b75d7d599db17af6e93"}],"designate/dnsutils.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"24d927a6c77d7fb9de8970ed9bda31e28cc6102f","unresolved":true,"context_lines":[{"line_number":356,"context_line":""},{"line_number":357,"context_line":"            xfr \u003d dns.query.xfr(srv[\u0027host\u0027], zone_name, relativize\u003dFalse,"},{"line_number":358,"context_line":"                                timeout\u003d1, port\u003dsrv[\u0027port\u0027], source\u003dsource,"},{"line_number":359,"context_line":"                                lifetime\u003d60)"},{"line_number":360,"context_line":"            raw_zone \u003d dns.zone.from_xfr(xfr, relativize\u003dFalse)"},{"line_number":361,"context_line":"            break"},{"line_number":362,"context_line":"        except eventlet.Timeout as t:"}],"source_content_type":"text/x-python","patch_set":3,"id":"1c01ccc0_72614a2d","line":359,"updated":"2022-11-22 18:23:42.000000000","message":"I have a concern about this value. Before, when it was None, there was no timeout. Setting this to 60 may impact anyone that has very large zones and/or a slow network. It means it must transfer in sixty seconds.\nI would prefer to set this to something high (a day?) or make it configurable.","commit_id":"dda3c31465683f5f64403b75d7d599db17af6e93"}]}