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

If target image is a block device, access it directly #22

Merged
merged 2 commits into from
Sep 29, 2023

Conversation

omnivagant
Copy link

This will allow a user to specify a block device, like an LVM volume or
ZFS zvol, as target.

@jirutka
Copy link
Member

jirutka commented Nov 26, 2021

This is a nice idea, but it’s incomplete – cleanup function (alpine-make-vm-image:143-146) must be modified.

@omnivagant
Copy link
Author

Like so?

In some sense I feel like I'm a bit lazy here in keeping the $nbd_dev name even if it isn't.

@jirutka
Copy link
Member

jirutka commented Jul 14, 2022

Yes, there are more things that should be renamed and documented to make sense after this change.

@omnivagant
Copy link
Author

What if we just call it $disk_dev instead, would that be enough?

@jirutka
Copy link
Member

jirutka commented Sep 24, 2023

Yes, that’s fine, but the help message needs to be updated as well.

@omnivagant
Copy link
Author

Of course, like so?

omni and others added 2 commits September 29, 2023 15:14
Access the block device directly.

This will allow a user to specify a block device, like an LVM volume or
ZFS zvol, as target.

Co-Authored-By: Jakub Jirutka <jakub@jirutka.cz>
@jirutka jirutka merged commit 352609f into alpinelinux:master Sep 29, 2023
2 checks passed
@jirutka
Copy link
Member

jirutka commented Sep 29, 2023

Yes, thanks!

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