Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update LXD Client Config #102

Closed
wants to merge 2 commits into from
Closed

Conversation

jtopjian
Copy link
Contributor

This commit updates the LXD discovery plugin to support the new LXD
client API.

This commit updates the LXD discovery plugin to support the new LXD
client API.
@jtopjian
Copy link
Contributor Author

jtopjian commented Jul 29, 2017

Hello,

The LXD project recently refactored their client API. We moved to this new API a few weeks ago in the Terraform LXD provider and can verify the changes are safe and work.

These are all internal changes. There are no changes to the gobetween configuration or any other user-facing pieces.

In order to help with testing, I have created a set of Terraform configurations that you can find here:

https://github.com/jtopjian/terraform-gobetween-dev

This will deploy a virtual machine to either AWS or OpenStack and create an environment with LXD (including two running containers), Go, and dependencies.

To use this, do the following:

  1. Download Terraform
  2. Clone the above repository
  3. Do:
$ cd terraform-gobetween-dev/aws
$ export AWS_SECRET_ACCESS_KEY=your aws secret access key
$ export AWS_ACCESS_KEY_ID=your aws access key id
$ terraform apply

If you prefer not to use Terraform, you can simply use the deploy script found in files/deploy.sh. Just copy this to any Ubuntu 16.04 virtual machine.

When the script is done running, you can then build a gobetween binary using whatever branch, then use the supplied /home/ubuntu/gobetween.toml file. This has all configuration necessary to connect to the supplied LXD server:

$ sudo ./bin/gobetween -c ~/gobetween.toml
$ curl localhost

Please let me know if you have any questions.

@illarion
Copy link
Collaborator

Sorry, I created duplicate issue without noticing that you've already done whole thing. My bad

@illarion illarion self-requested a review August 15, 2017 18:50
@jtopjian
Copy link
Contributor Author

No problem at all. Should I open an Issue+PR next time?

@illarion illarion self-assigned this Aug 15, 2017
@illarion illarion requested a review from yyyar August 15, 2017 18:55
@illarion
Copy link
Collaborator

I will leave it here just not to forget:

sudo dpkg --add-architecture i386
sudo apt install libacl1-dev:i386
sudo apt install libc-dev:i386

This lines are required in order to make a cross build

@jtopjian
Copy link
Contributor Author

@illarion Good call. I have updated the deploy.sh script with that :)

@illarion
Copy link
Collaborator

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants