)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"8d9c44e139055669d3c08699ed8084b103f8989e","unresolved":true,"context_lines":[{"line_number":13,"context_line":"of the argument name. This is based on customer\u0027s feedback."},{"line_number":14,"context_line":"I am keeping backward compatibility for setup-mtu-t1 argument"},{"line_number":15,"context_line":"in order not to break any existing tools that might depend on it."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"If you feel that this change might bring more headache than"},{"line_number":18,"context_line":"benefit, then -1 | -2 this patch. Thank you"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I939b21fa998c80cf921efeae3e8fa8c2b4ef4f50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a8758cf1_1d2cf306","line":18,"range":{"start_line":16,"start_character":0,"end_line":18,"end_character":43},"updated":"2022-12-02 17:35:26.000000000","message":"I\u0027d remove these lines as it applies to all patches we send :)","commit_id":"ca2b6d5c784ebf602c49b3ea737fd8982988fdc4"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"588baa8aaaec6c1a9a286259e8a7db03c3c744e9","unresolved":false,"context_lines":[{"line_number":13,"context_line":"of the argument name. This is based on customer\u0027s feedback."},{"line_number":14,"context_line":"I am keeping backward compatibility for setup-mtu-t1 argument"},{"line_number":15,"context_line":"in order not to break any existing tools that might depend on it."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"If you feel that this change might bring more headache than"},{"line_number":18,"context_line":"benefit, then -1 | -2 this patch. Thank you"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I939b21fa998c80cf921efeae3e8fa8c2b4ef4f50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"086eae6c_b63272ea","line":18,"range":{"start_line":16,"start_character":0,"end_line":18,"end_character":43},"in_reply_to":"a8758cf1_1d2cf306","updated":"2022-12-10 03:07:39.000000000","message":"Done","commit_id":"ca2b6d5c784ebf602c49b3ea737fd8982988fdc4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"3f7794384b823fca1bd1df78a5514157ac1076a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"327d7504_14536611","updated":"2022-11-10 09:12:12.000000000","message":"Thank you Miro for this patch.\n\nJust a small comment :)","commit_id":"ca2b6d5c784ebf602c49b3ea737fd8982988fdc4"}],"doc/source/ovn/migration.rst":[{"author":{"_account_id":34118,"name":"Arnau Verdaguer","email":"averdagu@redhat.com","username":"averdaguer"},"change_message_id":"3f7794384b823fca1bd1df78a5514157ac1076a3","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Subsequent steps start the migration via Ansible."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Plan for a 24-hour wait after the reduce-dhcp-t1 step to allow VMs to catch up"},{"line_number":25,"context_line":"with the new MTU size from DHCP. The default neutron ML2/OVS configuration has a"},{"line_number":26,"context_line":"dhcp_lease_duration of 86400 seconds (24h)."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Also, if there are instances using static IP assignment, the administrator"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ac3455f3_4c7525ee","line":25,"range":{"start_line":25,"start_character":31,"end_line":25,"end_character":32},"updated":"2022-11-10 09:12:12.000000000","message":"Maybe add here that even if the DHCP sets a different MTU value, the final decision on how to handle this new value falls into the OS.","commit_id":"ca2b6d5c784ebf602c49b3ea737fd8982988fdc4"},{"author":{"_account_id":34271,"name":"Miro Tomaska","display_name":"Miro Tomaska","email":"mtomaska@redhat.com","username":"mtomaska"},"change_message_id":"588baa8aaaec6c1a9a286259e8a7db03c3c744e9","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Subsequent steps start the migration via Ansible."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Plan for a 24-hour wait after the reduce-dhcp-t1 step to allow VMs to catch up"},{"line_number":25,"context_line":"with the new MTU size from DHCP. The default neutron ML2/OVS configuration has a"},{"line_number":26,"context_line":"dhcp_lease_duration of 86400 seconds (24h)."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Also, if there are instances using static IP assignment, the administrator"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fd0de17b_6d5d2ed9","line":25,"range":{"start_line":25,"start_character":31,"end_line":25,"end_character":32},"in_reply_to":"ac3455f3_4c7525ee","updated":"2022-12-10 03:07:39.000000000","message":"Thanks, I thought about it and I think I am just going to leave this as is. I think that explaining it further might either confuse someone who is not familiar or be too much unnecessary information for someone who is very knowledgeable","commit_id":"ca2b6d5c784ebf602c49b3ea737fd8982988fdc4"}]}