{"action":"created","issue":{"url":"https://api.github.com/repos/CanonicalLtd/netplan/issues/61","repository_url":"https://api.github.com/repos/CanonicalLtd/netplan","labels_url":"https://api.github.com/repos/CanonicalLtd/netplan/issues/61/labels{/name}","comments_url":"https://api.github.com/repos/CanonicalLtd/netplan/issues/61/comments","events_url":"https://api.github.com/repos/CanonicalLtd/netplan/issues/61/events","html_url":"https://github.com/CanonicalLtd/netplan/pull/61","id":387121304,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM1NjgwNDUz","number":61,"title":"Add support to override networkd UseMTU setting","user":{"login":"shashankram","id":21697719,"node_id":"MDQ6VXNlcjIxNjk3NzE5","avatar_url":"https://avatars0.githubusercontent.com/u/21697719?v=4","gravatar_id":"","url":"https://api.github.com/users/shashankram","html_url":"https://github.com/shashankram","followers_url":"https://api.github.com/users/shashankram/followers","following_url":"https://api.github.com/users/shashankram/following{/other_user}","gists_url":"https://api.github.com/users/shashankram/gists{/gist_id}","starred_url":"https://api.github.com/users/shashankram/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shashankram/subscriptions","organizations_url":"https://api.github.com/users/shashankram/orgs","repos_url":"https://api.github.com/users/shashankram/repos","events_url":"https://api.github.com/users/shashankram/events{/privacy}","received_events_url":"https://api.github.com/users/shashankram/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":32,"created_at":"2018-12-04T04:59:21Z","updated_at":"2018-12-04T21:49:37Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/CanonicalLtd/netplan/pulls/61","html_url":"https://github.com/CanonicalLtd/netplan/pull/61","diff_url":"https://github.com/CanonicalLtd/netplan/pull/61.diff","patch_url":"https://github.com/CanonicalLtd/netplan/pull/61.patch"},"body":"Currently, there is no way to override networkd's\r\nUseMTU flag via Netplan. This is useful to have\r\nas a dhcp-override option similar to the other\r\ndhcp-override options.\r\nIf the 'use-mtu' key is set to false in the netplan\r\nconfig file, then set 'UseMTU' as false in networkd's\r\nconfig.\r\n\r\nSigned-off-by: Shashank Ram <shashank08@gmail.com>\r\n\r\n\r\n## Description\r\n\r\n\r\n## Checklist\r\n\r\n- [x] Runs `make check` successfully.\r\n- [ ] Retains 100% code coverage (`make check-coverage`).\r\n- [x] New/changed keys in YAML format are documented.\r\n- [x] \\(Optional\\) Closes an open bug in Launchpad.\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/CanonicalLtd/netplan/issues/comments/444272051","html_url":"https://github.com/CanonicalLtd/netplan/pull/61#issuecomment-444272051","issue_url":"https://api.github.com/repos/CanonicalLtd/netplan/issues/61","id":444272051,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0NDI3MjA1MQ==","user":{"login":"cyphermox","id":119845,"node_id":"MDQ6VXNlcjExOTg0NQ==","avatar_url":"https://avatars0.githubusercontent.com/u/119845?v=4","gravatar_id":"","url":"https://api.github.com/users/cyphermox","html_url":"https://github.com/cyphermox","followers_url":"https://api.github.com/users/cyphermox/followers","following_url":"https://api.github.com/users/cyphermox/following{/other_user}","gists_url":"https://api.github.com/users/cyphermox/gists{/gist_id}","starred_url":"https://api.github.com/users/cyphermox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cyphermox/subscriptions","organizations_url":"https://api.github.com/users/cyphermox/orgs","repos_url":"https://api.github.com/users/cyphermox/repos","events_url":"https://api.github.com/users/cyphermox/events{/privacy}","received_events_url":"https://api.github.com/users/cyphermox/received_events","type":"User","site_admin":false},"created_at":"2018-12-04T21:49:37Z","updated_at":"2018-12-04T21:49:37Z","author_association":"COLLABORATOR","body":"@ssahani I get what you're saying:\r\n\r\n`If <varname>MTUBytes=</varname> is set, then this setting is ignored.`\r\n\r\nIt's already broken in this way in netplan, since we do set MTUBytes= in some cases already. Let's address this in a separate patch adding a warning (I don't think we should error out, since that would break existing valid configs that have been in use)."}}
{"actor":{"display_login":"cyphermox"}}