Bug 560678 - virt-manager provides reboot menu option that just gives an error
Summary: virt-manager provides reboot menu option that just gives an error
Keywords:
Status: CLOSED DUPLICATE of bug 532216
Alias: None
Product: Fedora
Classification: Fedora
Component: virt-manager
Version: 12
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Cole Robinson
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-02-01 15:36 UTC by Ray Strode [halfline]
Modified: 2010-02-27 02:33 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-02-27 02:33:20 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ray Strode [halfline] 2010-02-01 15:36:56 UTC
Right now if I click reboot in virt-manager, I get an ugly error dialog.  On first glance, the dialog looks like an internal error or something.  Apparently, the problem is the underlying stack can't do a soft reboot.

I'd propose one of:

a) If soft reboot fails, pop up a dialog saying something like "We can't do an orderly restart.  Would you like us to pull the plug and plug it back in" (in better words) with a "[ x ] don't ask me again" checkbox

or

b) Add some hooks in the underlying stack to ask whether or not reboot is likely to work.  Then query those hooks when constructing the menu, and hide the reboot menu item in the case the query comes back false.

It looks really rough around the edges the way it works right now.

Comment 1 Paul Jenner 2010-02-05 21:55:49 UTC
Effectively a duplicate of bug #532216 ?

Comment 2 Wade Mealing 2010-02-10 09:44:17 UTC
If it can't reboot, just shut the machine down then start it up again.  Why have the button there if it bothers .. arrrgh !.

Comment 3 Cole Robinson 2010-02-27 02:33:20 UTC
Yeah, this is one of those things that has just been that way in the UI for so long I've grown accustomed to ignoring it.

Libvirt certainly needs a better way to determine if an API is actually supported, which would be handy in this case. But I think the way to address this is to just fake the reboot option at the libvirt level, so all API users will benefit (more info in bug 496537).

Anyways, duping to 532216.

*** This bug has been marked as a duplicate of bug 532216 ***


Note You need to log in before you can comment on or make changes to this bug.