ALT Linux Distributions development
 help / color / mirror / Atom feed
* [devel-distro] installer-feature-* и rootfs
@ 2019-12-06 13:12 Ivan A. Melnikov
  2019-12-06 13:23 ` Антон Мидюков
                   ` (3 more replies)
  0 siblings, 4 replies; 14+ messages in thread
From: Ivan A. Melnikov @ 2019-12-06 13:12 UTC (permalink / raw)
  To: devel-distro

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

Привет.

Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
одноплатников вроде Таволги) у нас система поставляется в виде
образа rootfs и ставится без помощи installer'а.

Это означает, что на таких системах не отрабатывают пакеты
installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
Вот интересные особенности simply, сделанные через
installer-feature-* и соответсвенно *не* присутсвующие
в сборках под, например, RPi4:

- installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
  сам всегда так делал)
- installer-feature-lightdm-stage3 -- настраивает lightdm gtk
  greeter (появляется индикатор раскладки клавиатуры) и делает первого
  пользователя выбранным после первой загрузки (маст хев,
  в alt-workstation-mipsel я делал костыль вместо этой фичи)
- installer-feature-sudo-enable-by-default-stage3 -- sudo по
  умолчанию для всех в группе wheel, "как в Ubuntu";
- installer-feature-symlinks-from-sbin -- симлинки для ip,
  sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
  в $PATH простого пользователя.

Понятно, что эти фичи хотелось бы видеть на всех платформах,
под которые мы выпускаем simply.

Чтобы не перепиливать всё сразу, предлагается для начала
сделать что-нибудь для desktop-like сборок, для первоначальной
настройки которых используется alterator-setup. У него
есть специальные хуки (setup-postinstall.d), так что можно
можно после его работы запустить скрипты из installer-feature-*,
чтобы они донастроили систему.

Для начала, я реализовал такой хук в отдельном пакете:

http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git

Он пока проходит обкатку в тестовой таске #242450, первое
тестирование продемонстрировало его работоспособность.

Поскольку до совсем недавнего времени я с installer-feature-*
не сталкивался, хотелось бы услышать от людей, которые в этом
понимают (если такие есть) какие-то мысли по этому поводу
и, возможно, code review.

Как думаете, не стоит ли включить такой хук непосредственно
в alterator-setup, а не в отдельный пакет?

-- 
  wbr,
    iv m.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
@ 2019-12-06 13:23 ` Антон Мидюков
  2019-12-06 14:16 ` Alexey Shabalin
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 14+ messages in thread
From: Антон Мидюков @ 2019-12-06 13:23 UTC (permalink / raw)
  To: devel-distro

06.12.2019 20:12, Ivan A. Melnikov пишет:
> [...]
> Как думаете, не стоит ли включить такой хук непосредственно
> в alterator-setup, а не в отдельный пакет?
>
Я думаю, что стоит включить прямо в alterator-setup.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
  2019-12-06 13:23 ` Антон Мидюков
@ 2019-12-06 14:16 ` Alexey Shabalin
  2019-12-06 14:29   ` Mikhail Efremov
  2019-12-06 20:23   ` [devel-distro] installer-feature-symlinks-from-sbin Dmitry V. Levin
  2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
  2019-12-24 16:45 ` Mikhail Efremov
  3 siblings, 2 replies; 14+ messages in thread
From: Alexey Shabalin @ 2019-12-06 14:16 UTC (permalink / raw)
  To: Distributions development

пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
>
> Привет.
>
> Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
> одноплатников вроде Таволги) у нас система поставляется в виде
> образа rootfs и ставится без помощи installer'а.
>
> Это означает, что на таких системах не отрабатывают пакеты
> installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
> Вот интересные особенности simply, сделанные через
> installer-feature-* и соответсвенно *не* присутсвующие
> в сборках под, например, RPi4:
>
> - installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
>   сам всегда так делал)
> - installer-feature-lightdm-stage3 -- настраивает lightdm gtk
>   greeter (появляется индикатор раскладки клавиатуры) и делает первого
>   пользователя выбранным после первой загрузки (маст хев,
>   в alt-workstation-mipsel я делал костыль вместо этой фичи)
> - installer-feature-sudo-enable-by-default-stage3 -- sudo по
>   умолчанию для всех в группе wheel, "как в Ubuntu";

Ага, мы красавчики. Типа в пакете мы такое не делаем - нехорошо, но
все дистрибутивы выпускаем с этой фичей, что бы было удобно. Внесите
уже это в пакет sudo по-умолчанию и не извращайтесь с installer.

> - installer-feature-symlinks-from-sbin -- симлинки для ip,
>   sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
>   в $PATH простого пользователя.

Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
категорически нельзя - эти симлинки не принадлежат никакому пакету.
Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
право на жизнь такого пакета(тем более installer-featute)

>
> Понятно, что эти фичи хотелось бы видеть на всех платформах,
> под которые мы выпускаем simply.
>
> Чтобы не перепиливать всё сразу, предлагается для начала
> сделать что-нибудь для desktop-like сборок, для первоначальной
> настройки которых используется alterator-setup. У него
> есть специальные хуки (setup-postinstall.d), так что можно
> можно после его работы запустить скрипты из installer-feature-*,
> чтобы они донастроили систему.
>
> Для начала, я реализовал такой хук в отдельном пакете:
>
> http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git
>
> Он пока проходит обкатку в тестовой таске #242450, первое
> тестирование продемонстрировало его работоспособность.
>
> Поскольку до совсем недавнего времени я с installer-feature-*
> не сталкивался, хотелось бы услышать от людей, которые в этом
> понимают (если такие есть) какие-то мысли по этому поводу
> и, возможно, code review.
>
> Как думаете, не стоит ли включить такой хук непосредственно
> в alterator-setup, а не в отдельный пакет?

Моё мнение, что все конфиги и настройки лучше упаковывать в пакеты,
все остальное - это неконтролируемые изменения системы. Потом никогда
не найдешь, кто же тебе так "помог".
Из недавнего в телеграм-канале:
Человек собрал rpm-пакет в системе(без hasher, а почему нет если пакет
простенький?). У него появилась зависимость на /bin/service, которую
не предоставляет ни один пакет. Симлинк /bin/service появился
благодаря installer-feature-symlinks-from-sbin.
Руки бы оторвал таким помощничкам.
Итог: старайтесь не создавать проблем пользователям из лучших
побуждений. Ну не зачем запускать service простому пользователю.
А то что su- минус не используют - допинайте уже мантейнера, что бы
этот минус был по-умолчанию. Никто не может объяснить, зачем нужен su
без минуса.


-- 
Alexey Shabalin

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
  2019-12-06 13:23 ` Антон Мидюков
  2019-12-06 14:16 ` Alexey Shabalin
@ 2019-12-06 14:21 ` Mikhail Efremov
  2019-12-06 14:48   ` Антон Мидюков
  2019-12-24 16:45 ` Mikhail Efremov
  3 siblings, 1 reply; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-06 14:21 UTC (permalink / raw)
  To: devel-distro

On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> Привет.
> 
> Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
> одноплатников вроде Таволги) у нас система поставляется в виде
> образа rootfs и ставится без помощи installer'а.
> 
> Это означает, что на таких системах не отрабатывают пакеты
> installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
> Вот интересные особенности simply, сделанные через
> installer-feature-* и соответсвенно *не* присутсвующие
> в сборках под, например, RPi4:
> 
> - installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
>   сам всегда так делал)
> - installer-feature-lightdm-stage3 -- настраивает lightdm gtk
>   greeter (появляется индикатор раскладки клавиатуры) и делает первого
>   пользователя выбранным после первой загрузки (маст хев,
>   в alt-workstation-mipsel я делал костыль вместо этой фичи)
> - installer-feature-sudo-enable-by-default-stage3 -- sudo по
>   умолчанию для всех в группе wheel, "как в Ubuntu";
> - installer-feature-symlinks-from-sbin -- симлинки для ip,
>   sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
>   в $PATH простого пользователя.

JFYI, https://bugzilla.altlinux.org/37503
Я думаю заменить installer-feature-symlinks-from-sbin на пакет.

> Понятно, что эти фичи хотелось бы видеть на всех платформах,
> под которые мы выпускаем simply.

Я собирался посмотреть в alterator-setup на этот счет еще при выпуске
workstation, но так и не добрался.

> Чтобы не перепиливать всё сразу, предлагается для начала
> сделать что-нибудь для desktop-like сборок, для первоначальной
> настройки которых используется alterator-setup. У него
> есть специальные хуки (setup-postinstall.d), так что можно
> можно после его работы запустить скрипты из installer-feature-*,
> чтобы они донастроили систему.
> 
> Для начала, я реализовал такой хук в отдельном пакете:
> 
> http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git

Да, я как раз думал про такой вариант как самый простой.
Когда-то я делал подобное для alterator-livecd, см.
http://git.altlinux.org/people/sem/packages/?p=livecd-installer-features.git
Вообще надо бы сделать единые хуки для installer, livecd и setup, но
это много работы...

> Он пока проходит обкатку в тестовой таске #242450, первое
> тестирование продемонстрировало его работоспособность.
> 
> Поскольку до совсем недавнего времени я с installer-feature-*
> не сталкивался, хотелось бы услышать от людей, которые в этом
> понимают (если такие есть) какие-то мысли по этому поводу
> и, возможно, code review.

Я посмотрел, помимо хуков initinstall и postinstall есть еще preinstall.
В исталляторе они запускаются так:
initinstall - в самом начале, при запуске инсталлятора.
preinstall - после установки пакетов.
postinstall - в самом конце, после всех шагов исталлятора.
Можно ли все эти хуки запускать просто в setup-postinstall.d в общем
случае неизвестно, зависит от того, что делает конкретная
installer-feature. Например, preinstall-хук может проводить какую-то
подготовительную работу для последующих шагов инсталлятора.

> Как думаете, не стоит ли включить такой хук непосредственно
> в alterator-setup, а не в отдельный пакет?

Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
для использования в инсталляторе и могут работать неправильно при таком
нештатном использовании. Т.е. installer-features и
rootfs-installer-features - это грязные хаки, нельзя бездумно
использовать с ними произвольную installer-feature. Всегда нужно
внимательно эту installer-feature прочитать и убедиться, что ее можно
так использовать.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 14:16 ` Alexey Shabalin
@ 2019-12-06 14:29   ` Mikhail Efremov
  2019-12-06 20:23   ` [devel-distro] installer-feature-symlinks-from-sbin Dmitry V. Levin
  1 sibling, 0 replies; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-06 14:29 UTC (permalink / raw)
  To: devel-distro

On Fri, 6 Dec 2019 17:16:07 +0300 Alexey Shabalin wrote:
> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
> >
> > Привет.
> >
> > Как известно, для одноплатников вроде RPi (и завёрнутых в коробку
> > одноплатников вроде Таволги) у нас система поставляется в виде
> > образа rootfs и ставится без помощи installer'а.
> >
> > Это означает, что на таких системах не отрабатывают пакеты
> > installer-feature-*. Мы с jqt4@ посмотрели, что же мы теряем.
> > Вот интересные особенности simply, сделанные через
> > installer-feature-* и соответсвенно *не* присутсвующие
> > в сборках под, например, RPi4:
> >
> > - installer-feature-bell-off-stage3 -- выключает bell в X (xset -b,
> >   сам всегда так делал)
> > - installer-feature-lightdm-stage3 -- настраивает lightdm gtk
> >   greeter (появляется индикатор раскладки клавиатуры) и делает первого
> >   пользователя выбранным после первой загрузки (маст хев,
> >   в alt-workstation-mipsel я делал костыль вместо этой фичи)
> > - installer-feature-sudo-enable-by-default-stage3 -- sudo по
> >   умолчанию для всех в группе wheel, "как в Ubuntu";  
> 
> Ага, мы красавчики. Типа в пакете мы такое не делаем - нехорошо, но
> все дистрибутивы выпускаем с этой фичей, что бы было удобно. Внесите
> уже это в пакет sudo по-умолчанию и не извращайтесь с installer.

Надеюсь, что не все. Раньше это вообще было только в Simply.
Не надо такое в пакет.

> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
> >   sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> >   в $PATH простого пользователя.  
> 
> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
> категорически нельзя - эти симлинки не принадлежат никакому пакету.
> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
> право на жизнь такого пакета(тем более installer-featute)

Да, лучше так.

> Моё мнение, что все конфиги и настройки лучше упаковывать в пакеты,
> все остальное - это неконтролируемые изменения системы. Потом никогда
> не найдешь, кто же тебе так "помог".

Нельзя упаковать в пакеты все изменения, которые бывает нужно сделать в
конкретном дистрибутиве. Инсталлер - это набор хаков, всегда так было.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
@ 2019-12-06 14:48   ` Антон Мидюков
  2019-12-06 16:21     ` Антон Мидюков
  0 siblings, 1 reply; 14+ messages in thread
From: Антон Мидюков @ 2019-12-06 14:48 UTC (permalink / raw)
  To: devel-distro

06.12.2019 21:21, Mikhail Efremov пишет:
> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> [...]
>> Как думаете, не стоит ли включить такой хук непосредственно
>> в alterator-setup, а не в отдельный пакет?
> Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
> для использования в инсталляторе и могут работать неправильно при таком
> нештатном использовании. Т.е. installer-features и
> rootfs-installer-features - это грязные хаки, нельзя бездумно
> использовать с ними произвольную installer-feature. Всегда нужно
> внимательно эту installer-feature прочитать и убедиться, что ее можно
> так использовать.

Я думаю, что не надо включать в собираемый rootfs installer-features, 
которые не предназначены для этой rootfs.

Ограничить типом собираемой цели в mkimage-profiles не проблема.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 14:48   ` Антон Мидюков
@ 2019-12-06 16:21     ` Антон Мидюков
  2019-12-09 14:19       ` Mikhail Efremov
  0 siblings, 1 reply; 14+ messages in thread
From: Антон Мидюков @ 2019-12-06 16:21 UTC (permalink / raw)
  To: devel-distro

06.12.2019 21:48, Антон Мидюков пишет:
> 06.12.2019 21:21, Mikhail Efremov пишет:
>> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
>> [...]
>>> Как думаете, не стоит ли включить такой хук непосредственно
>>> в alterator-setup, а не в отдельный пакет?
>> Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
>> для использования в инсталляторе и могут работать неправильно при таком
>> нештатном использовании. Т.е. installer-features и
>> rootfs-installer-features - это грязные хаки, нельзя бездумно
>> использовать с ними произвольную installer-feature. Всегда нужно
>> внимательно эту installer-feature прочитать и убедиться, что ее можно
>> так использовать.
>
> Я думаю, что не надо включать в собираемый rootfs installer-features, 
> которые не предназначены для этой rootfs.
>
> Ограничить типом собираемой цели в mkimage-profiles не проблема.
>
Мысль о бездумном использовании правильная. Давайте введём новую 
категорию features-common, которые подходят и для installer, и для 
livecd-install, и для alterator-setup. Поместим их скрипты, например, в 
/usr/share/install-common/. Научим выполнять оттуда скрипты всех троих. 
А затем останется постепенно выделить общие фичи, проверив, что они 
подходят всем троим. Тогда точно никому ничего не поломаем. А в названии 
feature укажем -common. Как идея?

Не такая уж и большая работа, как мне кажется. Но результат будет не 
здесь и не сейчас, конечно.

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-symlinks-from-sbin
  2019-12-06 14:16 ` Alexey Shabalin
  2019-12-06 14:29   ` Mikhail Efremov
@ 2019-12-06 20:23   ` Dmitry V. Levin
  2019-12-08  9:45     ` Andrey Cherepanov
  1 sibling, 1 reply; 14+ messages in thread
From: Dmitry V. Levin @ 2019-12-06 20:23 UTC (permalink / raw)
  To: devel-distro

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

On Fri, Dec 06, 2019 at 05:16:07PM +0300, Alexey Shabalin wrote:
> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
[...]
> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
> >   sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> >   в $PATH простого пользователя.
> 
> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
> категорически нельзя - эти симлинки не принадлежат никакому пакету.
> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
> право на жизнь такого пакета(тем более installer-featute)

Удалить installer-feature-symlinks-from-sbin немедленно!


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-symlinks-from-sbin
  2019-12-06 20:23   ` [devel-distro] installer-feature-symlinks-from-sbin Dmitry V. Levin
@ 2019-12-08  9:45     ` Andrey Cherepanov
  2019-12-08 10:16       ` Dmitry V. Levin
  0 siblings, 1 reply; 14+ messages in thread
From: Andrey Cherepanov @ 2019-12-08  9:45 UTC (permalink / raw)
  To: Distributions development, Dmitry V. Levin, devel-distro



6 декабря 2019 г. 23:23:35 GMT+03:00, "Dmitry V. Levin" <ldv@altlinux.org> пишет:
>On Fri, Dec 06, 2019 at 05:16:07PM +0300, Alexey Shabalin wrote:
>> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
>[...]
>> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
>> >   sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
>> >   в $PATH простого пользователя.
>> 
>> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
>> категорически нельзя - эти симлинки не принадлежат никакому пакету.
>> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
>> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
>> право на жизнь такого пакета(тем более installer-featute)
>
>Удалить installer-feature-symlinks-from-sbin немедленно!

Не вопрос. Перенесите симлинки в пакеты и он станет просто не нужен. Делался-то этот пакет, чтобы не ждать годами некоторых мейнтейнеров.

-- 
Andrey Cherepanov
cas@altlinux.org


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-symlinks-from-sbin
  2019-12-08  9:45     ` Andrey Cherepanov
@ 2019-12-08 10:16       ` Dmitry V. Levin
  0 siblings, 0 replies; 14+ messages in thread
From: Dmitry V. Levin @ 2019-12-08 10:16 UTC (permalink / raw)
  To: Andrey Cherepanov; +Cc: Distributions development

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

On Sun, Dec 08, 2019 at 12:45:05PM +0300, Andrey Cherepanov wrote:
> 6 декабря 2019 г. 23:23:35 GMT+03:00, "Dmitry V. Levin" <ldv@altlinux.org> пишет:
> >On Fri, Dec 06, 2019 at 05:16:07PM +0300, Alexey Shabalin wrote:
> >> пт, 6 дек. 2019 г. в 16:12, Ivan A. Melnikov <iv@altlinux.org>:
> >[...]
> >> > - installer-feature-symlinks-from-sbin -- симлинки для ip,
> >> >   sysctl, chkconfig, service, blkid, resolvconf, lsusb, lsof
> >> >   в $PATH простого пользователя.
> >> 
> >> Жесть какая. Я уже высказывал свое отношение cas@'у, что так делать
> >> категорически нельзя - эти симлинки не принадлежат никакому пакету.
> >> Сделайте честный пакет, типа root-utils-user и упакуйте туда эти
> >> симлинки(что бы можно было это легко удалить). Но я бы вообще подверг
> >> право на жизнь такого пакета(тем более installer-featute)
> >
> >Удалить installer-feature-symlinks-from-sbin немедленно!
> 
> Не вопрос. Перенесите симлинки в пакеты и он станет просто не нужен. Делался-то этот пакет, чтобы не ждать годами некоторых мейнтейнеров.

Некоторые из этих симлинков бессмысленные, их не должно быть вообще.
Остальные переносите сами, вы такие же мантейнеры, как и все остальные.


-- 
ldv

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 16:21     ` Антон Мидюков
@ 2019-12-09 14:19       ` Mikhail Efremov
  2019-12-10  7:06         ` Ivan A. Melnikov
  0 siblings, 1 reply; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-09 14:19 UTC (permalink / raw)
  To: devel-distro

On Fri, 6 Dec 2019 23:21:40 +0700 Антон Мидюков wrote:
> 06.12.2019 21:48, Антон Мидюков пишет:
> > 06.12.2019 21:21, Mikhail Efremov пишет:  
> >> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> >> [...]  
> >>> Как думаете, не стоит ли включить такой хук непосредственно
> >>> в alterator-setup, а не в отдельный пакет?  
> >> Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
> >> для использования в инсталляторе и могут работать неправильно при таком
> >> нештатном использовании. Т.е. installer-features и
> >> rootfs-installer-features - это грязные хаки, нельзя бездумно
> >> использовать с ними произвольную installer-feature. Всегда нужно
> >> внимательно эту installer-feature прочитать и убедиться, что ее можно
> >> так использовать.  
> >
> > Я думаю, что не надо включать в собираемый rootfs installer-features, 
> > которые не предназначены для этой rootfs.

И также отдельно включать {rootfs,livecd}-installer-features явно и
только если хорошо знаешь, что делаешь. Пусть живет отдельно, иначе
может создаться впечатление, что это штатная вещь, а не грязный хак,
которым это на самом деле является.

> > Ограничить типом собираемой цели в mkimage-profiles не проблема.
> >  
> Мысль о бездумном использовании правильная. Давайте введём новую 
> категорию features-common, которые подходят и для installer, и для 
> livecd-install, и для alterator-setup. Поместим их скрипты, например, в 
> /usr/share/install-common/. Научим выполнять оттуда скрипты всех троих. 
> А затем останется постепенно выделить общие фичи, проверив, что они 
> подходят всем троим. Тогда точно никому ничего не поломаем. А в названии 
> feature укажем -common. Как идея?

Вот только проблемы хотя бы с тем, какие стадии вообще есть и когда
какие хуки должны запускаться. В alterator-setup вообще нет установки
пакетов, например. Поэтому не очевидно когда должны запускаться
preinstall-хуки. И это далеко не единственное различие.
Нужно продумать какие нужны хуки, в какой момент они должны
запускаться, какие аргументы/переменные им должны передаваться и как
они будут называться. И все это должно быть более-менее согласованно для
installer, livecd-install и alterator-setup.

> Не такая уж и большая работа, как мне кажется. Но результат будет не 
> здесь и не сейчас, конечно.

Я бы предпочел сначала хорошенько все продумать прежде чем что-то
делать. Иначе нагородим еще один жуткий набор костылей.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-09 14:19       ` Mikhail Efremov
@ 2019-12-10  7:06         ` Ivan A. Melnikov
  0 siblings, 0 replies; 14+ messages in thread
From: Ivan A. Melnikov @ 2019-12-10  7:06 UTC (permalink / raw)
  To: Distributions development

On Mon, Dec 09, 2019 at 05:19:54PM +0300, Mikhail Efremov wrote:
> On Fri, 6 Dec 2019 23:21:40 +0700 Антон Мидюков wrote:
> > 06.12.2019 21:48, Антон Мидюков пишет:
> > > 06.12.2019 21:21, Mikhail Efremov пишет:  
> > >> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> > >> [...]  
> > >>> Как думаете, не стоит ли включить такой хук непосредственно
> > >>> в alterator-setup, а не в отдельный пакет?  
> > >> Лучше отдельный пакет, см. выше. Хуки в installer-feature-* писались
> > >> для использования в инсталляторе и могут работать неправильно при таком
> > >> нештатном использовании. Т.е. installer-features и
> > >> rootfs-installer-features - это грязные хаки, нельзя бездумно
> > >> использовать с ними произвольную installer-feature. Всегда нужно
> > >> внимательно эту installer-feature прочитать и убедиться, что ее можно
> > >> так использовать.  
> > >
> > > Я думаю, что не надо включать в собираемый rootfs installer-features, 
> > > которые не предназначены для этой rootfs.
> 
> И также отдельно включать {rootfs,livecd}-installer-features явно и
> только если хорошо знаешь, что делаешь. Пусть живет отдельно, иначе
> может создаться впечатление, что это штатная вещь, а не грязный хак,
> которым это на самом деле является.
> 
> > > Ограничить типом собираемой цели в mkimage-profiles не проблема.
> > >  
> > Мысль о бездумном использовании правильная. Давайте введём новую 
> > категорию features-common, которые подходят и для installer, и для 
> > livecd-install, и для alterator-setup. Поместим их скрипты, например, в 
> > /usr/share/install-common/. Научим выполнять оттуда скрипты всех троих. 
> > А затем останется постепенно выделить общие фичи, проверив, что они 
> > подходят всем троим. Тогда точно никому ничего не поломаем. А в названии 
> > feature укажем -common. Как идея?
> 
> Вот только проблемы хотя бы с тем, какие стадии вообще есть и когда
> какие хуки должны запускаться. В alterator-setup вообще нет установки
> пакетов, например. Поэтому не очевидно когда должны запускаться
> preinstall-хуки. И это далеко не единственное различие.
> Нужно продумать какие нужны хуки, в какой момент они должны
> запускаться, какие аргументы/переменные им должны передаваться и как
> они будут называться. И все это должно быть более-менее согласованно для
> installer, livecd-install и alterator-setup.

Я просмотрел то, что используется в simply; там что-то применимое
для rootfs происходит только в postinstall. Так что я оставил запуск
только postinstall хуков и в таком виде отправлю это в Сизиф, p9
и применю.

> > Не такая уж и большая работа, как мне кажется. Но результат будет не 
> > здесь и не сейчас, конечно.
> 
> Я бы предпочел сначала хорошенько все продумать прежде чем что-то
> делать. Иначе нагородим еще один жуткий набор костылей.

Мне кажется, что по большому счёту существующих костылей вполне
достатночно, нужно просто ими аккуратнee воспользоваться.

Из того, что я увидел, большая часть всего очень специфична для
installer'а и не требуется или должна делаться совсем по-другому
для livecd и rootfs. Включение сервисов и настройки control логичнее
мне кажется превести на фичи m-p, там для этого есть более
логичные и переносимые механизмы (например,
installer-feature-sudo-enable-by-default или installer-feature-nfs*).
Оставшиеся немногие вещи (вроде подстановки первого пользователя
в lightdm) мне кажется можно перетащить хоть в /etc/firsttime.d
(если на она выполняется до alterator-setup, это надо в любом
случае поправить).

-- 
  wbr,
    iv m.



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
                   ` (2 preceding siblings ...)
  2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
@ 2019-12-24 16:45 ` Mikhail Efremov
  2019-12-25  8:21   ` Ivan A. Melnikov
  3 siblings, 1 reply; 14+ messages in thread
From: Mikhail Efremov @ 2019-12-24 16:45 UTC (permalink / raw)
  To: devel-distro

On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> 
> Для начала, я реализовал такой хук в отдельном пакете:
> 
> http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git

Может пора его в p9? Я бы тоже хотел его использовать на aarch64.

-- 
WBR, Mikhail Efremov


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [devel-distro] installer-feature-* и rootfs
  2019-12-24 16:45 ` Mikhail Efremov
@ 2019-12-25  8:21   ` Ivan A. Melnikov
  0 siblings, 0 replies; 14+ messages in thread
From: Ivan A. Melnikov @ 2019-12-25  8:21 UTC (permalink / raw)
  To: Distributions development

On Tue, Dec 24, 2019 at 07:45:57PM +0300, Mikhail Efremov wrote:
> On Fri, 6 Dec 2019 17:12:23 +0400 Ivan A. Melnikov wrote:
> > 
> > Для начала, я реализовал такой хук в отдельном пакете:
> > 
> > http://git.altlinux.org/people/iv/packages/?p=rootfs-installer-features.git
> 
> Может пора его в p9? Я бы тоже хотел его использовать на aarch64.
> 

#243514

Пересоздал чтобы сразу с последним alterator-setup.

-- 
  wbr,
    iv m.


^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2019-12-25  8:21 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-06 13:12 [devel-distro] installer-feature-* и rootfs Ivan A. Melnikov
2019-12-06 13:23 ` Антон Мидюков
2019-12-06 14:16 ` Alexey Shabalin
2019-12-06 14:29   ` Mikhail Efremov
2019-12-06 20:23   ` [devel-distro] installer-feature-symlinks-from-sbin Dmitry V. Levin
2019-12-08  9:45     ` Andrey Cherepanov
2019-12-08 10:16       ` Dmitry V. Levin
2019-12-06 14:21 ` [devel-distro] installer-feature-* и rootfs Mikhail Efremov
2019-12-06 14:48   ` Антон Мидюков
2019-12-06 16:21     ` Антон Мидюков
2019-12-09 14:19       ` Mikhail Efremov
2019-12-10  7:06         ` Ivan A. Melnikov
2019-12-24 16:45 ` Mikhail Efremov
2019-12-25  8:21   ` Ivan A. Melnikov

ALT Linux Distributions development

This inbox may be cloned and mirrored by anyone:

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

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


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