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

Add option to automatically shutdown netVM of child in qvm-shutdown #4102

Open
kq4x opened this Issue Jul 18, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@kq4x

kq4x commented Jul 18, 2018

Qubes OS version: R3.2

Affected component(s): qvm-shutdown

Hi there,
I have added a feature to the qvm-shutdown script in R3.2 . What it does is add an option to (recursively) shutdown the parent netVM of the VM. Since the code has changed since R3.2 I cannot add it to the new version.

I could not find the R3.2 branch in https://github.com/QubesOS/qubes-core-admin-client so I thought I'd post here.
The file in question is https://github.com/QubesOS/qubes-core-admin-client/blob/master/qubesadmin/tools/qvm_shutdown.py

I have attached the modified file and the diff compared to the original R3.2 qvm-shutdown file to this issue.
qvm-shutdown.stop-netvm.diff.txt
qvm-shutdown.stop-netvm.txt

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Jul 18, 2018

Member

The R3.2 version is here: https://github.com/QubesOS/qubes-core-admin/blob/release3.2/qvm-tools/qvm-shutdown (it was split into qubes-core-admin-client as part of R4.0 rewrite)

Member

marmarek commented Jul 18, 2018

The R3.2 version is here: https://github.com/QubesOS/qubes-core-admin/blob/release3.2/qvm-tools/qvm-shutdown (it was split into qubes-core-admin-client as part of R4.0 rewrite)

@kq4x

This comment has been minimized.

Show comment
Hide comment
@kq4x

kq4x Jul 18, 2018

Ok I will make a proper PR

kq4x commented Jul 18, 2018

Ok I will make a proper PR

@andrewdavidwong

This comment has been minimized.

Show comment
Hide comment
@andrewdavidwong

andrewdavidwong Jul 19, 2018

Member

I find the use of "parent" and "child" here confusing. What exactly is the "parent NetVM"? Is that just the VM's NetVM? If so, can we drop the word "parent"? A VM's NetVM is neither its parent nor its child.

Member

andrewdavidwong commented Jul 19, 2018

I find the use of "parent" and "child" here confusing. What exactly is the "parent NetVM"? Is that just the VM's NetVM? If so, can we drop the word "parent"? A VM's NetVM is neither its parent nor its child.

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