Concrete testing shows that its actually UnexpectedError.

[jameinel]

The content of the error is a ServerError with the values we were
checking, but it has been wrapped in an UnexpectedError which doesn't
seem to make the original error available.

Description of change

MAAS 2.0 doesn't support static-routes, and I misunderstood the error we were getting. We get a ServerError wrapped by a UnexpectedError, and errors.Wrap doesn't let you get to the original error.

QA steps

Against a MAAS 2.0 controller:

  $ juju bootstrap
  $ juju switch controller
  $ juju add-machine lxd:0
  $ juju status
  $ juju ssh 0/lxd/0

Without this patch we get an error and the 'fallback to lxdbr0' code kicks in and you get an unreachable container.

Documentation changes

No, as 2.1 didn't ship with this bug.

Bug reference

lp:1668359

[jameinel]

$$merge$$ since I tested it live

[jujubot]

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

[jujubot]

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10384

[jameinel]

$$merge$$ the failure was "euca service not available at this time", doesn't seem like my fault.

[jujubot]

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju