Make-initrd development discussion
 help / color / mirror / Atom feed
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] Fwd: [#269003] TESTED make-initrd.git=2.14.1-alt1
Date: Wed, 7 Apr 2021 15:13:27 +0200
Message-ID: <20210407131327.tblzuljdgbbsgrbe@example.org> (raw)
In-Reply-To: <a2ba677d-e65d-9e9c-cb05-5c4eec3688b0@gmail.com>

On Tue, Apr 06, 2021 at 08:38:27PM +0300, Leonid Krivoshein wrote:
> 
> 06.04.2021 11:44, Alexey Gladkov пишет:
> > On Tue, Apr 06, 2021 at 01:51:30AM +0300, Leonid Krivoshein wrote:
> > > 8. Можно сделать общее описание входа/выхода для всех поддерживаемых шагов и
> > > выполнять необходимые проверки до и после выполнения шага, чтобы не не
> > > делать этого внутри самих шагов. Такое описание будет полезно и для шага
> > > debug. Шаги могут быть транзитными (pass-thru).
> > Я не хочу навязывать что должны шага принимать и уж тем более нельзя
> > навязывать, что они должны возвращать. Например, waitdev ничего не
> > монтирует т.е. на выходе нет ничего кроме задержки перед следующим шагом.
> > 
> > Могут быть шаги, которые будут спрашивать что-то у пользователя. Описывать
> > такое очень сложно.
> 
> Можно сделать необязательным описание входа. Если не описано, считать, что
> на входе может быть что угодно и задача шага -- проверить это самостоятельно
> и вывести fatal(). Если же есть описание, проверку может делать сам
> pipeline. ANY -- что угодно, DEV -- устройство, DIR -- каталог, PASS -- шаг
> не обрабатывает вход, вместо этого он должен быть напрямую связан с выходом
> без обработки, т.е. в данном случае pipeline должен передать выход
> предыдущего шага на вход следующего шага или первого, который не PASS.
> 
> Иначе каждый шаг начинается с проверок и вывода fatal(), а в конце сейчас
> приходится использовать специально написанную для данного случая
> pass_thru_pipeline(). Именно эту однотипную и примитивную обработку
> предлагаю перетащить в pipeline, чтобы не делать её на каждом шаге.

Я в принципе не против перераспределения обязанностей между шагами и
pipelined. Но я считаю, что децентрализация лучше. Сейчас шаги
предоставлены себе. Они могут быть написаны на bash, на си, на lua.
Демону это не важно. Он лишь отвечает за несколько простых вещей:

1. подготовить директории для данных шага и для результата.
2. запустить программу шага и проконтролировать код возврата.

Он обменивается с шагами только 5ю переменными окружения, которые можно
обработать на любом языке. Остальное шаги делают сами.

Если хочется автоматизировать получение параметра для bash, то можно
написать common-script, который будет сорсится в шагах и который будет
делать:

check_parameter PARAM
param="$(get_parameter PARAM)"

Я не понимаю зачем вводить какое-то усложнение без видимого профита. Ты
рассказываешь про какие-то изменения, но мало пишешь, что это даст... ну
знаешь... было то-то, это непозволяет сделать вот это, если поменять это,
то получится так.

-- 
Rgrds, legion



  reply	other threads:[~2021-04-07 13:13 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-05 20:33 ` Leonid Krivoshein
2021-04-05 22:51   ` Leonid Krivoshein
2021-04-06  8:44     ` Alexey Gladkov
2021-04-06 17:38       ` Leonid Krivoshein
2021-04-07 13:13         ` Alexey Gladkov [this message]
2021-04-06  8:28   ` Alexey Gladkov
2021-04-06 16:38     ` Leonid Krivoshein
2021-04-06 19:05       ` Alexey Gladkov
2021-04-06 19:30         ` Alexey Gladkov
2021-04-06 23:13           ` Leonid Krivoshein
2021-04-07 12:28             ` Alexey Gladkov
2021-04-06 23:00         ` Leonid Krivoshein
2021-04-07 12:11           ` Alexey Gladkov
2021-04-06 23:59         ` Leonid Krivoshein
2021-04-07  1:51     ` Leonid Krivoshein
2021-04-07 12:57       ` Alexey Gladkov
2021-04-07 18:29         ` Leonid Krivoshein
2021-05-26 15:05         ` Leonid Krivoshein
2021-05-26 18:12           ` Alexey Gladkov
2021-05-26 19:25             ` Leonid Krivoshein
2021-05-27  8:37               ` Alexey Gladkov
2021-05-27 12:29                 ` Leonid Krivoshein
2021-05-27 13:53                   ` Alexey Gladkov
2021-05-27 15:10                     ` Leonid Krivoshein
2021-05-27 17:04                       ` Alexey Gladkov
2021-05-27 17:11                         ` Leonid Krivoshein
2021-05-30 20:34                     ` Leonid Krivoshein

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=20210407131327.tblzuljdgbbsgrbe@example.org \
    --to=gladkov.alexey@gmail.com \
    --cc=make-initrd@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

Make-initrd development discussion

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/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 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
		make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
	public-inbox-index make-initrd

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.make-initrd


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git