worker/provisioner: filter container tools to host #5244

Merged
merged 2 commits into from Apr 21, 2016

Conversation

Projects
None yet
2 participants
Member

axw commented Apr 21, 2016

In case there are multiple agent architectures available for
starting a container, ensure we filter to the host architecture.
We were doing this for LXC, but not for LXD or KVM. We would
previously have been luck on amd64, since it's always the first
in the list.

Fixes https://bugs.launchpad.net/juju-core/+bug/1572781

(Review request: http://reviews.vapour.ws/r/4671/)

axw added some commits Apr 21, 2016

worker/provisioner: filter container tools to host
In case there are multiple agent architectures available for
starting a container, ensure we filter to the host architecture.
We were doing this for LXC, but not for LXD or KVM. We would
previously have been luck on amd64, since it's always the first
in the list.

Fixes https://bugs.launchpad.net/juju-core/+bug/1572781
Member

axw commented Apr 21, 2016

$$merge$$

Contributor

jujubot commented Apr 21, 2016

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

@jujubot jujubot merged commit d0f6773 into juju:master Apr 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment