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

Image export over devlxd for virtual machines #13878

Merged
merged 5 commits into from
Aug 6, 2024

Conversation

markylaing
Copy link
Contributor

@markylaing markylaing commented Aug 5, 2024

Adds an endpoint to the LXD agent /dev/lxd socket listener that proxies requests to /1.0/images/{fingerprint}/export on the host LXD devlxd listener.

Additionally cleans up lxd-agent/devlxd.go and moves the security.devlxd.images config key from the container specific config key map into the shared container/VM config key map and regenerates the metadata.

I just worked on this to speed up my local test runs :)

Closes #12589

Single letter or abbreviated variable names should only be used in small
scopes and not in struct fields (see
https://google.github.io/styleguide/go/decisions.html#single-letter-variable-names).

Signed-off-by: Mark Laing <[email protected]>
Additionally, specify field names in hander structs.

Signed-off-by: Mark Laing <[email protected]>
@markylaing markylaing added the Feature New feature, not a bug label Aug 5, 2024
@markylaing markylaing self-assigned this Aug 5, 2024
@github-actions github-actions bot added the Documentation Documentation needs updating label Aug 5, 2024
Copy link

github-actions bot commented Aug 5, 2024

Heads up @ru-fu - the "Documentation" label was applied to this issue.

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.

LGTM thanks!

Please could we have an update to lxd-ci as well to catch regressions. Ta

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.

Thanks!

@tomponline tomponline merged commit da24dfd into canonical:main Aug 6, 2024
29 checks passed
@markylaing
Copy link
Contributor Author

LGTM thanks!

Please could we have an update to lxd-ci as well to catch regressions. Ta

Associated test PR: canonical/lxd-ci#255

tomponline added a commit that referenced this pull request Nov 11, 2024
#13878 Added `security.devlxd.images` for virtual machines but this key
was not live-updatable. There was no particular reason for this. The
endpoint is always available in the LXD agent and access is restricted
by the host daemon.

This PR allows the config key to be set while the instance is running.

Closes #14206
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation needs updating Feature New feature, not a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for security.devlxd.images on VMs
2 participants