From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id 016851FF139 for ; Tue, 24 Feb 2026 10:23:23 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id BD467574; Tue, 24 Feb 2026 10:24:17 +0100 (CET) Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 24 Feb 2026 10:24:12 +0100 To: "Michael Robert Jimenez" Subject: Re: [PATCH docs] sysadmin: add comment about update-initramfs needing to be run when changing the keyboard layout after installation due to KEYMAP=y in config for zfs-initramfs Message-Id: X-Mailer: aerc 0.20.0 References: <20260217152041.318107-1-m.jimenez@proxmox.com> In-Reply-To: <20260217152041.318107-1-m.jimenez@proxmox.com> From: "Shannon Sterz" X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1771925037012 X-SPAM-LEVEL: Spam detection results: 0 AWL 0.089 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: VAG3MLDWRTVXXFB4SUIW7MHWWAJSJ65I X-Message-ID-Hash: VAG3MLDWRTVXXFB4SUIW7MHWWAJSJ65I X-MailFrom: s.sterz@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Michael Robert Jimenez , pve-devel@lists.proxmox.com X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: hey, note that your commit message should not exceed 72 characters as outlined in our developer documentation [1]. [1]: https://pve.proxmox.com/wiki/Developer_Documentation#Commits_and_Commi= t_Messages in this case may i suggest something like this as the main commit message: > sysadmin: document running update-initramfs on keyboard layout change and then a more elaborate explanation in the body (form stoiko's message in chate): > zfs-initramfs adds KEYMAP =3D y to the initramfs.conf > (/usr/share/initramfs-tools/conf-hooks.d/zfs). this in turn causes > /etc/console-setup to be included in the initramfs and this is not > updated automatically when reconfiguring the keyboard-configuration. > so initramfs needs to be update when changing the keyboard layout in > these cases. that should make it easier to glimpse what this commit is about while preserving more information. thanks :) On Tue Feb 17, 2026 at 4:20 PM CET, Michael Robert Jimenez wrote: > Signed-off-by: Michael Robert Jimenez > --- > keyboard-layout.adoc | 20 ++++++++++++++++++++ > sysadmin.adoc | 4 ++++ > 2 files changed, 24 insertions(+) > create mode 100644 keyboard-layout.adoc > > diff --git a/keyboard-layout.adoc b/keyboard-layout.adoc > new file mode 100644 > index 0000000..4693753 > --- /dev/null > +++ b/keyboard-layout.adoc > @@ -0,0 +1,20 @@ > +[[chapter_keyboard_layout]] > +Keyboard Layout > +--------------- > +ifdef::wiki[] > +:pve-toplevel: > +endif::wiki[] > + > + > +While the {pve} Installer initially configures the keyboard layout, it i= s possible to change it later by running: nit: the above line exceeds 80 characters which is the limit we try to enforce for documentation here > + > +---- > +# dpkg-reconfigure keyboard-configuration > +---- > + nit: unnecessary empty line above. > + > +You need to update the initramfs to persist the change after rebooting: > + > +---- > +# update-initramfs -k all -u > +---- > diff --git a/sysadmin.adoc b/sysadmin.adoc > index dd43f73..4d55835 100644 > --- a/sysadmin.adoc > +++ b/sysadmin.adoc > @@ -52,6 +52,8 @@ See Also > > * link:/wiki/Kernel_Samepage_Merging_(KSM)[Kernel Samepage Merging (KSM)= ] > > +* link:/wiki/Keyboard_Layout[Keyboard Layout] > + > endif::wiki[] > > > @@ -85,6 +87,8 @@ include::system-booting.adoc[] > > include::kernel-samepage-merging.adoc[] > > +include::keyboard-layout.adoc[] > + > endif::wiki[] > >