Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
2.0 lxd provider #801
Conversation
pmatulis
added some commits
Jan 13, 2016
pmatulis
assigned
evilnick
Jan 15, 2016
mbruzek
reviewed
Jan 15, 2016
| +LXD itself (this document is ultimately about running LXD with Juju) it does | ||
| +so assuming you are setting up LXD for the first time. | ||
| + | ||
| +**Note:** If you are running iptables (firewall) or even an iptables frontend |
mbruzek
Jan 15, 2016
Contributor
Recommending that we turn off the firewall is not going to fly with some companies. Do we know what ports to disable in ufw to get LXD to run?
pmatulis
Jan 15, 2016
Contributor
Yeah, this was taken from the existing config-LXC. I never looked into it.
mbruzek
Jan 15, 2016
Contributor
Perhaps the core developers could give us some feedback on which ports to enable on ufw which is installed on Ubuntu server by default IIRC.
pmatulis
Jan 20, 2016
Contributor
To be clear, the disabling of the firewall was meant as a temporary troubleshooting technique only. That should be clarified.
mbruzek
reviewed
Jan 15, 2016
| + | ||
| +# Prerequisites and installation of Juju and LXD | ||
| + | ||
| + - The LXD local provider will not work on Ubuntu 12.04 LTS and backporting to |
mbruzek
Jan 15, 2016
Contributor
It seems the convention for this document is NN.NN LTS (codename). I don't see the convention used here. Can we use (Precise) and (Trusty) here?
mbruzek
reviewed
Jan 15, 2016
| +'ubuntu-xenial'. | ||
| + | ||
| +The first time an image is needed the store will supply it and any subsequent | ||
| +requests will be satisfied by the LXD cache. In this way, a store is sollicited |
mbruzek
reviewed
Jan 15, 2016
| + | ||
| +# Other useful commands | ||
| + | ||
| +There is a cornucopia of commands available. Some common ones not yet covered |
mbruzek
Jan 15, 2016
Contributor
While "cornucopia" is correctly used here it stuck out when I read it. Did "plethora", "abundance", "multiple", or "many" not work in this context?
|
@petermatulis thanks for letting me take a look at this. Line 54 satisfies the problem I encountered. I found a few other minor suggestions, but it looks great! |
evilnick
reviewed
Jan 20, 2016
| +`~/.juju/environments.yaml` file. Create one with | ||
| + | ||
| +```bash | ||
| +juju generate-config |
|
looks good apart from the fact that some bits will not exists in 2.0. I recommend we land it with othe fixes and then update it when we know how the authentication model will actually work |
pmatulis
assigned
pmatulis
and unassigned
evilnick
Jan 20, 2016
pmatulis
added some commits
Jan 20, 2016
added a commit
that referenced
this pull request
Jan 20, 2016
pmatulis commentedJan 15, 2016
No description provided.