Bug 1014666 - RFE: allow user to specify custom sendkey sequences with gsettings
Summary: RFE: allow user to specify custom sendkey sequences with gsettings
Keywords:
Status: CLOSED WONTFIX
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: 2013-10-02 13:56 UTC by Zdenek Kabelac
Modified: 2020-09-01 23:59 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-09-01 23:59:11 UTC
Embargoed:


Attachments (Terms of Use)

Description Zdenek Kabelac 2013-10-02 13:56:20 UTC
Description of problem:

I believe it would be quite useful to have this shortcut available in the 'Send key' many (just like  Ctrl+Alt+Fxx)  - maybe under submenu 'Debug'.
So in case I attach virt-manager to a frozen kvm box I could easily see where it's get trapped.  Yeah I know I could use virsh-something for that - but using a mouse friendly many would be cool.


Version-Release number of selected component (if applicable):
virt-manager-0.10.0-2.git948b5359.fc21.noarch

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Cole Robinson 2013-10-02 17:56:48 UTC
Isn't this sysrq turned off by default in distro kernels? If it's something that you need to compile a custom kernel for, then it's sufficiently obscure that I don't want to add an option to the UI for it, when as you point out there's a virsh sendkey option for.

I've thought in the past about adding some gsettings/gconf way of allowing users to register their own key sequences, we can use this to track that.

Comment 2 Zdenek Kabelac 2013-10-03 07:59:36 UTC
(In reply to Cole Robinson from comment #1)
> Isn't this sysrq turned off by default in distro kernels? If it's something
> that you need to compile a custom kernel for, then it's sufficiently obscure

IMHO it's compiled-in in every Fedora kernel - and it's just disabled for security reasons for normal users.  For a developer you may easily enable
it by setting through sysctl settings: 'kernel.sysrq = 1'
This will unlock all options.

> that I don't want to add an option to the UI for it, when as you point out
> there's a virsh sendkey option for.

Well when you have a lot of graphical clicking windows from various remote boxes - you do not really want to bother with all the virsh ugliness. 
(And I've never tried that, I just assume it could probably work somehow in there).

> I've thought in the past about adding some gsettings/gconf way of allowing
> users to register their own key sequences, we can use this to track that.

I'm not a gnome expert here, in fact I'd to go away from all the horrible Gnome3 thing and permanent unusability  in Rawhide (it's been quite common to have to broken there for months...) - so I'm just openbox/xfce user.

Comment 3 Cole Robinson 2013-10-03 13:17:42 UTC
> 
> > I've thought in the past about adding some gsettings/gconf way of allowing
> > users to register their own key sequences, we can use this to track that.
> 
> I'm not a gnome expert here, in fact I'd to go away from all the horrible
> Gnome3 thing and permanent unusability  in Rawhide (it's been quite common
> to have to broken there for months...) - so I'm just openbox/xfce user.

gsettings + dconf is what we use to store persistent settings no matter what desktop environment you are on. so the solution I propose would be independent of gnome shell

Comment 4 Cole Robinson 2020-09-01 23:59:11 UTC
The goal going forward is to keep virt-manager's console support fairly slim and basic and punt more advanced cases to more appropriate tools. This is described more in the design document: https://github.com/virt-manager/virt-manager/blob/master/DESIGN.md

I was onboard with adding more keycombo support to virt-manager for a while but nowadays if it should live anywhere it should be in virt-viewer IMO


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