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

Inform about TemplateVM shutdown after update #9415

Open
joorei opened this issue Aug 17, 2024 · 3 comments · May be fixed by QubesOS/qubes-desktop-linux-manager#222
Open

Inform about TemplateVM shutdown after update #9415

joorei opened this issue Aug 17, 2024 · 3 comments · May be fixed by QubesOS/qubes-desktop-linux-manager#222
Assignees
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: manager/widget C: updates diagnosed Technical diagnosis has been performed (see issue comments). good first issue This is a recommended issue for first-time contributors. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. ux User experience

Comments

@joorei
Copy link

joorei commented Aug 17, 2024

How to file a helpful issue

Qubes OS release

4.2.2 (R4.2)

Brief summary

Updater shuts down TemplateVMs without warning after update.

Steps to reproduce

Successfully update a TemplateVM until you are guided into the following window

Screenshot_2024-08-17_06-41-35

Expected behavior

The UI should at least inform about imminent shutdowns. Being able to see and adjust the actions applied to each individual VM that was affected by the update would be even better.

Actual behavior

As seen in the image above, the UI does not inform about the looming shutdown of the updated VM. Instead, it informs about imminent AppVM actions only (with #9024 adjusting the wording to cover not only their restarts but also shutdowns, which depend on the type of AppVM).

The explicit mention of no restarts in the case of updated TemplateVMs (without any updated AppVMs), potentially misleads the user into thinking that no shutdowns are applied either.

Additional notes

Rather recently there was a similar issue in which the shutdown of TemplateVMs is described as intentional #9024 (comment). However, this is not communicated via the UI and the fix for that issue (https://github.com/QubesOS/qubes-desktop-linux-manager/pull/188/files) does not seem to cover any changes regarding the handling of TemplateVMs, as they were not the focus of that issue.

I chose to submit this as a "bug" because of the (subjectively perceived) misleading character. Please feel free to convert it into an "enhancement" if you disagree.

Just for context: Setting in which this issue was encountered

I used/misused a TemplateVM to setup a ZFS pool and ran it in the background to provide an AppVM A with a LUKS encrypted zvol block storage device, without A being aware of its ZFS nature. Within A this device was decrypted and the resulting (decrypted) block storage device attached to AppVM B. The automatic shutdown of the TemplateVM after its update may have resulted in a premature device destruction, which I will now investigate further.

I'm aware that this is not the purpose of TemplateVMs, but they seemed best suited, as ZFS must keep state outside of the home directory, so an AppVM can't be used. I considered a Standalone VM, but automatically being able to access Debian repositories while still having any other networking disabled was too tempting.

@joorei joorei added P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. labels Aug 17, 2024
@andrewdavidwong
Copy link
Member

Related: #9228

@andrewdavidwong andrewdavidwong added C: manager/widget ux User experience needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. C: updates affects-4.2 This issue affects Qubes OS 4.2. labels Aug 17, 2024
@marmarta marmarta added the good first issue This is a recommended issue for first-time contributors. label Oct 2, 2024
alimirjamali added a commit to alimirjamali/qubes-desktop-linux-manager that referenced this issue Oct 4, 2024
@alimirjamali
Copy link

PR Submitted

Review priority: low

Looking at qubes-vm-update which is responsible for upgrading TemplateVMs in background, it does the shutdown after successful update. Only a note has to be added to Qubes OS Update GUI intro page (at the bottom):

qubes_update

alimirjamali added a commit to alimirjamali/qubes-desktop-linux-manager that referenced this issue Oct 4, 2024
@alimirjamali
Copy link

@marmarta Just recognized recent good first issue markings of some of the issues. I guess it is better to avoid consuming all of them and leave some for new contributors. I will advertise them on forum.

@andrewdavidwong andrewdavidwong added diagnosed Technical diagnosis has been performed (see issue comments). pr submitted A pull request has been submitted for this issue. and removed needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. labels Oct 5, 2024
alimirjamali added a commit to alimirjamali/qubes-desktop-linux-manager that referenced this issue Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: manager/widget C: updates diagnosed Technical diagnosis has been performed (see issue comments). good first issue This is a recommended issue for first-time contributors. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. pr submitted A pull request has been submitted for this issue. T: bug Type: bug report. A problem or defect resulting in unintended behavior in something that exists. ux User experience
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants