/juju

Lxd container type no disk #4526

Closed
wants to merge 14 commits into
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@tych0 @jameinel
@tych0
Contributor

tych0 commented Feb 25, 2016

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

jameinel and others added some commits Feb 22, 2016

@jameinel
Update the lxdclient code to be compatible with 2.0.0beta3. 
There is one glaring ommision, namely that Ips is no longer accessible from ContainerInfo,
which was returned by ListContainers. So we'll have to figure out how to properly build
our Instances. But I'm submitting a request for more information. Everything else is
available.
2373852
@jameinel
Start removing the addresses code since we have to do it differently.
d494c07
@jameinel
Merge remote-tracking branch 'upstream/lxd-container-type' into lxd-c… 
…ontainer-type-2.0.0beta3
5ab2691
@jameinel
and now we're on to the broken stuff.
5b5917f
@jameinel
It compiles against the proposed lxd branch. 
It doesn't pass any test suites. In fact, it segfaults. But it does compile.

This rips out all the code that dealt with writing to disk in order to get
certificates and configuration working with the LXD code.
The nice part is that it is removing a bunch of code. Bad part is it doesn't work yet.
fa3c8a1
lxd: use real upstream hash for new client API 
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
22dd01d
lxd: really fix compilation :) 
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
98c547a
lxd: fix segfault in tests 
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
afd32d9
lxd: implement Addresses() call for lxd provider 
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
517f93b
lxd: s/limits.cpus/limits.cpu 
This key was renamed a while ago.

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
3d54800
lxd: fix parsing of limits.memory 
Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
640b8a0
lxd: get rid of lxdNewClient 
This is dead code.

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
0317890
lxd: include default port in hostname 
This code is all kind of a clusterfuck, but when we generate the remote
address for a LXD, we need to include the default port as well, since we
set https_address to ::, which binds to 8443 by default.

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
03ff20b
lxd: plumb the server cert through 
This adds a config option for the server cert and the plumbing to get it
through.

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>
e1de2e4
@tych0
Contributor

tych0 commented Feb 25, 2016

Closing in favor of #4537

@tych0 tych0 closed this Feb 25, 2016

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