27390 2012-05-31 23:57:40 +0400 [PATCH] lone systemd service support 2013-03-18 15:10:51 +0400 1 1 4 Development Sisyphus service unstable all Linux CLOSED FIXED http://git.altlinux.org/people/viy/packages/?p=service.git;a=commit;h=a4217a0e9b96b5ba35590b913a60e5d27c8ade52 P3 normal --- 27685 28704 28705 1 viy placeholder aen glebfm ldv legion piastryyy placeholder vt zerg qa-sisyphus oldest_to_newest 131577 0 viy 2012-05-31 23:57:40 +0400 Support for packages that have no sysVinit script but a systemd service. http://git.altlinux.org/people/viy/packages/?p=service.git;a=commit;h=a4217a0e9b96b5ba35590b913a60e5d27c8ade52 132041 1 ldv 2012-06-24 03:06:53 +0400 Proper support of a systemd-only service means that certain chkconfig calls are replaced by corresponding systemctl cals; just omitting chkconfig calls wouldn't qualify to be called "a support". 132070 2 viy 2012-06-26 17:00:16 +0400 (В ответ на комментарий №1) > Proper support of a systemd-only service means that certain chkconfig calls are replaced by corresponding systemctl cals; just omitting chkconfig calls > wouldn't qualify to be called "a support". Let me recall that the corresponding systemctl calls are performed during the service(8) call (namely, /sbin/service "$1" condrestart), which is _NOT_ omitted in the proposed patch. I recall, systemd services can be in enabled or disabled state, but system V scripts also have "not installed/uninstalled" state (when a system V script exists, but there are no symlinks in /etc/rc.d) so they need an additional chkconfig call. systemd scripts does not have this state and just don't need an additional chkconfig call, so we are right to just skip it. "у нас все точно" (С). 132072 3 viy 2012-06-26 17:06:02 +0400 the whole idea about the service(8) command is that this command is a portable wrapper that hides the actual implementation (sysV, systemd, upstart,...) is used. So anayway using the actual systemd calls instead of service would be a bad style. For example, if user did use sysV init, nothing good happen if we call systemctl. 132073 4 ldv 2012-06-26 17:14:36 +0400 How do you propose to implement "systemctl daemon-reload" and "systemctl disable specified.service" commands with "service"? Up to now this is implemented in chkconfig. 132076 5 viy 2012-06-26 17:57:36 +0400 (В ответ на комментарий №4) > How do you propose to implement "systemctl daemon-reload" and "systemctl > disable specified.service" commands with "service"? Up to now this is > implemented in chkconfig. Indeed. And I was even patching that code... I am deeply sorry. Proposed patch is invalid, chkconfig should be tuned instead. 138277 6 piastryyy 2013-03-01 14:32:58 +0400 I created a patch that seems to fix the problem for me: http://git.altlinux.org/people/piastry/packages/?p=service.git;a=commitdiff;h=950540de8cf8e5987c5694a529efded393ed8bfa Thoughts? 138306 7 viy 2013-03-01 23:08:53 +0400 (В ответ на комментарий №6) > I created a patch that seems to fix the problem for me: > http://git.altlinux.org/people/piastry/packages/?p=service.git;a=commitdiff;h=950540de8cf8e5987c5694a529efded393ed8bfa Thanks! This is an important fix. 138430 8 repository-robot 2013-03-05 20:41:44 +0400 service-0.5.24-alt1 -> sisyphus: * Thu Feb 28 2013 Dmitry V. Levin <ldv@altlinux> 0.5.24-alt1 - Implemented systemd-only service support in {post,preun}_service (closes: #27390) and native support of systemd services. Due to systemctl limitations, when systemd is active, post_service will no longer be able to enable sysv-only services. - Added /sbin/sd_booted utility. 138599 9 zerg 2013-03-12 18:13:01 +0400 chkconfig --add some.service не работает. При этом systemctl enable some.service отрабатывает нормально Т.е. %post_service не работает. 138604 10 ldv 2013-03-12 18:34:19 +0400 (In reply to comment #9) > chkconfig --add some.service > не работает. При этом > systemctl enable some.service > отрабатывает нормально > > Т.е. %post_service не работает. Если systemd активен, то %post_service использует systemctl вместо chkconfig, и если при этом сервис sysvinit-only, то он не будет включен. Об этом написано в %changelog'е пакета. 138605 11 ldv 2013-03-12 18:37:03 +0400 См. тж. systemd.preset(5) и /lib/systemd/system-preset/99-default.preset 138606 12 zerg 2013-03-12 18:52:20 +0400 Тогда вручную должно работать, как systemctl enable тотжесамый.service 138607 13 zerg 2013-03-12 18:54:07 +0400 (В ответ на комментарий №11) > См. тж. systemd.preset(5) и /lib/systemd/system-preset/99-default.preset Т.е. вне зависимости от наличия %post_service в пакете, сервисы, указанные в preset-е включаются специальными триггерами? 138609 14 ldv 2013-03-12 18:55:07 +0400 (In reply to comment #12) > Тогда вручную должно работать, как > systemctl enable тотжесамый.service systemctl enable тотжесамый.service работает. 138610 15 ldv 2013-03-12 18:57:40 +0400 (In reply to comment #13) > (В ответ на комментарий №11) > > См. тж. systemd.preset(5) и /lib/systemd/system-preset/99-default.preset > Т.е. вне зависимости от наличия %post_service в пакете, сервисы, указанные в > preset-е включаются специальными триггерами? Нет, автоматически preset-ы сейсас никто не включает, их включает %post_service при первой установке пакета. Хотя это действительно можно было бы реализовать файлтриггерами, %post_service все равно нужен, так что зачем? 138612 16 zerg 2013-03-12 19:17:36 +0400 Зачем пытаться кое-что кое-где возможно если повезет включить, когда надо *включить* ? Или firetrigger нужно сделать и игнорировать chkconfig в %post_service или chkconfig должен включать вне зависимости от preset-ов. 138613 17 zerg 2013-03-12 19:30:13 +0400 Оказывается, механизм, аналогичый system-preset присутствует в init-файлах, но в каждом индивидуально. В поле "chkconfig:" первым параметром написано, запускать ли его не каких-то runlevel-ах по умолчанию. Аналог этой информации для systemd у нас на данный момент отсутствует. 138701 18 zerg 2013-03-15 15:07:51 +0400 Может, убрать из %post_service поддержку systemd и сделать firetrigger для включения systemd-шных сервисов? А то как-то странновато в результате получается в случае с systemd. 138702 19 zerg 2013-03-15 15:10:24 +0400 И удобно будет: положил preset и не паришься больше ни о чем. Иначе приходится городить костыли. У меня, например, installer-feature-kdesktop-services, а у других свои.