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

Init updates #6

Merged
merged 4 commits into from
Oct 25, 2022
Merged

Init updates #6

merged 4 commits into from
Oct 25, 2022

Conversation

masnax
Copy link
Contributor

@masnax masnax commented Oct 25, 2022

This addresses @stgraber's comments from IRC:

  • There is now a helper to get the LXD client rather than calling ConnectLXD right away.
  • Reverts initialization to use the API again. The helper here is basically initDataNodeApply from LXD with the project related parts stripped out.
  • Uses zfs for the storage driver
  • Bootstraps LXD with a lxdfan0 network device

Signed-off-by: Max Asnaashari <max.asnaashari@canonical.com>
This is so we don't get a client earlier than we need it.

Signed-off-by: Max Asnaashari <max.asnaashari@canonical.com>
Signed-off-by: Max Asnaashari <max.asnaashari@canonical.com>
Signed-off-by: Max Asnaashari <max.asnaashari@canonical.com>
StoragePools: []api.StoragePoolsPost{storage},
Profiles: []api.ProfilesPost{profile},
},
initData := initDataNode{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@masnax out of interest, why not use api.InitLocalPreseed here? They look the same?

Copy link
Member

@tomponline tomponline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The initDataNode struct seems unnecessary, but I don't want to hold this PR up, so approving.

Copy link
Member

@tomponline tomponline left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wait, you did use the newly exported structs, you just did a revert first. Makes sense.

@stgraber stgraber merged commit 1f046fb into canonical:main Oct 25, 2022
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.

None yet

3 participants