{"action":"created","issue":{"url":"https://api.github.com/repos/Igalia/snabb/issues/488","repository_url":"https://api.github.com/repos/Igalia/snabb","labels_url":"https://api.github.com/repos/Igalia/snabb/issues/488/labels{/name}","comments_url":"https://api.github.com/repos/Igalia/snabb/issues/488/comments","events_url":"https://api.github.com/repos/Igalia/snabb/issues/488/events","html_url":"https://github.com/Igalia/snabb/pull/488","id":183190073,"number":488,"title":"set interface mtu based on lwaftr v4/v6 mtu","user":{"login":"mwiget","id":3111088,"avatar_url":"https://avatars.githubusercontent.com/u/3111088?v=3","gravatar_id":"","url":"https://api.github.com/users/mwiget","html_url":"https://github.com/mwiget","followers_url":"https://api.github.com/users/mwiget/followers","following_url":"https://api.github.com/users/mwiget/following{/other_user}","gists_url":"https://api.github.com/users/mwiget/gists{/gist_id}","starred_url":"https://api.github.com/users/mwiget/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mwiget/subscriptions","organizations_url":"https://api.github.com/users/mwiget/orgs","repos_url":"https://api.github.com/users/mwiget/repos","events_url":"https://api.github.com/users/mwiget/events{/privacy}","received_events_url":"https://api.github.com/users/mwiget/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-10-15T05:25:50Z","updated_at":"2016-10-15T09:16:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Igalia/snabb/pulls/488","html_url":"https://github.com/Igalia/snabb/pull/488","diff_url":"https://github.com/Igalia/snabb/pull/488.diff","patch_url":"https://github.com/Igalia/snabb/pull/488.patch"},"body":"Fix for https://github.com/Igalia/snabb/issues/487\r\n\r\nIt calculates the MTU for the interface based on the ipv4_mtu and ipv6_mtu, by taking the numerically larger one and add 14 to cover the size of the ethernet header. The change will also print out the mtu to STDOUT together with the interface name.\r\n\r\nFixing README files for snabbvmx ipv4_interface and ipv6_interface settings.\r\n"},"comment":{"url":"https://api.github.com/repos/Igalia/snabb/issues/comments/253972646","html_url":"https://github.com/Igalia/snabb/pull/488#issuecomment-253972646","issue_url":"https://api.github.com/repos/Igalia/snabb/issues/488","id":253972646,"user":{"login":"chrgraf","id":22005482,"avatar_url":"https://avatars.githubusercontent.com/u/22005482?v=3","gravatar_id":"","url":"https://api.github.com/users/chrgraf","html_url":"https://github.com/chrgraf","followers_url":"https://api.github.com/users/chrgraf/followers","following_url":"https://api.github.com/users/chrgraf/following{/other_user}","gists_url":"https://api.github.com/users/chrgraf/gists{/gist_id}","starred_url":"https://api.github.com/users/chrgraf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrgraf/subscriptions","organizations_url":"https://api.github.com/users/chrgraf/orgs","repos_url":"https://api.github.com/users/chrgraf/repos","events_url":"https://api.github.com/users/chrgraf/events{/privacy}","received_events_url":"https://api.github.com/users/chrgraf/received_events","type":"User","site_admin":false},"created_at":"2016-10-15T09:16:49Z","updated_at":"2016-10-15T09:16:49Z","body":"Hi Marcel,\n\nI have meant the [edit interfaces] mtu or even better by issuing a operational command show interface ge000 | match mtu.\n\nHaving two places to configure mtu (edit interface and edit softwire] does create unnecessary conflicts.\n\nIdeally if the [edit softwire] has no mtu configured it would be good if it would be derived from the ge000 mtu.\n\nMany thanks\n\nChristian\n\nVon meinem Windows Phone gesendet\n________________________________\nVon: Marcel Wiget<mailto:notifications@github.com>\nGesendet: ‎15.‎10.‎2016 09:54\nAn: Igalia/snabb<mailto:snabb@noreply.github.com>\nCc: Christian Graf<mailto:cgraf@juniper.net>; Mention<mailto:mention@noreply.github.com>\nBetreff: Re: [Igalia/snabb] set interface mtu based on lwaftr v4/v6 mtu (#488)\n\n\n@chrgraf<https://github.com/chrgraf>: There is no mtu other than the tunnel-payload-mtu and tunnel-path-mru, both are mandatory. If no br-instance is configured, then I set the MTU to DEFAULT_MTU = 9500, which is used in transparent mode, so all packets will be sent to the vMX 0/0/x interface, where the MTU from the vMX will be applied.\n\n—\nYou are receiving this because you were mentioned.\nReply to this email directly, view it on GitHub<https://github.com/Igalia/snabb/pull/488#issuecomment-253969428>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AU_G6v1v7SmAI4jy3OcWBlPwKAyoWlmkks5q0IbHgaJpZM4KXngU>.\n"}}
{"actor":{"display_login":"chrgraf"}}