From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM autolearn=ham autolearn_force=no version=3.4.1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ya.ru; s=mail; t=1638744034; bh=RRhpNnWw+Hs8lXi3xMA/mAbOagUqJWistN64ib0xLiw=; h=In-Reply-To:From:Subject:References:Date:Message-ID:To; b=DLl+tGnZ8gBmVrd+eA0jb06jVAVF2tpxollhUrkKjGPP8wGpLtlTLNoOdtQaFHN6/ W2/VG45bjkFnCvSqK2QGyppqo50lsuPtbUzTMXDrxHMF8GvfggZ/W6lTX+OEjB5p3M 8xnapf8v2irWatqsysX5B6nmBeqlQnxa3KWttkWU= Authentication-Results: iva8-bad74b408dcf.qloud-c.yandex.net; dkim=pass header.i=@ya.ru X-Yandex-Fwd: 2 Message-ID: <9ac2f87f-da88-6963-9ade-d99d198bcfdf@ya.ru> Date: Mon, 6 Dec 2021 05:40:31 +0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.2 Content-Language: ru To: devel-distro@lists.altlinux.org References: <45ac498a-58c3-1ee2-c421-b9067fe23ebb@ya.ru> <9343d143-47d5-b2c2-d3e3-f6e51dc258e8@ya.ru> From: =?UTF-8?B?0JDQvdGC0L7QvSDQnNC40LTRjtC60L7Qsg==?= In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [devel-distro] Q: virt-install vs vm/builder qcow2 X-BeenThere: devel-distro@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: Distributions development List-Id: Distributions development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 05 Dec 2021 22:40:36 -0000 Archived-At: List-Archive: 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/ для > каждого случая в отдельности, а не менять use/tty. > Я думаю, что можно сделать поведение зависимым от наличия темы grub. Нет темы grub, значит и gfxterm не нужен, использовать console. В iso образах с grub сейчас такая логика. Но этого недостаточно для того, чтобы работала virsh console. Нужно настраивать serial. Одновременная настройка console и serial приводит к артефактам, также как одновременная настройка console и gfxterm. Так что одновременная их настройка не кажется хорошей идеей. Из всего могу сделать вывод, что нужен внешний инструмент для донастройки образов. Так как всем не угодишь. Кто-то этот образ будет с графикой запускать, а кто-то без. Патч для включения console предлагаю такой: commit d0f4dfb7e15a9ae291dacf49a0030348c87258a4 (HEAD -> next) Author: Anton Midyukov 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" -- С уважением, Антон Мидюков