#2732 F36 Change: No ifcfg by default
Closed: Accepted a year ago by tstellar. Opened a year ago by bcotton.

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.


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

Metadata Update from @sgallagh:
- Issue tagged with: meeting

a year ago

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

a year ago

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

Metadata Update from @tstellar:
- Issue close_status updated to: Accepted
- Issue status updated to: Closed (was: Open)

a year ago

Login to comment on this ticket.