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] Upcoming changes
Date: Thu, 4 May 2023 11:05:52 +0200
Message-ID: <ZFN1cAZuWz8M0oD6@example.org> (raw)
In-Reply-To: <a6c019c0-f681-b3bd-2517-e25a48e867f1@gmail.com>

On Thu, May 04, 2023 at 07:05:15AM +0300, Leonid Krivoshein wrote:
> Привет!
> 
> 
> On 5/3/23 17:35, Alexey Gladkov wrote:
> > Привет!
> >
> > Я хотел бы рассказать, что готовится. Я сейчас тестирую некоторые
> > изменения в архитектуре.
> >
> > * Я переписал ueventd с bash + утилита inotify на нормальный си. Новый код
> >    легче читать и понять.
> 
> Это же здорово! И в какой-то степени ожидаемо. Только я чур на systemd 
> не намекал. :-) Есть ли шанс, что уберутся sleep .1 по всему коду? Даёшь 
> тепло в дом другим способом!

sleep'ов стенет меньше, но всё ещё останутся. Чтобы от них избавится, то
нужно придумать сущность, которая бы без поллигинга контролировала условия
загрузки.

Возможно, следующим шагом попробую эту область переделать, если никто
раньше не предложит что-то.

> > * polld перестал быть отдельным демоном (сервис остаётся на месте) и
> >    становится просто отдельной очередью в ueventd. Скрипты, которые
> >    выполняются в ueventd. Очередь запускается и останавливается из сервиса.
> >
> > Несмотря на то, что изменение большое я не думаю, что это как-то затронет
> > фичи не из репозитория.
> >
> > Пользуясь случаем я также хотел спросить, стоит ли присылать в этот список
> > рассылки патчсеты, которые готовятся для master ?
> 
> Тех, кто разберётся в твоём коде, не так уж много, и всё увидят в 
> конечном продукте и дадут обратную связь так или иначе.

Я стараюсь писать как можно проще. Судя по всему, мне не удалось писать
понятно ((

> Или ты сам себе не доверяешь и хочешь перестраховаться, чтобы кто-то
> поревьювил?

Я никогда себе не доверял. Тут скорее вопрос в том, есть ли тут желающие
поревьювить. Иначе такие патчи будут просто спамом.

> В принципе хорошая практика даже в Сизиф не отправлять, без тестирования
> на регулярках, а для этого можно договориться с antohami@, но и это
> совершенно необязательно, т.к. в худшем случае забракуется очередная
> партия регулярок. Ну и системы немного поломаются у тех, кто на Сизифе
> сидит, а их не так много, я полагаю.

Я отправляю изменения не прямо уж без тестирования. У меня всё-таки есть
end-to-end тесты для разных схем загрузки и разных вендоров. Там
тестируется как kickstart, так и сама загрузка. Разумеется, это не
настоящее железо и не настоящая система, но всё-таки.

-- 
Rgrds, legion



  reply	other threads:[~2023-05-04  9:05 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-03 14:35 Alexey Gladkov
2023-05-04  4:05 ` Leonid Krivoshein
2023-05-04  9:05   ` Alexey Gladkov [this message]
2023-05-04  9:39     ` Leonid Krivoshein
2023-05-04 14:15       ` Alexey Gladkov
2023-05-05  3:06         ` 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=ZFN1cAZuWz8M0oD6@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