Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 2238854
Summary: | webUI: when installing in Russian, console layout is set to US | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Adam Williamson <awilliam> |
Component: | anaconda | Assignee: | Adam Williamson <awilliam> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | high | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | anaconda-maint, jkonecny, robatino, vslavik, w |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | anaconda-40.7-1 anaconda-39.32.4-1.fc39 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2023-10-13 16:20:20 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 2187794, 2231339 |
Description
Adam Williamson
2023-09-13 21:53:45 UTC
Fixing this might be a bit fun. We can try to replicate something like the very-special-purpose logic `set_x_keyboard_defaults` uses: "if there's more than one layout in the xkb layout list, and the second one does not support ASCII, pick that for the vconsole layout". Or, we can just pass the whole list of xkb layouts to `convert_layouts`, which should ultimately dump the problem on localed and its good ol' kbd-model-map lookup table...but that will actually only give us the right answer if we feed it the list "ru,us", whereas the order we actually want in X is "us,ru" (I think kbd-model-map was never updated after https://bugzilla.redhat.com/show_bug.cgi?id=1039185 ). We could possibly add new lines to kbd-model-map that are the same as the current ones for all "XX,us" entries, except reversed to be "us,XX". That would...probably solve the problem. Proposing as a Final blocker for F40, as a violation of "If a particular keyboard layout has been configured for the system, that keyboard layout must be used: ... When logging in at a console" (for these "fun" switched cases, we always read that to mean "if you install in Russian, you should get 'ru' as your console layout"). OK, so I think I have a plan to fix both this and https://bugzilla.redhat.com/show_bug.cgi?id=2239213 : make anaconda try less hard. Drop the special-case code for setting the console keymap on this live install path, and also drop the special-case code for setting it for switched layouts in the old set_x_keyboard_defaults path. In both cases, just get the X layouts right, then ask localed to convert. Right now this gives wrong results, but I have a patch that fixes localed to handle these cases better. If I can get that merged we can make anaconda much simpler. I need to tidy up both anaconda and systemd patches a bit and update the test suites, but I did some preliminary testing and it looks good. Thanks Adam for providing the fix and all the other work you have done for this! don't thank me till we work out what I broke :P FEDORA-2023-8d954317bd has been submitted as an update to Fedora 39. https://bodhi.fedoraproject.org/updates/FEDORA-2023-8d954317bd FEDORA-2023-8d954317bd has been pushed to the Fedora 39 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-8d954317bd` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-8d954317bd See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates. FEDORA-2023-8d954317bd has been pushed to the Fedora 39 stable repository. If problem still persists, please make note of it in this bug report. |