Fix cc_keyboard in mantic #4361
Conversation
localectl set-x11-keymap is currently broken for new debian based installs. Workaround it by writing /etc/default/keyboard directly LP: #2030788
| f'XKBVARIANT="{variant}"', | ||
| f'XKBOPTIONS="{options}"', | ||
| "", | ||
| 'BACKSPACE="guess"', # This is provided on default installs |
There was a problem hiding this comment.
Might be a nitpick... Is it possible to allow the user to configure this, but still default to "guess" incase they do not configure this option.
There was a problem hiding this comment.
While a good thought, the implementation is highly debian-specific given that this implementation is writing directly to /etc/default/keyboard. Surfacing such and option is something that won't be supportable easily on any other distribution that is calling localectl directly, because that command doesn't support setting backspace values.
cloudinit/distros/debian.py
Outdated
| util.write_file( | ||
| filename="/etc/default/keyboard", | ||
| content=contents, | ||
| mode=644, |
There was a problem hiding this comment.
Mode here needs to be octal not decimal, this resulted in some very bad behavior on mantic when cloud-init wrote the file with perms 1204 :)
root@testlocale-manticdaily:~# localectl status
Could not get properties: Access Denied
root@testlocale-manticdaily:~# ls -l /etc/default/keyboard
-rw-r----T 1 root root 120 Aug 25 02:31 /etc/default/keyboard
More details in the bug
Proposed Commit Message
Additional Context
https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/2030788
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1038762
man 5 keyboardTest Steps
I haven't been able to verify this manually, so it may be worth asking the submitter to verify first