fix for bug 1707248

[hmlanigan]

Description of change

avoid panic by adding distributionGroupFinder to containerProvisioner.

QA steps

1. juju bootstrap localhost
2. juju deploy mysql -n 2 --to lxd,lxd
3. ensure that 0/lxd/0 and 1/lxd/0 are provisioned.

Documentation changes

N/A

Bug reference

https://bugs.launchpad.net/juju/+bug/1728066

[jujubot]

Can one of the admins verify this patch?

[jameinel]

The change seems ok, but why didn't the test suite catch it? Is it a case that it would have been caught in the CI tests, but you actually need to provision a container?

[hmlanigan]

$$merge$$

[hmlanigan]

It fell in the cracks with test cases in the test suite. The container brokers test that StartInstance() works within out the instance starting. The container_initializer tests called the provisioner correctly, but containers are never actually provisioned. CI tests don't provision containers. Yes, you'd need provision a container within a juju machine to hit this.

[jujubot]

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju