Re: [make-initrd] [PATCH v1 21/41] fork pipeline: bootchain sysvinit script now use external hooks

[Leonid Krivoshein <klark.devel@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2982 bytes --]


27.09.2021 20:22, Leonid Krivoshein пишет:
>
> 27.09.2021 16:57, Alexey Gladkov пишет:
>> On Mon, Sep 27, 2021 at 04:11:43PM +0300, Leonid Krivoshein wrote:
>>> [...]
>>>>> Другим компонентам bootchain при расширении (а теперь это уже более
>>>>> десятка фич и подпакетов) тоже может потребоваться подобная 
>>>>> инициализация.
>>>>> Здесь в одном коммите видно, куда переехал код функции prepare(). 
>>>>> Хуки нужны
>>>>> для того, чтобы вызывать подобную инициализацию из определённого 
>>>>> места, так
>>>>> как базовый пакет bootchain-core может не знать, с какими 
>>>>> компонентами
>>>>> bootchain собран initramfs. Соответственно, включаемые скрипты 
>>>>> кладутся в
>>>>> /lib/bootchain-prepare.d. Если потребуется, по аналогии можно 
>>>>> добавить в
>>>>> stop() хуки для деинициализации.
>>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L32-L38 
>>>>
>>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L61
>>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L109 
>>>>
>>>> https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L114 
>>>>
>>>>
>>>> Ты имеешь в виду вот такие хуки ?
>>> Ну очень похожие, хотя use_hooks() явно проще -- без блокировки 
>>> консоли, без
>>> бита выполнения, просто соурсим исходники:
>> Эти хуки уже давно есть и можно использовать. Не нужно дублировать 
>> этот же
>> функционал.
>
> Дойдя до этого места не понял, как их можно использовать...
>
> Делаю скрипт выполняемым, а класть-то его куда? В 
> /lib/initrd/pre/bootchain-core/ ?
>

Проверочная сборка показала, что этот путь не работает.

Если загрузиться с параметрами waitdev=UUID=1234 waitdev_timeout=180 
rdshell и запустить из rdshell руками 
/lib/initrd/pre/bootchain-core/300-waitdev , только тогда видно, что 
правильно отрабатывает. Но автоматом хук не запускается. В коде есть 
механизм запуска, но он не используется ни в одной фиче и в data/ его 
тоже нет, только сам этот скрипт rc. Вот изменения, на всякий случай:



-- 
Best regards,
Leonid Krivoshein.


[-- Attachment #2: waitdev.diff --]
[-- Type: text/x-patch, Size: 1916 bytes --]

commit d5340fea7efcf9995b7391a17b9781462556e661
Author: Leonid Krivoshein <klark@altlinux.org>
Date:   Tue Sep 28 00:14:28 2021 +0300

    core+waitdev: use common way for hooks call instead internal use_hooks()
    
    See also:
    https://lists.altlinux.org/pipermail/make-initrd/2021-September/000685.html

diff --git a/bootchain-core/data/etc/rc.d/init.d/bootchain b/bootchain-core/data/etc/rc.d/init.d/bootchain
index 7bdb1cf..1538339 100755
--- a/bootchain-core/data/etc/rc.d/init.d/bootchain
+++ b/bootchain-core/data/etc/rc.d/init.d/bootchain
@@ -17,24 +17,9 @@ NAME=bootchained
 PIDFILE="/var/run/$NAME.pid"
 ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME --displayname $NAME"
 
-use_hooks()
-{
-	local hook hdir="$1"
-
-	if [ -d "$hdir" ]; then
-		# shellcheck disable=SC2012
-		for hook in $(ls -1 -- "$hdir"/* |sort) _; do
-			[ -s "$hook" ] ||
-				continue
-			. "$hook"
-		done
-	fi
-}
-
 start() {
 	RETVAL=0
 	if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then
-		use_hooks /lib/bootchain-prepare.d
 		start_daemon --background $ARGS "$NAME"
 		RETVAL=$?
 	fi
diff --git a/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev b/bootchain-waitdev/data/lib/initrd/pre/bootchain-core/300-waitdev
old mode 100644
new mode 100755
similarity index 71%
rename from bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev
rename to bootchain-waitdev/data/lib/initrd/pre/bootchain-core/300-waitdev
index f7315a3..5e0f040
--- a/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev
+++ b/bootchain-waitdev/data/lib/initrd/pre/bootchain-core/300-waitdev
@@ -1,10 +1,13 @@
 #!/bin/bash -efu
 
+. /.initrd/initenv
+
 dir=/.initrd/bootchain/waitdev
 mkdir -p -- "$dir"
 
 [ -z "${WAITDEV_TIMEOUT-}" ] ||
-	printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir/TIMECNT"
+	printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir"/TIMECNT
+
 i=0
 while [ "$i" -lt "${WAITDEV:-0}" ]; do
 	touch "$dir/$i"