Operation not supported: cannot change keymap setting on vnc graphics about fog-libvirt HOT 13 CLOSED

Released as 0.4.1. =)

from fog-libvirt.

Hello all,

I just updated to foreman 1.15.3. Running debian 8.9 on my foreman host, and the same on my compute hosts. I ran into this issue after updating.

I've manually patched /usr/share/foreman/vendor/ruby/2.1.0/gems/fog-libvirt-0.4.0/lib/fog/libvirt/requests/compute/update_display.rb according to https://github.com/fog/fog-libvirt/pull/43/files on my foreman install and this fixed the issue for me.

Thanks for the patch!

from fog-libvirt.

@santachago could you check if reverting #30 fixes that issue?

from fog-libvirt.

@tbrisker looking at #30 I don't get the whole current situation, as that was also about vnc

from fog-libvirt.

@mmoll IIRC #30 in my case was caused by having a non-auto setting for the keymap, which was failing because fog-libvirt wasn't passing a keymap and libvirt was defaulting to auto (which was different then the existing setting and thus failed to change it). That being said, I think there may be a better solution then this. Sadly I don't have the setup I had previously to try and fix it at the moment.

from fog-libvirt.

If i remove line number 16 -> display[:keymap] = options[:keymap] || xml_elements(domain.xml_desc, "graphics", "keymap")

vms with no keymap defined on the config work fine. If a vm has a value defined it fails with the same error.

from fog-libvirt.

possible fix in #43

from fog-libvirt.

@santachago @tbrisker @mmoll Could you guys please test if master fixed the issue please?

from fog-libvirt.

I can only say it fixed it for me (which is expected as I wrote the patch ;)).

from fog-libvirt.

@santachago any update here? ☝️

from fog-libvirt.

@mmoll Thanks for the feeback. 👍

from fog-libvirt.

I guess a 0.4.1 release would be nice then...

from fog-libvirt.

Cuting a release now. =)

from fog-libvirt.