The first crash could be fixed by handling that case inside libnetplan. The "ipvX.method" defaults to "disabled" and it will be added to the "passthrough" section. At some point post-merging we'd need to check if there are addresses in the list and if "ipvX.method" is in the passthrough section and remove it (or just omit it when writing the passthrough to the keyfile).
We also could do it during the merging. There are 2 cases to consider: the existing configuration is read before or after the new YAML. So when we parse the "addresses" list we'd need to see if "ipvX.method: disabled" is present in the passthrough and when we parse the passthrough and find the "ipvX.method: disabled" we need to see if there is something in the addresses list. None of this solutions sounds great through... we probably shouldn't be involving the passthrough keys in the libnetplan logic... -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/2088459 Title: Problems with NetworkManager and merging of existing configuration To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/netplan.io/+bug/2088459/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
