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

Various storage cleanups #6066

Merged
merged 5 commits into from Aug 12, 2019

Conversation

@monstermunchkin
Copy link
Member

commented Aug 12, 2019

No description provided.

monstermunchkin added some commits May 2, 2019

storage: Remove shared code from backends
This removes common code from the storage backends to the shared code
section.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
lxd: Remove ContainerCanRestore from storage interface
The function ContainerCanRestore is only used by zfs, and therefore
should be zfs specific.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
migration: Remove unused Snapshots() function from interface
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
lxd: Remove Image{Umount,Mount} from storage interface
These functions are not called from anywhere outside of the actual
storage backend code.

Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
test: Remove the attached testvolume
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
@lxc-jenkins

This comment has been minimized.

Copy link

commented Aug 12, 2019

Testsuite passed

@stgraber stgraber merged commit 804bcd7 into lxc:master Aug 12, 2019

5 checks passed

Branch target Branch target is correct
Details
DCO All commits signed-off
Details
Testsuite Build finished.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.