)]}'
{"neutron/cmd/ovn/migration_mtu.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5c7aeceb22a8efcd1b7021fb79fe4d8716abeca9","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"# map of network types to migrate and the difference in overhead size when"},{"line_number":23,"context_line":"# converted to Geneve."},{"line_number":24,"context_line":"NETWORK_TYPE_OVERHEAD \u003d {"},{"line_number":25,"context_line":"    \u0027vxlan\u0027: 8,"},{"line_number":26,"context_line":"    \u0027gre\u0027: 16,"},{"line_number":27,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_586fddfb","line":24,"updated":"2020-07-16 22:27:16.000000000","message":"My only nit is with the name, maybe if we just listed the overhead in the comment it would make it clear - Geneve:38 (in our usage), Vxlan:30, GRE:22.","commit_id":"eaad06f41c07967e66d238078170ccf6d928687a"},{"author":{"_account_id":13686,"name":"Frode Nordahl","email":"fnordahl@ubuntu.com","username":"fnordahl"},"change_message_id":"8b3c2b15c84b6e5db22cf194fd6152f2e0eb84a8","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"# map of network types to migrate and the difference in overhead size when"},{"line_number":23,"context_line":"# converted to Geneve."},{"line_number":24,"context_line":"NETWORK_TYPE_OVERHEAD \u003d {"},{"line_number":25,"context_line":"    \u0027vxlan\u0027: 8,"},{"line_number":26,"context_line":"    \u0027gre\u0027: 16,"},{"line_number":27,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_a17084dc","line":24,"in_reply_to":"bf51134e_586fddfb","updated":"2020-07-17 05:19:02.000000000","message":"I see what you mean, let\u0027s do both. Thx!","commit_id":"eaad06f41c07967e66d238078170ccf6d928687a"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"3f3ca19316dd06cbf59b469a90f7e229490e01d0","unresolved":false,"context_lines":[{"line_number":26,"context_line":"# converted to Geneve."},{"line_number":27,"context_line":"NETWORK_TYPE_OVERHEAD_DIFF \u003d {"},{"line_number":28,"context_line":"    \u0027vxlan\u0027: GENEVE_ENCAP_OVERHEAD - constants.VXLAN_ENCAP_OVERHEAD,"},{"line_number":29,"context_line":"    \u0027gre\u0027:   GENEVE_ENCAP_OVERHEAD - constants.GRE_ENCAP_OVERHEAD,"},{"line_number":30,"context_line":"}"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bf51134e_a79c145c","line":29,"updated":"2020-07-17 08:19:57.000000000","message":"pep8: E241 multiple spaces after \u0027:\u0027","commit_id":"8f6704560a62c6c32ccb785fc12621da328f34c2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6a56e725f682ac71133b283b9c7077d1eca34192","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# Overhead size of Geneve is configurable, use the recommended value"},{"line_number":24,"context_line":"GENEVE_ENCAP_OVERHEAD \u003d 38"},{"line_number":25,"context_line":"# map of network types to migrate and the difference in overhead size when"},{"line_number":26,"context_line":"# converted to Geneve."},{"line_number":27,"context_line":"NETWORK_TYPE_OVERHEAD_DIFF \u003d {"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_b3da8f48","line":24,"updated":"2020-07-17 14:43:07.000000000","message":"Yeah, the one in neutron-lib is just the minimum(30), 38 is only defined in devstack/lib/ovn_agent","commit_id":"fa0f807700ef49eb98f09d53bebcb558a08e3d3b"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3f56e5cc765a27aeed184f937b9493f0eb2d65a5","unresolved":false,"context_lines":[{"line_number":24,"context_line":"GENEVE_ENCAP_OVERHEAD \u003d 38"},{"line_number":25,"context_line":"# map of network types to migrate and the difference in overhead size when"},{"line_number":26,"context_line":"# converted to Geneve."},{"line_number":27,"context_line":"NETWORK_TYPE_OVERHEAD_DIFF \u003d {"},{"line_number":28,"context_line":"    \u0027vxlan\u0027: GENEVE_ENCAP_OVERHEAD - constants.VXLAN_ENCAP_OVERHEAD,"},{"line_number":29,"context_line":"    \u0027gre\u0027: GENEVE_ENCAP_OVERHEAD - constants.GRE_ENCAP_OVERHEAD,"},{"line_number":30,"context_line":"}"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_525df4ea","line":27,"updated":"2020-07-17 09:15:24.000000000","message":"+1","commit_id":"fa0f807700ef49eb98f09d53bebcb558a08e3d3b"}]}