YAML configs wiped out from /run/netplan

Bug #1764869 reported by Mathieu Trudel-Lapierre on 2018-04-17
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
netplan.io (Ubuntu)
Undecided
Unassigned
Xenial
Undecided
Unassigned
Artful
Undecided
Unassigned
nplan (Ubuntu)
Undecided
Unassigned
Xenial
Undecided
Unassigned
Artful
Undecided
Unassigned

Bug Description

[Impact]
Netboot users, and any scenario where YAML configuration is written to /run/netplan.

[Test case]
1) Run 'sudo netplan apply'
2) Verify that existing configuration *.yaml files in /run/netplan is not removed.

[Regression Potential]
/run/netplan is meant as a location for yaml configuration generated by other parts of the system, or by the boot processes, to be used and complement / override local config in /etc/netplan. Additional files are created in /run/netplan (such as wpa configuration when using wifi) which must be removed. Removal of the netplan configuration files may lead to incomplete configuration for network devices. This may cause a system to lose connectivity. Changes in network connectivity following "netplan apply" where configuration didn't otherwise change in /etc/netplan/*.yaml may indicate a regression.

---

Any .yaml file in /run/netplan are wiped out when 'netplan generate' is run. This is wrong, we might actually want to have .yaml files there for netplan configuration.

What we don't want, however, is for generated wpa-*.conf files to be left around: wpasupplicant configuration really does need to go, as it will be generated again by 'netplan generate'.

Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in netplan.io (Ubuntu):
status: New → Confirmed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package netplan.io - 0.35

---------------
netplan.io (0.35) bionic; urgency=medium

  * debian/postinst: fix version check for when to write breadcrumbs.
    (LP: #1756742)
  * bonds/bridges: Support specifying time-based values with "ms" suffix when
    the value should be in milliseconds; while keeping support for the previous
    behavior of handling values are pure seconds when no suffix is present.
    (LP: #1745597)
  * IPv6: accept-ra should default to being unset, so that the kernel default
    can be used. (LP: #1732002)
  * DHCPv4: add a "dhcp-identifier: mac" field that can be set to fix interop
    with Windows Server-based DHCP servers which don't support RFC 4361.
    (LP: #1738998)
  * docs, examples: reformat comments in examples, add standalone example files
  * debian/docs, debian/netplan.io.install: install doc and examples in the
    right locations.
  * netplan try: allow users to preview/test and approve network configuration
    changes, and automatically revert if they are not accepted. (LP: #1764824)
  * networkd: don't wipe out /run/netplan on generate: we do want to keep any
    YAML configurations in that directory, we just need to remove generated
    wpasupplicant configs. (LP: #1764869)
  * debian/control: add bash-completion to Build-Depends to make sure we do
    install completion files in the right location.

 -- Mathieu Trudel-Lapierre <email address hidden> Tue, 17 Apr 2018 21:49:21 -0400

Changed in netplan.io (Ubuntu):
status: Confirmed → Fix Released
Changed in nplan (Ubuntu):
status: New → In Progress
description: updated
Changed in nplan (Ubuntu):
status: In Progress → Fix Released
Changed in netplan.io (Ubuntu Xenial):
status: New → Invalid
Changed in netplan.io (Ubuntu Artful):
status: New → Invalid

Hello Mathieu, or anyone else affected,

Accepted nplan into artful-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nplan/0.32~17.10.4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-artful to verification-done-artful. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-artful. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in nplan (Ubuntu Artful):
status: New → Fix Committed
tags: added: verification-needed verification-needed-artful
Brian Murray (brian-murray) wrote :

Hello Mathieu, or anyone else affected,

Accepted nplan into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nplan/0.32~16.04.5 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in nplan (Ubuntu Xenial):
status: New → Fix Committed
tags: added: verification-needed-xenial

Verification-done for artful: nplan 0.32~17.10.4
Verification-done for xenial: nplan 0.32~16.04.5

Verified that a random file "/run/netplan/toto.yaml" present on the system is removed when running "netplan apply" with the previous version of nplan, and remains in place when the new version is installed: additionally, wpa-* files are removed as they should by the process.

ubuntu@heroic-ibex:~$ sudo mkdir /run/netplan
ubuntu@heroic-ibex:~$ sudo touch /run/netplan/wpa-toto.conf
ubuntu@heroic-ibex:~$ sudo touch /run/netplan/someotherfile
ubuntu@heroic-ibex:~$ sudo touch /run/netplan/toto.yaml
ubuntu@heroic-ibex:~$ ls -l /run/netplan
total 0
-rw-r--r-- 1 root root 0 May 15 15:22 someotherfile
-rw-r--r-- 1 root root 0 May 15 15:22 toto.yaml
-rw-r--r-- 1 root root 0 May 15 15:21 wpa-toto.conf
ubuntu@heroic-ibex:~$ sudo netplan apply
ubuntu@heroic-ibex:~$ ls -l /run/netplan
total 0
-rw-r--r-- 1 root root 0 May 15 15:22 someotherfile
-rw-r--r-- 1 root root 0 May 15 15:22 toto.yaml
ubuntu@heroic-ibex:~$

tags: added: verification-done-artful verification-done-xenial
removed: verification-needed verification-needed-artful verification-needed-xenial
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers