From: "Антон Мидюков" <midyukov-anton@ya.ru>
To: devel-distro@lists.altlinux.org
Subject: Re: [devel-distro] Q: virt-install vs vm/builder qcow2
Date: Mon, 6 Dec 2021 05:40:31 +0700
Message-ID: <9ac2f87f-da88-6963-9ade-d99d198bcfdf@ya.ru> (raw)
In-Reply-To: <YazNn+42O8KpYBb6@lks.home>
05.12.2021 21:33, Konstantin Lepikhov пишет:
> Hi Антон!
>
> On 12/05/2021, at 05:41:20 PM you wrote:
>
>>>> Он получается с gfxterm. Что мне кажется неправильным.
>>> ? gfxterm подключается только если выставлена переменная GRUB_UI:
>>>
>>> ifdef GRUB_UI
>>> GRUB_CFG := $(GRUB_CFG) gfxterm
>>> endif
>>>
>>
>> Этот код не имеет отношение к целям vm/, только для distro/, т.е. iso.
> Да, поэтому я его и умянул, что на результат он не влияет.
>
>>
>> Я посмотрел в grub вывод команды:
>> terminal_output
>> gfxterm
>>
>> Дело в том, что в собираемом образе в /etc/sysconfig/grub2 прописан
>> GRUB_TERMINAL_OUTPUT='gfxterm'
> Потому что это прилетает с grub2-common. Собственно, GRUB_TERMINAL это
> поведение и меняет. В общем, может быть стоит сделать use/grub/<console|terminal> для
> каждого случая в отдельности, а не менять use/tty.
>
Я думаю, что можно сделать поведение зависимым от наличия темы grub.
Нет темы grub, значит и gfxterm не нужен, использовать console.
В iso образах с grub сейчас такая логика.
Но этого недостаточно для того, чтобы работала virsh console.
Нужно настраивать serial.
Одновременная настройка console и serial приводит к артефактам,
также как одновременная настройка console и gfxterm.
Так что одновременная их настройка не кажется хорошей идеей.
Из всего могу сделать вывод, что нужен внешний инструмент для
донастройки образов. Так как всем не угодишь.
Кто-то этот образ будет с графикой запускать, а кто-то без.
Патч для включения console предлагаю такой:
commit d0f4dfb7e15a9ae291dacf49a0030348c87258a4 (HEAD -> next)
Author: Anton Midyukov <antohami@altlinux.org>
Date: Mon Dec 6 04:30:31 2021 +0700
build-vm: add 20-grub-terminal script for setup terminal_output
By default, GRUB_TERMINAL_OUTPUT='gfxterm' is configured.
This creates problems for virtual machines and single-board computers,
since they need output to console.
diff --git a/features.in/build-vm/image-scripts.d/20-grub-terminal b/features.in/build-vm/image-scripts.d/20-grub-terminal
new file mode 100755
index 0000000000..c898df7b06
--- /dev/null
+++ b/features.in/build-vm/image-scripts.d/20-grub-terminal
@@ -0,0 +1,14 @@
+#!/bin/sh -efu
+
+GRUB_CFG=/etc/sysconfig/grub2
+
+[ -s "$GRUB_CFG" ] || exit 0
+
+GRUBTHEME="$(cat "$GRUB_CFG" |grep '^GRUB_THEME=.*' |tail -n 1)"
+GRUBTHEME="${GRUBTHEME%/*}"
+GRUBTHEME="${GRUBTHEME##*/}"
+
+[ -z "$GRUBTHEME" ] || exit 0
+
+sed '/^GRUB_TERMINAL_OUTPUT=.*/d' -i "$GRUB_CFG"
+echo 'GRUB_TERMINAL_OUTPUT="console"' >> "$GRUB_CFG"
--
С уважением, Антон Мидюков <antohami@altlinux.org>
next prev parent reply other threads:[~2021-12-05 22:40 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-02 20:19 Konstantin Lepikhov
2021-12-03 0:32 ` Антон Мидюков
2021-12-03 8:23 ` Konstantin Lepikhov
2021-12-03 9:22 ` Антон Мидюков
2021-12-03 9:48 ` Konstantin Lepikhov
2021-12-03 9:55 ` Антон Мидюков
2021-12-03 10:14 ` Konstantin Lepikhov
2021-12-03 10:32 ` Антон Мидюков
2021-12-03 10:41 ` Konstantin Lepikhov
2021-12-03 10:54 ` Антон Мидюков
2021-12-03 11:04 ` Konstantin Lepikhov
2021-12-03 13:07 ` Konstantin Lepikhov
2021-12-03 14:21 ` Антон Мидюков
2021-12-04 21:04 ` Konstantin Lepikhov
2021-12-05 6:06 ` Антон Мидюков
2021-12-05 8:48 ` Konstantin Lepikhov
2021-12-05 8:54 ` Антон Мидюков
2021-12-05 10:16 ` Konstantin Lepikhov
2021-12-05 10:41 ` Антон Мидюков
2021-12-05 14:33 ` Konstantin Lepikhov
2021-12-05 22:40 ` Антон Мидюков [this message]
2021-12-05 22:48 ` Антон Мидюков
2021-12-06 10:36 ` Konstantin Lepikhov
2021-12-06 10:43 ` Антон Мидюков
2021-12-06 12:05 ` Konstantin Lepikhov
2022-04-29 11:41 ` Konstantin Lepikhov
2022-04-29 12:16 ` Антон Мидюков
2022-04-29 16:18 ` Konstantin Lepikhov
2022-04-29 16:54 ` Антон Мидюков
2022-04-29 18:04 ` Konstantin Lepikhov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9ac2f87f-da88-6963-9ade-d99d198bcfdf@ya.ru \
--to=midyukov-anton@ya.ru \
--cc=devel-distro@lists.altlinux.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
ALT Linux Distributions development
This inbox may be cloned and mirrored by anyone:
git clone --mirror http://lore.altlinux.org/devel-distro/0 devel-distro/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 devel-distro devel-distro/ http://lore.altlinux.org/devel-distro \
devel-distro@lists.altlinux.org devel-distro@lists.altlinux.ru devel-distro@lists.altlinux.com
public-inbox-index devel-distro
Example config snippet for mirrors.
Newsgroup available over NNTP:
nntp://lore.altlinux.org/org.altlinux.lists.devel-distro
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git