Bug 24989

Created attachment 4781 [details]
add systemd support

(In reply to comment #0)
> Прошу добавить поддержку systemd в service.
> 
> PS: В приведенном патче мне не нравится двойная поверка. Если от неё
> избавлятся, то надо убирать check_service().

Двойная статическая проверка не страшна.  А вот зачем нужна эта строка:
echo "Redirecting to /bin/systemctl $OPTIONS $SERVICE.service" >/dev/stderr
?

(В ответ на комментарий №2)
> (In reply to comment #0)
> 
> Двойная статическая проверка не страшна.  А вот зачем нужна эта строка:
> echo "Redirecting to /bin/systemctl $OPTIONS $SERVICE.service" >/dev/stderr > ?

Согласен, это строка не нужна, даже вредна.
PS: вот к чему приводит copy-paste из других дистрибутивов.

Created attachment 4830 [details]
обновлённый

обновление поддержки systemd

(In reply to comment #4)
> Created an attachment (id=4830) [details]
> обновлённый

Тут написано, что в случае когда $SYSTEMCTL есть, $SERVICEDIR/$SERVICE тоже есть, а $SYSTEMDSERVICEDIR/$SERVICE.service нет, нужно запускать $SYSTEMCTL.
Другими словами, что SYSTEMCTL умеет запускать $SERVICEDIR/$SERVICE.
Это правильно?

(В ответ на комментарий №5)
> Тут написано, что в случае когда $SYSTEMCTL есть, $SERVICEDIR/$SERVICE тоже
> есть, а $SYSTEMDSERVICEDIR/$SERVICE.service нет, нужно запускать $SYSTEMCTL.
> Другими словами, что SYSTEMCTL умеет запускать $SERVICEDIR/$SERVICE.
> Это правильно?

да.
systemd умеет работать с init-скриптами SysV.
Если присутствуют оба файла foo.service и init.d/foo, то systemd будет использовать foo.service, а вот если его нет, тогда уже init.d/foo
Поэтому я делал так:
- если есть systemctl, то используем его во всех случаях, и для foo.service и для init.d/foo
- если нет systemctl, тогда используем init.d/foo

Правда есть ещё одна проблема: когда systemctl есть в системе, но мы по прежнему используем SysVinit. Если systemd не запущен, то systemctl сделать ничего не сможет. Надо бы еще проверять действительно systemd запущен.

PS: в исправленном патче ошибка - нужен пробел перед -f

(In reply to comment #6)
> systemd умеет работать с init-скриптами SysV.

тогда патч можно будет упростить

> Правда есть ещё одна проблема: когда systemctl есть в системе, но мы по
> прежнему используем SysVinit. Если systemd не запущен, то systemctl сделать
> ничего не сможет. Надо бы еще проверять действительно systemd запущен.

конечно, надо, но как?

(В ответ на комментарий №7)

> > Правда есть ещё одна проблема: когда systemctl есть в системе, но мы по
> > прежнему используем SysVinit. Если systemd не запущен, то systemctl сделать
> > ничего не сможет. Надо бы еще проверять действительно systemd запущен.
> 
> конечно, надо, но как?

Пока только придумал проверять наличие директории
/sys/fs/cgroup/systemd
Её создаёт сам systemd, и без cgroup работать не будет.

Created attachment 4832 [details]
proposed patch

(In reply to comment #8)
> Пока только придумал проверять наличие директории
> /sys/fs/cgroup/systemd
> Её создаёт сам systemd, и без cgroup работать не будет.

Можно попробовать потестировать приложенный патч.

(В ответ на комментарий №10)
> (In reply to comment #8)
> > Пока только придумал проверять наличие директории
> > /sys/fs/cgroup/systemd
> > Её создаёт сам systemd, и без cgroup работать не будет.
> 
> Можно попробовать потестировать приложенный патч.

Опробую.
Сейчас посмотрел initscrips  у fedora.
Они ещё модифицировали /etc/rc.d/functions для перенаправления команд на systemctl.
Я так понял для тех, кто не пользуется командой service, а использует /etc/init.d/foo напрямую.
Нужна ли нам такая заглушка?

И нашёл как они проверяют запущен ли systemd 
/bin/mountpoint -q /sys/fs/cgroup/systemd
Может так и правильней - это именно точка монтирования.

опробовал этот вариант патча - работает нормально.
Достаточно ли проверять только наличие директории? Или лучше
/bin/mountpoint -q /sys/fs/cgroup/systemd?

PS: ждём в сизифах нашего города :)

Created attachment 4836 [details]
proposed change

(In reply to comment #12)
> опробовал этот вариант патча - работает нормально.
> Достаточно ли проверять только наличие директории? Или лучше
> /bin/mountpoint -q /sys/fs/cgroup/systemd?

Новая редакция патча, с проверкой на mountpoint.
Вроде бы должна устроить всех, проверьте.

проверил - работает.

service-0.5.21-alt1 -> sisyphus:

* Thu Mar 17 2011 Dmitry V. Levin <ldv@altlinux> 0.5.21-alt1
- service: added systemd support (closes: #24989).

Решил написать сюда. Функционал, реализованный сейчас в service не позволяет использовать post_service и preun_service скрипты в спеке пакета с поддержкой systemd без дополнительных манипуляций. Данный патч исправляет данную проблему и позволяет сделать спек независимым от используемой системы - systemd или sysvinit.

http://git.altlinux.org/people/piastry/packages/?p=service.git;a=commitdiff;h=950540de8cf8e5987c5694a529efded393ed8bfa

(In reply to comment #17)
> Решил написать сюда. Функционал, реализованный сейчас в service не позволяет
> использовать post_service и preun_service скрипты в спеке пакета с поддержкой
> systemd без дополнительных манипуляций.

Это не сюда, это в #27390.