Bug 847334 - RFE: allow force hard reset of VMs
Summary: RFE: allow force hard reset of VMs
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Virtualization Tools
Classification: Community
Component: virt-manager
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Cole Robinson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-08-10 15:12 UTC by Adam Spiers
Modified: 2013-03-18 00:05 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-03-16 22:03:54 UTC
Embargoed:


Attachments (Terms of Use)
patch for adding force reset to shutdown menu (14.37 KB, patch)
2013-03-16 00:35 UTC, John Doyle
no flags Details | Diff

Description Adam Spiers 2012-08-10 15:12:35 UTC
Every other hypervisor GUI I've used allows forcing a hard reset (i.e. power cycle) of a VM as a single action.  With virt-manager you have to first force off, and then start (based on usage of 0.9.1 - I could not find any information on http://virt-manager.org/ regarding later releases).  This is pretty annoying if you have to do it regularly (e.g. when testing PXE boot environments).  

Of course the same can be automated via virsh, but the current deficiency of virt-manager is a clear (albeit minor) usability issue, and presumably very easy to fix.

Comment 1 Dave Allan 2013-02-22 21:24:47 UTC
Implementing reset should be as simple as UI to call virDomainReset

Comment 2 John Doyle 2013-03-16 00:35:58 UTC
Created attachment 710930 [details]
patch for adding force reset to shutdown menu

I agree, it would be useful so here's a patch which adds it to the shutdown menu. Works as Dave suggested.

Comment 3 Cole Robinson 2013-03-16 22:03:54 UTC
(In reply to comment #2)
> Created attachment 710930 [details]
> patch for adding force reset to shutdown menu
> 
> I agree, it would be useful so here's a patch which adds it to the shutdown
> menu. Works as Dave suggested.

Thanks John, patch looks good, pushed upstream.

http://git.fedorahosted.org/cgit/virt-manager.git/commit/?id=9f4fddba0454b42df7ea0893f672084852cf2e4b

(small nit: for git patches it's generally easier on the maintainer to use git format-patch for generating the patch, rather than git show)

Comment 4 John Doyle 2013-03-18 00:05:53 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > Created attachment 710930 [details]
> > patch for adding force reset to shutdown menu
> > 
> > I agree, it would be useful so here's a patch which adds it to the shutdown
> > menu. Works as Dave suggested.
> 
> Thanks John, patch looks good, pushed upstream.
> 
> http://git.fedorahosted.org/cgit/virt-manager.git/commit/
> ?id=9f4fddba0454b42df7ea0893f672084852cf2e4b
> 
> (small nit: for git patches it's generally easier on the maintainer to use
> git format-patch for generating the patch, rather than git show)

Sorry Cole, will remember for the future. Thanks.


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