openvswitch-switch isn't installed if netplan schema defines an ovs bridge

Bug #1891608 reported by Björn Tillenius on 2020-08-14
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
curtin
Undecided
Ryan Harper
curtin (Ubuntu)
Undecided
Unassigned
Xenial
Undecided
Unassigned
Bionic
Undecided
Unassigned
Focal
Undecided
Unassigned
Groovy
Undecided
Unassigned

Bug Description

If the network config defines an openvswitch bridge, curtin doesn't install the openvswitch-switch package, which is needed. For examle, a schema like this:

  network:
    version: 2
    bridges:
      br-empty:
        openvswitch: {}

Looking at the current code in curtin, it only handles the case where's there's a top-level 'openvswitch' section, or the renderer is 'openvswitch' (which was the previous proposal for openvswitch support in netplan)

Related branches

Revision history for this message
Ryan Harper (raharper) wrote :

Yeah, the implementation changed after curtin landed this.

Changed in curtin:
status: New → Confirmed
Ryan Harper (raharper) on 2020-08-14
Changed in curtin:
assignee: nobody → Ryan Harper (raharper)
status: Confirmed → In Progress
Revision history for this message
Server Team CI bot (server-team-bot) wrote :

This bug is fixed with commit 0f6a06ba to curtin on branch master.
To view that commit see the following URL:
https://git.launchpad.net/curtin/commit/?id=0f6a06ba

Changed in curtin:
status: In Progress → Fix Committed
Revision history for this message
Ryan Harper (raharper) wrote :

This bug was fixed in the package curtin - 20.1-20-g1304d3ea-0ubuntu1

---------------
curtin (20.1-20-g1304d3ea-0ubuntu1) groovy; urgency=medium

  * New upstream snapshot.
    - curthooks: uefi_find_grub_device_ids handle type:mount without path
      (LP: #1892242)
    - netplan openvswitch yaml changed (LP: #1891608)
    - tools/curtainer: do not wait for snapd.seeded.service
    - tools/curtainer: enable using ubuntu-minimal images
    - vmtests: add Groovy [Paride Legovini]
    - Drop the Eoan vmtests (EOL) [Paride Legovini]
    - tools: rename remove-vmtest-release to vmtest-remove-release
    - Snooze the tests failing because of LP: #1861941 for two more months
      [Paride Legovini]
    - LP: #1671951 is Fix Released => Drop the PPA [Paride Legovini]
    - swaps: handle swapfiles on btrfs (LP: #1884161)
    - curtainer: fail is masking of zfs-mount or zfs-share fails
      [Paride Legovini]
    - multipath: handle multipath nvme name fields correctly (LP: #1878041)
    - curtainer: mask the zfs-mount and zfs-share services [Paride Legovini]
    - tools/jenkins-runner: shuffle test-cases to randomize load
      [Paride Legovini]
    - Add Trusty/UEFI/HWE-X vmtest, drop realpath add, drop shell code
    - LP: #1881977 - Install realpath on Trusty UEFI. [Lee Trager]
    - vmtests: fix PreservePartitionWipeVg storage config
    - Fix mdraid name creates broken configuration
      [James Falcon] (LP: #1803933)
    - vmtests: update skiptests
    - vmtest: allow installed centos images to reboot (LP: #1881011)

 -- Ryan Harper <email address hidden> Thu, 20 Aug 2020 09:10:33 -0500

Changed in curtin (Ubuntu Groovy):
status: New → Fix Released
Revision history for this message
Ryan Harper (raharper) wrote :

Tomorrow's Curtin daily:ppa should have this (and as you can see, other fixes)

https://launchpad.net/~curtin-dev/+archive/ubuntu/daily

Revision history for this message
Björn Tillenius (bjornt) wrote :

Thanks Ryan!

I tested the fix on focal and it seems to work as expected.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers