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

[Alexey Gladkov <gladkov.alexey@gmail.com>]

On Sun, Sep 26, 2021 at 10:53:08PM +0300, Leonid Krivoshein wrote:
> 
> 26.09.2021 17:09, Alexey Gladkov пишет:
> > On Fri, Sep 24, 2021 at 06:57:45PM +0300, Leonid Krivoshein wrote:
> > 
> > Что это за хуки и зачем они нужны ?
> 
> Отлично, уже дошли до первого принципиального и важного изменения!
> Полагаю, тут больше камень в огород отсутствия описаний, это я поправлю.
> 
> Стартовый скрипт SysVinit ранее включал prepare() для предварительной
> подготовки каталогов для компонента waitdev, отделённого теперь в другую
> фичу.

Когда ты говоришь про prepare, ты про выполнение скриптов перед сервисом
(udev, ueventd, etc.) ?

> Другим компонентам 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

Ты имеешь в виду вот такие хуки ?

> > > ---
> > >   .../data/etc/rc.d/init.d/bootchain            | 33 ++++++++++---------
> > >   .../data/lib/bootchain-prepare.d/300-waitdev  | 12 +++++++
> > >   2 files changed, 29 insertions(+), 16 deletions(-)
> > >   create mode 100644 make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev
> > > 
> > > diff --git
> > > a/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain
> > > b/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain
> > > index 89d9de5..7bdb1cf 100755
> > > --- a/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain
> > > +++ b/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain
> > > @@ -1,6 +1,6 @@
> > >   #!/bin/bash
> > >   ### BEGIN INIT INFO
> > > -# Provides:            pipeline
> > > +# Provides:            bootchain
> > >   # Required-Start:      uevent udev
> > >   # Should-Start:
> > >   # Required-Stop:
> > > @@ -13,27 +13,28 @@
> > >   . /.initrd/initenv
> > >   . /etc/init.d/template
> > >   -NAME=pipelined
> > > +NAME=bootchained
> > >   PIDFILE="/var/run/$NAME.pid"
> > >   ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME --displayname
> > > $NAME"
> > >   -prepare() {
> > > -	local dir i n
> > > -
> > > -	dir=/.initrd/pipeline/waitdev
> > > -	mkdir -p -- "$dir"
> > > -
> > > -	i=0
> > > -	while [ "$i" -lt "${WAITDEV:-0}" ]; do
> > > -		touch "$dir/$i"
> > > -		i=$(($i + 1))
> > > -	done
> > > +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-}" = pipeline ]; then
> > > -		prepare
> > > +	if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then
> > > +		use_hooks /lib/bootchain-prepare.d
> > >   		start_daemon --background $ARGS "$NAME"
> > >   		RETVAL=$?
> > >   	fi
> > > @@ -43,7 +44,7 @@ start() {
> > >   stop() {
> > >   	stop_daemon $ARGS "$NAME"
> > >   	RETVAL=$?
> > > -	[ ! -f "$PIDFILE" ] || rm -f -- "$PIDFILE"
> > > +	rm -f -- "$PIDFILE"
> > >   	return $RETVAL
> > >   }
> > >   diff --git a/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev
> > > new file mode 100644
> > > index 0000000..f7315a3
> > > --- /dev/null
> > > +++ b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev
> > > @@ -0,0 +1,12 @@
> > > +#!/bin/bash -efu
> > > +
> > > +dir=/.initrd/bootchain/waitdev
> > > +mkdir -p -- "$dir"
> > > +
> > > +[ -z "${WAITDEV_TIMEOUT-}" ] ||
> > > +	printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir/TIMECNT"
> > > +i=0
> > > +while [ "$i" -lt "${WAITDEV:-0}" ]; do
> > > +	touch "$dir/$i"
> > > +	i=$((1 + $i))
> > > +done
> > > -- 
> > > 2.21.0
> > > 
> > > 
> > > _______________________________________________
> > > Make-initrd mailing list
> > > Make-initrd@lists.altlinux.org
> > > https://lists.altlinux.org/mailman/listinfo/make-initrd
> > > 
> 
> -- 
> Best regards,
> Leonid Krivoshein.
> 
> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Rgrds, legion