theforeman / foreman_fog_proxmox

Foreman plugin to add Proxmox compute resource based on fog-proxmox gem
GNU General Public License v3.0
105 stars 30 forks source link

Creating a new host sets keyboard for VM #183

Closed l00d3r closed 3 years ago

l00d3r commented 3 years ago

Describe the bug Creating a new host on proxmox using foreman sets the keyboard for a VM while creating a VM using proxmox web UI does not. If using the console feature of foreman or directly from proxmox the noVNC console does not correctly translate the keystrokes from browser to VM and is impossible to use using et-ee keymap from client (for example ":" gets translated to "Ö", can't exit vi)

To Reproduce Create a VM using foreman and try to use console with some other keymap.

Expected behavior By default keymap attribute of VM config is not needed as it is not mandatory when creating a VM using proxmox web UI and should at least be allowing not setting the keyboard attribute.

Examples

VM config created with foreman root@proxmox:~# qm config 113 agent: 0 balloon: 0 bios: seabios boot: order=scsi0;ide2;net0 cores: 2 cpu: cputype=kvm64 ide2: none,media=cdrom keyboard: en-us kvm: 1 memory: 2512 name: leigh-grando net0: virtio=02:FB:4C:4F:36:26,bridge=vmbr1,firewall=0,link_down=0 numa: 0 onboot: 1 ostype: l26 scsi0: HP:vm-113-disk-0,cache=none,size=10G scsihw: virtio-scsi-pci shares: 0 smbios1: uuid=7f14a895-5130-4f85-a640-8832ea9783a0 sockets: 1 vga: std vmgenid: aab608cf-6df5-4c60-9f2b-a6e4b5f4a71c

VM config created with proxmox web UI root@proxmox:~# qm config 206 boot: cn bootdisk: scsi0 cores: 1 hotplug: disk,cpu memory: 1536 name: syslog net0: virtio=00:22:FF:00:00:14,bridge=vmbr1 onboot: 1 ostype: l26 scsi0: HP:vm-206-disk-0,size=32G scsihw: virtio-scsi-pci smbios1: uuid=c8668f3f-f46b-4a06-9bf3-58942b4965b3 sockets: 1 startup: order=206 vmgenid: ce5568be-628c-4eab-be22-0872e6ee6ee4