Skip to content

Use existing QEMU session for copy-in/copy-out #147

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

A0lson
Copy link

@A0lson A0lson commented Jul 30, 2021

When --copy-using-image is specified, copy operations
will be done using the main QEMU session instead of separate ones.

This avoids the time spent starting up separate QEMU sessions before/after
the main one to do file copying.

@A0lson A0lson force-pushed the copy_quicker branch 2 times, most recently from 1e9839b to ef19336 Compare July 30, 2021 22:29
@A0lson A0lson force-pushed the copy_quicker branch 3 times, most recently from 74a95c8 to a467e7a Compare August 27, 2021 22:15
@A0lson
Copy link
Author

A0lson commented Aug 27, 2021

This revision will also need ALSchwalm/transient-baseimages#1 before the new tests will pass.

@A0lson A0lson changed the base branch from adam-dev to master September 10, 2021 16:39
@A0lson A0lson marked this pull request as ready for review September 10, 2021 16:49
@A0lson
Copy link
Author

A0lson commented Sep 10, 2021

Its mostly ready but tests won't pass until the PRs for the baseimages go through first.

@ALSchwalm
Copy link
Owner

transient-baseimages now has a 6 tag with the alpine rsync stuff: https://github.com/ALSchwalm/transient-baseimages/releases/tag/6

@A0lson A0lson force-pushed the copy_quicker branch 4 times, most recently from a86f70a to b5f01f3 Compare September 11, 2021 05:52
When --copy-using-image is specified, copy operations will be done using the main QEMU session instead of separate ones.

This avoids the time spent starting up separate QEMU sessions before/after the main one to do file copying.

However, use of this option requires that scp/rsync be present in the image
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