Do not include NetworkManager support for legacy network configuration files by in new installations.
After a week, there are no votes waiting for additional votes.
-1
I don't think we're ready for this Change without the proposers making cloud-init read+write NM keyfiles.
I agree. I don't think we're ready for this change yet. :pensive: -1
See https://bugzilla.redhat.com/show_bug.cgi?id=2014701
Metadata Update from @sgallagh: - Issue tagged with: meeting
I don't want us to just say "no". This is equivalent to asking NM maintainers to care for 130K lines of crufty code, and we can't really do that. I think it is OK to ask them to delay the proposal to allow necessary adjustments in other places though.
I understand that cloud-init is a very important use case and we should take compat with cloud-init into account. cloud-init is written in Python (https://git.launchpad.net/cloud-init/tree/). I think it should be totally feasible to create a patch for cloud-init to support creation of the new NM format. Looking at https://git.launchpad.net/cloud-init/tree/tests/unittests/test_net.py, I see that the code renders the same config in a couple of formats: ifcfg, systemd.network, Debian's interfaces. Adding one more format shouldn't be hard.
So if we can scope some plan for cloud-init support, I'd suggest changing the change to deprecate the support in F36, and move the actual removal to F37, and plan to update cloud-init before that.
@lkundrak, @till, @thaller, @acabral: how much work would it be to add support to cloud-init and who could do this work?
In today's meeting, FESCo approved this proposal:
change is approved. cloud-init will gain requires for the new subpackage, change owners to communicate in https://bugzilla.redhat.com/show_bug.cgi?id=2014701 wrt complete removal time frame. complete removal of the subpackage/code will be coordinated by another change in the future (+8, 0, -0)
Metadata Update from @bcotton: - Issue untagged with: meeting - Issue tagged with: pending announcement
Thank you for approving the change. We are happy to support cloud-init developers to add support for keyfiles. Identifying issues like these is a good outcome for us for the change proposal.
@lkundrak has submitted a pull request to cloud-init upstream to support NetworkManager properly: https://github.com/canonical/cloud-init/pull/1224
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/F4ADFMQD37N6ALAWTMWMXHI5QOVNQFKU/
Metadata Update from @tstellar: - Issue close_status updated to: Accepted - Issue status updated to: Closed (was: Open)
Login to comment on this ticket.