CTRL+ALT+DEL Button missing

[Bjoern]

Since RPC9 moved to spice, the CTRL+ALT+DEL Button missing in the console.
So windows clients are having issues now to pass this keystroke to the spice window, since it's locally grabbed.

Some bug fixes around that issue :

http://cgit.freedesktop.org/spice/spice-html5/commit/?id=828d975b2db7231d5560f4ca3cf0487ce5c80d4b
https://access.redhat.com/solutions/1288163 RedHat OSP

[Bjoern (bjoern-t)]

In spice.css we need to add:

.spice-controlpane
{
    text-align: left;
    display: inline-block;
    padding: 10px;
    min-width: 800px;
    min-height: 20px;
    margin-top: 5px;
    border: solid #222222 1px;
    background-color: #333333;
    -webkit-box-shadow: 5px 5px 5px rgba(0, 0, 0, 0.2);
    -moz-box-shadow: 5px 5px 5px rgba(0, 0, 0, 0.2);
    box-shadow: 5px 5px 5px rgba(0, 0, 0, 0.2);
    -moz-border-radius: 10px;
    -webkit-border-radius: 10px;
    border-radius: 10px;
}

in spice_auto.html we need to add :

        <div id="spice-area">
          <div id="spice-controlpane" class="spice-controlpane">
            <button type="button" onclick="sendCtrlAltDel()">
               Send Ctrl-Alt-Delete
            </button>
          </div>
        </div>

below the spice-screen div

[Bjoern (bjoern-t)]

I just tested in my deployment and it works

[Bjoern (bjoern-t)]

We probably add this after installing nova-spice inside the nova_spice_console_container

[Andy McCrae (andrew-mccrae)]

I think this should be fixed upstream and not in the os-ansible-deployment repo. We could "hack" around this, but I don't think that is the appropriate approach to this.

This bug means nobody anywhere can access windows instances using spice console so won't just affect os-ansible-delpoyment.

[Bjoern (bjoern-t)]

Hi Andy,

I disagree we should implement it now and not risk loosing customers because it'll take a long time until it gets fixed at the right places

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: juno
Review: https://review.openstack.org/165540

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: icehouse
Review: https://review.openstack.org/165541

[Andy McCrae (andrew-mccrae)]

Patches applied to rpc_support plays in juno/icehouse.
Patch applied to external git for rpc_support in master - so not relevant to the os-ansible-deployment work (marking trunk as "wont fix")

[Bjoern (bjoern-t)]

This issue needs to be fixed in the trunk as well for the next RPC11 release.

[Andy McCrae (andrew-mccrae)]

rpc-support plays were removed from os-ansible-deployment in trunk so that fix does not live in os-ansible-deployment.

[Bjoern (bjoern-t)]

Thanks. Did you open a rpcextras issue already ?

[Nolan Brubaker (nolan-brubaker)]

The rpc-extras issue is here: https://github.com/rcbops/rpc-extras/issues/42

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/165540
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=3508eaf3f5c3ecf4ee49307e6f25d6ae24af0984
Submitter: Jenkins
Branch: juno

commit 3508eaf3f5c3ecf4ee49307e6f25d6ae24af0984
Author: Andy McCrae <email address hidden>
Date: Wed Mar 18 18:05:22 2015 +0000

    Add ctrl-alt-del button to spice console

    * Replace the spice_auto.html and spice.css files
    * Adds a ctrl-alt-del button

    Closes-Bug: #1423669
    Change-Id: I77a9b2a75abbdcdbfa9a57740eee402b87cfca06

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/165541
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=107d0f3bd49aca996577461b9e259cf0bd3248d9
Submitter: Jenkins
Branch: icehouse

commit 107d0f3bd49aca996577461b9e259cf0bd3248d9
Author: Andy McCrae <email address hidden>
Date: Wed Mar 18 18:05:22 2015 +0000

    Add ctrl-alt-del button to spice console

    * Replace the spice_auto.html and spice.css files
    * Adds a ctrl-alt-del button

    Closes-Bug: #1423669
    Change-Id: I77a9b2a75abbdcdbfa9a57740eee402b87cfca06
    (cherry picked from commit 3508eaf3f5c3ecf4ee49307e6f25d6ae24af0984)

[Felipe Reyes (freyes)]

There is an upstream bug for this issue https://bugs.freedesktop.org/show_bug.cgi?id=89613

[Felipe Reyes (freyes)]

Patch sent for review in upstream http://lists.freedesktop.org/archives/spice-devel/2015-June/020267.html