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

qubes-manager (4.0 RC4) VM state not refreshed on shutdown #3572

Closed
lunarthegrey opened this Issue Feb 11, 2018 · 9 comments

Comments

Projects
None yet
4 participants
@lunarthegrey

Qubes OS version:

Qubes OS 4.0 RC4

Affected TemplateVMs:

N/A


Steps to reproduce the behavior:

Using the latest qubes-manager 4.0.14-1.fc25. When shutting down a VM from Qube Manager, the VM's "State" turns from green to yellow, and appears to have been fully shutdown if you run "qvm-ls" in dom0. Yet the yellow dot remains, making users think the VM is still shutting down. When pressing the "Refresh qube list" button, it actually appears that the VM had already shut down, but the state was unchanged until the button is pressed. So for some reason the state of a VM doesn't get refreshed when it has shut down.

Expected behavior:

The VM's state turns from green -> yellow -> blank

Actual behavior:

The VM's state turns from green -> yellow, but not blank until the "Refresh qube list" button is pressed.

General notes:

Nothing further to note.


Related issues:

None that I'm aware of.

@andrewdavidwong

This comment has been minimized.

Show comment
Hide comment
@andrewdavidwong

andrewdavidwong Feb 12, 2018

Member

@marmarta: I noticed that @marmarek has been assigning most qubes-manager bugs to you lately. In order to help out, I'm also going to start assigning them to you. However, if you ever think that I've assigned an issue to you that shouldn't be assigned to you, please feel free to unassign yourself or assign it to someone else. Likewise, if you find that this is not helpful in general, please let me know.

Member

andrewdavidwong commented Feb 12, 2018

@marmarta: I noticed that @marmarek has been assigning most qubes-manager bugs to you lately. In order to help out, I'm also going to start assigning them to you. However, if you ever think that I've assigned an issue to you that shouldn't be assigned to you, please feel free to unassign yourself or assign it to someone else. Likewise, if you find that this is not helpful in general, please let me know.

@marmarta

This comment has been minimized.

Show comment
Hide comment
@marmarta

marmarta Feb 13, 2018

@andrewdavidwong - thanks, this is a great idea :)

@andrewdavidwong - thanks, this is a great idea :)

@marmarta

This comment has been minimized.

Show comment
Hide comment
@marmarta

marmarta Feb 13, 2018

Due to asynchronous way shutting down works, and the way we tried to make the new Qube Manager be less un-bugfixable than the last one, it's not yet possible to do that without a major rewrite. I wonder whether it wouldn't be a good idea to just hide status buttons, and (one day) restore them, once the major rewrite comes through.

Due to asynchronous way shutting down works, and the way we tried to make the new Qube Manager be less un-bugfixable than the last one, it's not yet possible to do that without a major rewrite. I wonder whether it wouldn't be a good idea to just hide status buttons, and (one day) restore them, once the major rewrite comes through.

@lunarthegrey

This comment has been minimized.

Show comment
Hide comment
@lunarthegrey

lunarthegrey Feb 13, 2018

@marmarta I see. Would it be possible to just have the status refresh at a certain interval? I think that may be a better solution than removing the whole feature.

@marmarta I see. Would it be possible to just have the status refresh at a certain interval? I think that may be a better solution than removing the whole feature.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Feb 13, 2018

Member
Member

marmarek commented Feb 13, 2018

@lunarthegrey

This comment has been minimized.

Show comment
Hide comment
@lunarthegrey

lunarthegrey Feb 13, 2018

@marmarek Yes. But have the function that the button executes, automatically execute at a certain interval?

lunarthegrey commented Feb 13, 2018

@marmarek Yes. But have the function that the button executes, automatically execute at a certain interval?

@marmarta

This comment has been minimized.

Show comment
Hide comment
@marmarta

marmarta Feb 20, 2018

That's not a great idea at the moment. I mean, those automatic refreshes were pretty much what caused the old qubes manager to be so buggy - I'm pretty afraid to re-introduce this feature. At the moment, the designated use cases for the Qube Manager are not really the everyday operations (the domains and devices widgets are for that) - it's more like "when you actually have to create/copy/update/in other ways manage your qubes". So, hopefully one not-so-distant day we'll have better, event-aware mechanism for updating the manager state, but today, sadly no.

That's not a great idea at the moment. I mean, those automatic refreshes were pretty much what caused the old qubes manager to be so buggy - I'm pretty afraid to re-introduce this feature. At the moment, the designated use cases for the Qube Manager are not really the everyday operations (the domains and devices widgets are for that) - it's more like "when you actually have to create/copy/update/in other ways manage your qubes". So, hopefully one not-so-distant day we'll have better, event-aware mechanism for updating the manager state, but today, sadly no.

@lunarthegrey

This comment has been minimized.

Show comment
Hide comment
@lunarthegrey

lunarthegrey Feb 20, 2018

@marmarta Sounds good. Thank you for the explanation. I completely understand now.

@marmarta Sounds good. Thank you for the explanation. I completely understand now.

@marmarta

This comment has been minimized.

Show comment
Hide comment

It should have been fixed by QubesOS/qubes-manager@8753119#diff-e66f321e61dbeb6d68507cd260b4c4aa and later commits.

@marmarta marmarta closed this Jul 15, 2018

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