MAAS bridge script does not handle post-up command changes from curtin (0.1.0~bzr399-0ubuntu1~16.04.1)

Bug #1631940 reported by Andrew McDermott
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju
Triaged
Low
Unassigned

Bug Description

Curtin version (0.1.0~bzr399-0ubuntu1~16.04.1) now emits post-up commands for aliases.

From the changelog @ https://launchpad.net/ubuntu/xenial/+source/curtin/+changelog

 curtin/net: use post-up for interface alias, resolve 120 second time out
      on Trusty when using interface aliases

If aliases are used in MAAS we get an /etc/network/interfaces file that looks like:

auto ens3
iface ens3 inet static
    address 192.168.20.201/24
    gateway 192.168.20.1
    mtu 1500
    post-up ifup ens3:1

auto ens3:1
iface ens3:1 inet static
    address 192.168.20.202/24

However, the MAAS bridge script does not parse post-up commands, leaving the name 'ens3:1' unchanged after bridging.

For example:

$ ./add-juju-bridge.py --interfaces-to-bridge="ens3 ens3:1"/etc/network/interfaces

auto ens3
iface ens3 inet manual
    mtu 1500
    post-up ifup ens3:1

auto br-ens3
iface br-ens3 inet static
    address 192.168.20.201/24
    gateway 192.168.20.1
    post-up ifup ens3:1
    bridge_ports ens3

auto br-ens3:1
iface br-ens3:1 inet static
    address 192.168.20.202/24

The alias names in the post-up commands should now be 'br-ens3:1'.

Changed in juju:
milestone: none → 2.1.0
status: New → Triaged
importance: Undecided → High
Revision history for this message
Anastasia (anastasia-macmood) wrote :

Removing 2.1 milestone as we will not be addressing this issue in 2.1.

tags: added: eda requires-functional-test
Changed in juju:
milestone: 2.1-rc2 → none
Revision history for this message
Anastasia (anastasia-macmood) wrote :

This behavior is still present in 2.1-rc2.

Ian Booth (wallyworld)
tags: removed: maas
Revision history for this message
Canonical Juju QA Bot (juju-qa-bot) wrote :

This bug has not been updated in 2 years, so we're marking it Low importance. If you believe this is incorrect, please update the importance.

Changed in juju:
importance: High → Low
tags: added: expirebugs-bot
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers