[make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified

[make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified

[Leonid Krivoshein]

---
  .../data/lib/bootchain/overlayfs              | 21 ++++++++++---------
  1 file changed, 11 insertions(+), 10 deletions(-)

diff --git 
a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs 
b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
index 23b869b..48f9b7c 100755
--- a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
+++ b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
@@ -1,26 +1,27 @@
  #!/bin/bash -efu
  -. pipeline-sh-functions
+. bootchain-sh-functions
   target="$(get_parameter OVERLAYFS)"
   [ -n "$target" ] || [ -n "$prevdir" ] ||
-	fatal "no previous step to use as lower directory"
+	fatal "no previous step results to use as lower directory"
  -pipe="${target:-$prevdir}"
+chain="${target:-$prevdir}"
  lowerdirs=
  -while [ -n "$pipe" ]; do
-	name="${pipe%%,*}"
+while [ -n "$chain" ]; do
+	name="${chain%%,*}"
   	dir="$(resolve_target "$name")"
  	lowerdirs="${lowerdirs:+$lowerdirs,}$dir"
  -	pipe="${pipe#$name}"
-	pipe="${pipe#,}"
+	chain="${chain#$name}"
+	chain="${chain#,}"
  done
  -mkdir -p -- "$datadir/rw" "$datadir/work"
+run mkdir -p -- "$datadir/rw" "$datadir/work"
  -run mount -o 
"lowerdir=$lowerdirs,upperdir=$datadir/rw,workdir=$datadir/work" \
-	-t overlay overlay "$destdir"
+opts="lowerdir=$lowerdirs,upperdir=$datadir/rw,workdir=$datadir/work"
+
+run mount -t overlay -o "$opts" -- overlay "$destdir"
-- 
2.21.0

Re: [make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified

[Alexey Gladkov]

On Fri, Sep 24, 2021 at 06:56:41PM +0300, Leonid Krivoshein wrote:
> ---
>  .../data/lib/bootchain/overlayfs              | 21 ++++++++++---------
>  1 file changed, 11 insertions(+), 10 deletions(-)
> 
> diff --git
> a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
> b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
> index 23b869b..48f9b7c 100755
> --- a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
> +++ b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
> @@ -1,26 +1,27 @@
>  #!/bin/bash -efu
>  -. pipeline-sh-functions
> +. bootchain-sh-functions
>   target="$(get_parameter OVERLAYFS)"
>   [ -n "$target" ] || [ -n "$prevdir" ] ||
> -	fatal "no previous step to use as lower directory"
> +	fatal "no previous step results to use as lower directory"
>  -pipe="${target:-$prevdir}"
> +chain="${target:-$prevdir}"
>  lowerdirs=
>  -while [ -n "$pipe" ]; do
> -	name="${pipe%%,*}"
> +while [ -n "$chain" ]; do
> +	name="${chain%%,*}"
>   	dir="$(resolve_target "$name")"
>  	lowerdirs="${lowerdirs:+$lowerdirs,}$dir"
>  -	pipe="${pipe#$name}"
> -	pipe="${pipe#,}"
> +	chain="${chain#$name}"
> +	chain="${chain#,}"
>  done
>  -mkdir -p -- "$datadir/rw" "$datadir/work"
> +run mkdir -p -- "$datadir/rw" "$datadir/work"
>  -run mount -o
> "lowerdir=$lowerdirs,upperdir=$datadir/rw,workdir=$datadir/work" \
> -	-t overlay overlay "$destdir"
> +opts="lowerdir=$lowerdirs,upperdir=$datadir/rw,workdir=$datadir/work"
> +
> +run mount -t overlay -o "$opts" -- overlay "$destdir"

Очередной "intermediate commit" (c) ? В описании одна строчка:

overlayfs step modified

То что он modified говорит сам патч, да я это вижу. Добавился
bootchain-sh-functions, какие-то переименования ...

Что это за изменения ?
Зачем это делается ?
Что это решает ?

Леонид, такие патчи не возможно ревьювить. Это какая-то нарезка каких-то
логически необоснованных изменений.

-- 
Rgrds, legion

Re: [make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified

[Leonid Krivoshein]

26.09.2021 16:56, Alexey Gladkov пишет:
> On Fri, Sep 24, 2021 at 06:56:41PM +0300, Leonid Krivoshein wrote:
>> [...]
>> diff --git
>> a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
>> b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
>> [...]
> Очередной "intermediate commit" (c) ? В описании одна строчка:
>
> overlayfs step modified
>
> То что он modified говорит сам патч, да я это вижу. Добавился
> bootchain-sh-functions, какие-то переименования ...
>
> Что это за изменения ?
> Зачем это делается ?
> Что это решает ?
>
> Леонид, такие патчи не возможно ревьювить. Это какая-то нарезка каких-то
> логически необоснованных изменений.

Базовые шаги mountfs, overlayfs, getimage и даже waitdev в нашей 
загрузочной цепочке вообще не используются, изменения в них 
непринципиальны и ни на что не повлияют. Если 38/41 (added symlinks) 
рабочий при обновлении с любой версии пакета, то первые три можно не 
трогать совсем. Более полезные изменения, отражённые в документации, 
затронули waitdev и rootfs. Первый раз я всё же старался побыстрее дать 
общую картину крупным планом, поэтому не уделил внимание commit message. 
Остальные переименования в коде выглядят вполне логично, если смотреть 
их после основного кода, а он в самом конце.


-- 
Best regards,
Leonid Krivoshein.

Re: [make-initrd] [PATCH v1 12/41] fork pipeline: overlayfs step modified

[Alexey Gladkov]

On Sun, Sep 26, 2021 at 10:40:33PM +0300, Leonid Krivoshein wrote:
> 
> 26.09.2021 16:56, Alexey Gladkov пишет:
> > On Fri, Sep 24, 2021 at 06:56:41PM +0300, Leonid Krivoshein wrote:
> > > [...]
> > > diff --git
> > > a/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
> > > b/make-initrd/features/bootchain-core/data/lib/bootchain/overlayfs
> > > [...]
> > Очередной "intermediate commit" (c) ? В описании одна строчка:
> > 
> > overlayfs step modified
> > 
> > То что он modified говорит сам патч, да я это вижу. Добавился
> > bootchain-sh-functions, какие-то переименования ...
> > 
> > Что это за изменения ?
> > Зачем это делается ?
> > Что это решает ?
> > 
> > Леонид, такие патчи не возможно ревьювить. Это какая-то нарезка каких-то
> > логически необоснованных изменений.
> 
> Базовые шаги mountfs, overlayfs, getimage и даже waitdev в нашей загрузочной
> цепочке вообще не используются, изменения в них непринципиальны и ни на что
> не повлияют.

Если это непринципиальные изменения и нигде не используются, то не делай
их вообще. Чем меньше кода для первоначального обсуждения, тем лучше.

> Если 38/41 (added symlinks) рабочий при обновлении с любой
> версии пакета, то первые три можно не трогать совсем. Более полезные
> изменения, отражённые в документации, затронули waitdev и rootfs. Первый раз
> я всё же старался побыстрее дать общую картину крупным планом, поэтому не
> уделил внимание commit message. Остальные переименования в коде выглядят
> вполне логично, если смотреть их после основного кода, а он в самом конце.

Я надеюсь в следующей версии патчей последовательность будет логичной ))

Я почему так на этом настаиваю: эти коммиты потом будут читать в том числе
и твои коллеги, когда добавлять что-то будут и у них не будет тебя,
который скажет "переименования в коде выглядят логично, если смотреть их
после основного кода". Они и я будут делать git-log/git-blame и читать или
вспоминать последовательно.

-- 
Rgrds, legion