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

Don't allow to shutdown a VM if it's serving any block devices to other VMs #570

Closed
marmarek opened this Issue Mar 8, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@marmarek
Member

marmarek commented Mar 8, 2015

Reported by joanna on 16 May 2012 13:18 UTC
A typical scenario would be to shutdown usbvm for backup, while having the usbvm serving the actual block device for backup...

Migrated-From: https://wiki.qubes-os.org/ticket/570

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by marmarek on 20 May 2012 12:43 UTC
IMHO this should be done in qubes-manager - as a warning before shutting down VM which provide device for backup.
VM providing some block device to any other VM%2C can be stopped safely, especially for backup. Anyway this other VM most likely will also be stopped.

BTW The same was for network dependency: firstly we implemented blocker (don't allow netvm shutdown when used by other VM) but later reconsidered that this wasn't good idea and the better was some warning and allow to shutdown used netvm.

Member

marmarek commented Mar 8, 2015

Comment by marmarek on 20 May 2012 12:43 UTC
IMHO this should be done in qubes-manager - as a warning before shutting down VM which provide device for backup.
VM providing some block device to any other VM%2C can be stopped safely, especially for backup. Anyway this other VM most likely will also be stopped.

BTW The same was for network dependency: firstly we implemented blocker (don't allow netvm shutdown when used by other VM) but later reconsidered that this wasn't good idea and the better was some warning and allow to shutdown used netvm.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by joanna on 21 May 2012 08:21 UTC
Agree.

Member

marmarek commented Mar 8, 2015

Comment by joanna on 21 May 2012 08:21 UTC
Agree.

@marmarek marmarek removed the C: core label Mar 8, 2015

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Mar 8, 2015

Member

Comment by aga on 25 May 2012 14:40 UTC
Warning on shutting down a VM that attaches block devs to some other VMs:
http://git.qubes-os.org/?p=aga/qubes-manager.git;a=commit;h=d2a75bae4cdcc14f156c87bc1c9fc2ee77c3d4fc

Member

marmarek commented Mar 8, 2015

Comment by aga on 25 May 2012 14:40 UTC
Warning on shutting down a VM that attaches block devs to some other VMs:
http://git.qubes-os.org/?p=aga/qubes-manager.git;a=commit;h=d2a75bae4cdcc14f156c87bc1c9fc2ee77c3d4fc

@marmarek marmarek closed this Mar 8, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment