Fix not_networks parameter (negative space constraints) for MAAS 1.9 #6541

Merged
merged 2 commits into from Nov 4, 2016

Conversation

Projects
None yet
3 participants

A backport of PR #6528 for juju 2.0.

With MAAS 1.9 the not_networks parameter (for negative space constraints) should be specified as multiple values, not a comma separated list.

Fixes bug #1636969
There is a separate PR (against gomaasapi) to fix the same issue for MAAS 2.x where "not_networks" becomes "not_subnets".

QA
Bootstrap specifying negative space constraints. Juju must honour those constraints in the machine it picks.

bz2 approved these changes Nov 3, 2016

Looks good, thanks!

$$merge$$

Contributor

jujubot commented Nov 3, 2016

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

Contributor

jujubot commented Nov 3, 2016

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

$$merge$$

Contributor

jujubot commented Nov 4, 2016

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

@jujubot jujubot merged commit 053b04f into juju:2.0 Nov 4, 2016

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment