Skip to content

Conversation

@tonynguien
Copy link
Contributor

@tonynguien tonynguien commented Oct 23, 2025

Problem

To update keymapping, /usr/bin/delphix-startup-screen uses sudo localectl set-x11-keymap which is no longer the supported mechanism in Ubuntu 24.04 release. Below shows the error when running the command

delphix@localhost:~$ sudo localectl set-x11-keymap us
Setting X11 and console keymaps is not supported in Debian.

Solution

Set the keyboard layout by editing /etc/default/keyboard and applying it with setupcon. This mechanism avoids localectl set-x11-keymap (which is disabled on Debian/Ubuntu builds).

Notes:
- This change does NOT trigger update-initramfs; only applies to the running system's console. Delphix
doesn't have TTY interactions before pivot-root, e.g. encrypted root passphrase, and
rescue shell can be done with the default "us" layout.
- dpkg-reconfigure keyboard-configuration is another way to update keymap which would also do update-initramfs which is unnecessary and more involved, mounting and unmounting EFI partition.

Testing Done

Updating to different keyboard layouts and verified dumpkeys(1) produced different key mappings.

-rw-r-----  1 delphix staff 470395 Oct 24 19:19 dumpkeys.gb
-rw-r-----  1 delphix staff 446504 Oct 24 19:21 dumpkeys.jp
-rw-r-----  1 delphix staff 453750 Oct 24 19:21 dumpkeys.us.1
-rw-r-----  1 delphix staff 477181 Oct 24 19:23 dumpkeys.de

@tonynguien tonynguien force-pushed the dlpx/pr/tonynguien/d4b30b5f-e06d-4148-9a8a-0f29a0d0b486 branch 2 times, most recently from 4a96b21 to f0743fc Compare October 24, 2025 17:24
@tonynguien tonynguien force-pushed the dlpx/pr/tonynguien/d4b30b5f-e06d-4148-9a8a-0f29a0d0b486 branch from f0743fc to 3f3e3c7 Compare October 24, 2025 19:55
@tonynguien tonynguien marked this pull request as ready for review October 24, 2025 19:56
@tonynguien tonynguien merged commit b4b9b8f into develop Oct 24, 2025
21 of 25 checks passed
@tonynguien tonynguien deleted the dlpx/pr/tonynguien/d4b30b5f-e06d-4148-9a8a-0f29a0d0b486 branch October 24, 2025 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants