ALT Linux Distributions development
 help / color / mirror / Atom feed
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>


  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