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

Storage Volume Copy using in-memory pipe and migration logic #6367

Merged
merged 6 commits into from Oct 30, 2019

Conversation

@tomponline
Copy link
Member

tomponline commented Oct 30, 2019

  • Updates Storage volume copy to use the migration logic with a bi-directional in-memory pipe.
  • Adds progress tracker indicator to storage migration functions so that when doing a local copy one can disable the progress tracker on one side.
Compatible with the io.ReadWriteCloser interface.

Used for simulating WebsocketIO between local endpoints.

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
@tomponline tomponline requested a review from stgraber Oct 30, 2019
@lxc-jenkins

This comment has been minimized.

Copy link

lxc-jenkins commented Oct 30, 2019

Testsuite passed

tomponline added 5 commits Oct 30, 2019
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
…gration logic

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
…SourceArgs and MigrationTargetArgs

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
…ackProgress bool

Signed-off-by: Thomas Parrott <thomas.parrott@canonical.com>
@tomponline tomponline force-pushed the tomponline:tp-storage-copy-pipe branch from 05f3e1a to abfbf86 Oct 30, 2019
@stgraber stgraber merged commit f0955d3 into lxc:master Oct 30, 2019
4 of 5 checks passed
4 of 5 checks passed
Testsuite Test started
Details
Branch target Branch target is correct
Details
DCO All commits signed-off
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tomponline tomponline deleted the tomponline:tp-storage-copy-pipe branch Oct 30, 2019
@lxc-jenkins

This comment has been minimized.

Copy link

lxc-jenkins commented Oct 30, 2019

Testsuite passed

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.