Skip to content

Set correct xenial image for LXD#45

Merged
blackboxsw merged 1 commit into
canonical:masterfrom
lucasmoura:fix-xenial-image-on-lxd
Nov 12, 2020
Merged

Set correct xenial image for LXD#45
blackboxsw merged 1 commit into
canonical:masterfrom
lucasmoura:fix-xenial-image-on-lxd

Conversation

@lucasmoura
Copy link
Copy Markdown

We are only using the images:ubuntu/16.04/cloud when we are creating virtual machines. However, this is wrong, since we always need to use that machine, for containers and virtual machines. The reason for that is that other xenial images will not have the necessary requirements to allow the use of lxc exec commands

Comment thread pycloudlib/lxd/cloud.py Outdated
if release == "xenial":
# xenial needs to launch images:ubuntu/16.04/cloud
# because it contains the HWE kernel which has vhost-vsock support
return self.XENIAL_VM_IMAGE
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

let's rename this to XENIAL_IMAGE_VSOCK_SUPPORT

@lucasmoura lucasmoura force-pushed the fix-xenial-image-on-lxd branch from 1b59fa5 to 98c45c2 Compare November 12, 2020 16:36
@lucasmoura
Copy link
Copy Markdown
Author

@blackboxsw Updated

Copy link
Copy Markdown
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

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

LGTM

@blackboxsw blackboxsw merged commit e9791a9 into canonical:master Nov 12, 2020
@lucasmoura lucasmoura mentioned this pull request Nov 13, 2020
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