)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c5c1d426b4b385d6a6c8ecaab6666c07c558f782","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a63a5d86_6f9635db","updated":"2023-08-28 06:47:14.000000000","message":"Thanks","commit_id":"a6d3b97ec44fe1f79ae0795548c1ebd7577656d0"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"0ba75f6b313227f575a99280052212b326e7f8ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b0ce9a22_e6f12e3a","updated":"2023-10-31 14:56:44.000000000","message":"I think it\u0027s an improvement. Additional wording revamp may be helpful as well but this doesn\u0027t make this addition wrong.","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d979cec7146ee11c4f773a10cd891174b03836f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c649385c_f8427093","updated":"2023-11-03 18:53:50.000000000","message":"Tried to merge https://review.opendev.org/c/openstack/neutron/+/894620 into here so we get something merged.","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"051cc322a6b0851f414851193297d5a8bac9ab22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d6821cc1_c1f07126","updated":"2023-09-21 12:32:09.000000000","message":"feel free to amend this patch if you think that that is needed, but given the latest comments in the bug report, I think that the current text is mostly correct still and IMO it will be better to have a warning that is a bit overcautious than the other way round. or none at all, like the current status","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"9985960ed578f0f9a3e5fcd8c6cf04b0d234143d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"273d1f50_e3440361","updated":"2023-11-10 12:38:46.000000000","message":"fine for me to get this moving","commit_id":"e4542bca8019b4f4e096ec33b296e1d2f59d2479"}],"doc/source/ovn/gaps.rst":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3cbe54237d59c41b16b883ea39a8c545fe06c5ae","unresolved":true,"context_lines":[{"line_number":76,"context_line":"  OVN does not fragment IPv4 packets when the MTU of the target network is"},{"line_number":77,"context_line":"  smaller than the MTU of the source network. Instead affected packets will"},{"line_number":78,"context_line":"  be silently dropped. OVN will also not generate an ICMP response for"},{"line_number":79,"context_line":"  packets that have the DF bit set, making path MTU discovery impossible."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"References"},{"line_number":82,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"b61e213e_bf988a78","line":79,"updated":"2023-08-29 18:07:06.000000000","message":"In this case we actually do have a bug that I believe will lead to a fix, so think it\u0027s useful to add as a reference.","commit_id":"a6d3b97ec44fe1f79ae0795548c1ebd7577656d0"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"ed3503f3e1837c651367d137f37eb9a05e599748","unresolved":false,"context_lines":[{"line_number":76,"context_line":"  OVN does not fragment IPv4 packets when the MTU of the target network is"},{"line_number":77,"context_line":"  smaller than the MTU of the source network. Instead affected packets will"},{"line_number":78,"context_line":"  be silently dropped. OVN will also not generate an ICMP response for"},{"line_number":79,"context_line":"  packets that have the DF bit set, making path MTU discovery impossible."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"References"},{"line_number":82,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"7ca018b4_454e6ab5","line":79,"in_reply_to":"b61e213e_bf988a78","updated":"2023-08-31 09:28:26.000000000","message":"Agreed, done","commit_id":"a6d3b97ec44fe1f79ae0795548c1ebd7577656d0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9338b1a243b5c7a0eabca84f78758a33b0a9659b","unresolved":true,"context_lines":[{"line_number":79,"context_line":"  packets that have the DF bit set, making path MTU discovery impossible."},{"line_number":80,"context_line":"  There is an open bug report to fix this issue [9]_."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"References"},{"line_number":83,"context_line":"----------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":".. [1] https://github.com/ovn-org/ovn/blob/master/TODO.rst"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fcf91969_89b0524f","line":82,"updated":"2023-09-14 15:16:11.000000000","message":"It is working in one direction: when MTU external \u003c MTU internal. In this case, if the Neutron config flag \"ovn_emit_need_to_frag\" is set, the GW LRP receives the \"config:gateway_mtu:VALUE\" configuration and handles correctly the MTU in both directions.\n\nOpen BZs to core OVN:\n* https://bugzilla.redhat.com/show_bug.cgi?id\u003d2238494: E/W routed traffic does not handle different network MTU sizes\n* https://bugzilla.redhat.com/show_bug.cgi?id\u003d2238969: N/S routed traffic does not handle different network MTU sizes, if MTU external \u003e MTU internal","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"396fddf528c6399821a0ef8380304363f5a20173","unresolved":true,"context_lines":[{"line_number":79,"context_line":"  packets that have the DF bit set, making path MTU discovery impossible."},{"line_number":80,"context_line":"  There is an open bug report to fix this issue [9]_."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"References"},{"line_number":83,"context_line":"----------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":".. [1] https://github.com/ovn-org/ovn/blob/master/TODO.rst"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dbd9611c_98ed6701","line":82,"in_reply_to":"30d364a2_d687136b","updated":"2024-04-24 10:42:33.000000000","message":"this is generally required for using jumbo frames internellay and normal 1500 byte mtu on the wan side\n\nthe workaround for this is to actully use a dataceneter router to do the mediation and refagmentation upstream neutron but it would be better if ovn could also supprot this long term.","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"564d95cbd13abe87a79cfd3a4a7340e0bf9a8531","unresolved":true,"context_lines":[{"line_number":79,"context_line":"  packets that have the DF bit set, making path MTU discovery impossible."},{"line_number":80,"context_line":"  There is an open bug report to fix this issue [9]_."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"References"},{"line_number":83,"context_line":"----------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":".. [1] https://github.com/ovn-org/ovn/blob/master/TODO.rst"}],"source_content_type":"text/x-rst","patch_set":2,"id":"30d364a2_d687136b","line":82,"in_reply_to":"ad156cec_150e7085","updated":"2023-12-05 06:47:53.000000000","message":"Which specific info are you talking about? I could not reproduce any direction to work, so I can\u0027t really mention that. Not sure if it is worth mentioning BZ links. Feel free to update the patch though if you want.","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"bcf9aacff3d03c0c817afdc747f6ce4e8dc3ac91","unresolved":true,"context_lines":[{"line_number":79,"context_line":"  packets that have the DF bit set, making path MTU discovery impossible."},{"line_number":80,"context_line":"  There is an open bug report to fix this issue [9]_."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"References"},{"line_number":83,"context_line":"----------"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":".. [1] https://github.com/ovn-org/ovn/blob/master/TODO.rst"}],"source_content_type":"text/x-rst","patch_set":2,"id":"ad156cec_150e7085","line":82,"in_reply_to":"fcf91969_89b0524f","updated":"2023-12-01 11:03:11.000000000","message":"@frickler: do You want to add that info to this doc?","commit_id":"08a167df2530f76f32f63c379159c9cdf7a24053"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"396fddf528c6399821a0ef8380304363f5a20173","unresolved":true,"context_lines":[{"line_number":28,"context_line":"  routers have this capability implemented in [4]_ and we have an open RFE to"},{"line_number":29,"context_line":"  fill this gap [5]_."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"* East/West Fragmentation"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"  The core OVN implementation does not support east/west fragmentation. There is"},{"line_number":34,"context_line":"  no known production use-case for this feature hence we don\u0027t even have an RFE"}],"source_content_type":"text/x-rst","patch_set":4,"id":"e447a64d_bdd026c9","side":"PARENT","line":31,"updated":"2024-04-24 10:42:33.000000000","message":"so ya this was not ture\n\nthe usecase fo neutron router interconnecting internal and external netowrks with diffent mtus was well know and it was dicuss in the past before neutron gained mtu supprot when my team at intel wanted to add an openflow based router for ml2/ovs\n\nthe common case where this would differ for east west taffic si whn you are interconencitn vlan and geneve network via a neutron router.","commit_id":"0e5c91c4996418147cc329f60ce8b0d0eb539f72"}]}
