24989 2011-01-28 20:10:48 +0300 add systemd support 2013-03-01 14:18:21 +0400 1 1 4 Development Sisyphus service unstable all Linux CLOSED FIXED P3 normal --- 1 shaba placeholder aen evg glebfm ldv legion piastryyy placeholder radik shaba vt qa-sisyphus oldest_to_newest 117775 0 shaba 2011-01-28 20:10:48 +0300 Прошу добавить поддержку systemd в service. PS: В приведенном патче мне не нравится двойная поверка. Если от неё избавлятся, то надо убирать check_service(). 117776 1 4781 shaba 2011-01-28 20:11:50 +0300 Created attachment 4781 add systemd support 117991 2 ldv 2011-02-05 00:38:05 +0300 (In reply to comment #0) > Прошу добавить поддержку systemd в service. > > PS: В приведенном патче мне не нравится двойная поверка. Если от неё > избавлятся, то надо убирать check_service(). Двойная статическая проверка не страшна. А вот зачем нужна эта строка: echo "Redirecting to /bin/systemctl $OPTIONS $SERVICE.service" >/dev/stderr ? 118037 3 shaba 2011-02-07 14:00:22 +0300 (В ответ на комментарий №2) > (In reply to comment #0) > > Двойная статическая проверка не страшна. А вот зачем нужна эта строка: > echo "Redirecting to /bin/systemctl $OPTIONS $SERVICE.service" >/dev/stderr > ? Согласен, это строка не нужна, даже вредна. PS: вот к чему приводит copy-paste из других дистрибутивов. 118950 4 4830 shaba 2011-03-02 02:19:01 +0300 Created attachment 4830 обновлённый обновление поддержки systemd 119046 5 ldv 2011-03-05 22:27:18 +0300 (In reply to comment #4) > Created an attachment (id=4830) [details] > обновлённый Тут написано, что в случае когда $SYSTEMCTL есть, $SERVICEDIR/$SERVICE тоже есть, а $SYSTEMDSERVICEDIR/$SERVICE.service нет, нужно запускать $SYSTEMCTL. Другими словами, что SYSTEMCTL умеет запускать $SERVICEDIR/$SERVICE. Это правильно? 119047 6 shaba 2011-03-06 01:11:10 +0300 (В ответ на комментарий №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 119048 7 ldv 2011-03-06 01:19:12 +0300 (In reply to comment #6) > systemd умеет работать с init-скриптами SysV. тогда патч можно будет упростить > Правда есть ещё одна проблема: когда systemctl есть в системе, но мы по > прежнему используем SysVinit. Если systemd не запущен, то systemctl сделать > ничего не сможет. Надо бы еще проверять действительно systemd запущен. конечно, надо, но как? 119053 8 shaba 2011-03-06 01:55:02 +0300 (В ответ на комментарий №7) > > Правда есть ещё одна проблема: когда systemctl есть в системе, но мы по > > прежнему используем SysVinit. Если systemd не запущен, то systemctl сделать > > ничего не сможет. Надо бы еще проверять действительно systemd запущен. > > конечно, надо, но как? Пока только придумал проверять наличие директории /sys/fs/cgroup/systemd Её создаёт сам systemd, и без cgroup работать не будет. 119054 9 4832 ldv 2011-03-06 02:12:02 +0300 Created attachment 4832 proposed patch 119055 10 ldv 2011-03-06 02:12:58 +0300 (In reply to comment #8) > Пока только придумал проверять наличие директории > /sys/fs/cgroup/systemd > Её создаёт сам systemd, и без cgroup работать не будет. Можно попробовать потестировать приложенный патч. 119058 11 shaba 2011-03-06 02:57:44 +0300 (В ответ на комментарий №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 Может так и правильней - это именно точка монтирования. 119132 12 shaba 2011-03-09 22:19:27 +0300 опробовал этот вариант патча - работает нормально. Достаточно ли проверять только наличие директории? Или лучше /bin/mountpoint -q /sys/fs/cgroup/systemd? PS: ждём в сизифах нашего города :) 119240 13 4836 ldv 2011-03-14 20:41:04 +0300 Created attachment 4836 proposed change 119241 14 ldv 2011-03-14 20:42:21 +0300 (In reply to comment #12) > опробовал этот вариант патча - работает нормально. > Достаточно ли проверять только наличие директории? Или лучше > /bin/mountpoint -q /sys/fs/cgroup/systemd? Новая редакция патча, с проверкой на mountpoint. Вроде бы должна устроить всех, проверьте. 119305 15 shaba 2011-03-16 19:44:05 +0300 проверил - работает. 119315 16 repository-robot 2011-03-17 01:45:10 +0300 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). 138269 17 piastryyy 2013-03-01 10:39:32 +0400 Решил написать сюда. Функционал, реализованный сейчас в service не позволяет использовать post_service и preun_service скрипты в спеке пакета с поддержкой systemd без дополнительных манипуляций. Данный патч исправляет данную проблему и позволяет сделать спек независимым от используемой системы - systemd или sysvinit. http://git.altlinux.org/people/piastry/packages/?p=service.git;a=commitdiff;h=950540de8cf8e5987c5694a529efded393ed8bfa 138276 18 ldv 2013-03-01 14:18:21 +0400 (In reply to comment #17) > Решил написать сюда. Функционал, реализованный сейчас в service не позволяет > использовать post_service и preun_service скрипты в спеке пакета с поддержкой > systemd без дополнительных манипуляций. Это не сюда, это в #27390. 4781 2011-01-28 20:11:50 +0300 2011-03-02 02:19:01 +0300 add systemd support add-systemd-support.patch text/plain 1304 shaba RnJvbSBkN2ViY2MyOGQ3MGJhMWI1YmI2MzA3M2U1YWU5YTMxYjAzMWNjZmI5IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBBbGV4ZXkgU2hhYmFsaW4gPHNoYWJhQGFsdGxpbnV4Lm9yZz4K RGF0ZTogRnJpLCAyOCBKYW4gMjAxMSAyMDowMTo0OCArMDMwMApTdWJqZWN0OiBbUEFUQ0hdIGFk ZCBzeXN0ZW1kIHN1cHBvcnQKCi0tLQogc2VydmljZS9zZXJ2aWNlIHwgICAxMSArKysrKysrKyst LQogMSBmaWxlcyBjaGFuZ2VkLCA5IGluc2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pCgpkaWZm IC0tZ2l0IGEvc2VydmljZS9zZXJ2aWNlIGIvc2VydmljZS9zZXJ2aWNlCmluZGV4IDJlNWI0MjEu LmQ2ZjMxZGMgMTAwNjQ0Ci0tLSBhL3NlcnZpY2Uvc2VydmljZQorKysgYi9zZXJ2aWNlL3NlcnZp Y2UKQEAgLTksNiArOSw4IEBAIGV4cG9ydCBQQVRICiAKIFZFUlNJT049InNlcnZpY2UgdmVyc2lv biAwLjkxLWFsdCIKIFNFUlZJQ0VESVI9Ii9ldGMvaW5pdC5kIgorU1lTVEVNQ1RMPSIvYmluL3N5 c3RlbWN0bCIKK1NZU1RFTURTRVJWSUNFRElSPSIvbGliL3N5c3RlbWQvc3lzdGVtIgogU0VSVklD RT0KIE9QVElPTlM9CiAKQEAgLTM1LDcgKzM3LDcgQEAgY2hlY2tfc2VydmljZSgpCiAJWyAtbiAi JHtTRVJWSUNFIyMqLyp9IiBdIHx8CiAJCWZhdGFsICIkU0VSVklDRTogSW52YWxpZCBzZXJ2aWNl IG5hbWUiCiAKLQlbIC14ICIkU0VSVklDRURJUi8kU0VSVklDRSIgXSB8fAorCVsgLXggIiRTRVJW SUNFRElSLyRTRVJWSUNFIiBdIHx8IFsgLXggIiRTWVNURU1DVEwiIC1hIC1mICIkU1lTVEVNRFNF UlZJQ0VESVIvJHtTRVJWSUNFfS5zZXJ2aWNlIiBdIHx8CiAJCWZhdGFsICIkU0VSVklDRTogVW5y ZWNvZ25pemVkIHNlcnZpY2UiCiB9CiAKQEAgLTY5LDcgKzcxLDEyIEBAIGNhc2UgIiQxIiBpbgog CQlzaGlmdAogCQljaGVja19zZXJ2aWNlCiAJCWNkIC8gfHwgZXhpdAotCQkiJFNFUlZJQ0VESVIv JFNFUlZJQ0UiICIkQCIKKwkJaWYgWyAteCAiJFNFUlZJQ0VESVIvJFNFUlZJQ0UiIF07IHRoZW4K KwkJCSIkU0VSVklDRURJUi8kU0VSVklDRSIgIiRAIgorCQllbGlmIFsgLXggIiRTWVNURU1DVEwi IC1hIC1mICIkU1lTVEVNRFNFUlZJQ0VESVIvJHtTRVJWSUNFfS5zZXJ2aWNlIiBdOyB0aGVuCisJ CQllY2hvICJSZWRpcmVjdGluZyB0byAvYmluL3N5c3RlbWN0bCAkT1BUSU9OUyAkU0VSVklDRS5z ZXJ2aWNlIiA+L2Rldi9zdGRlcnIKKwkJCWV4ZWMgJFNZU1RFTUNUTCAiJEAiICR7U0VSVklDRX0u c2VydmljZQorCQlmaQogCQlleGl0ICQ/CiAJCTs7CiBlc2FjCi0tIAoxLjcuMy41Cgo= 4830 2011-03-02 02:19:01 +0300 2011-03-06 02:12:02 +0300 обновлённый add-systemd-support.patch text/plain 980 shaba ZGlmZiAtLWdpdCBhL3NlcnZpY2Uvc2VydmljZSBiL3NlcnZpY2Uvc2VydmljZQppbmRleCAyZTVi NDIxLi5lZjFmMDE3IDEwMDY0NAotLS0gYS9zZXJ2aWNlL3NlcnZpY2UKKysrIGIvc2VydmljZS9z ZXJ2aWNlCkBAIC05LDYgKzksOCBAQCBleHBvcnQgUEFUSAogCiBWRVJTSU9OPSJzZXJ2aWNlIHZl cnNpb24gMC45MS1hbHQiCiBTRVJWSUNFRElSPSIvZXRjL2luaXQuZCIKK1NZU1RFTUNUTD0iL2Jp bi9zeXN0ZW1jdGwiCitTWVNURU1EU0VSVklDRURJUj0iL2xpYi9zeXN0ZW1kL3N5c3RlbSIKIFNF UlZJQ0U9CiBPUFRJT05TPQogCkBAIC0zNSw3ICszNyw3IEBAIGNoZWNrX3NlcnZpY2UoKQogCVsg LW4gIiR7U0VSVklDRSMjKi8qfSIgXSB8fAogCQlmYXRhbCAiJFNFUlZJQ0U6IEludmFsaWQgc2Vy dmljZSBuYW1lIgogCi0JWyAteCAiJFNFUlZJQ0VESVIvJFNFUlZJQ0UiIF0gfHwKKwlbIC14ICIk U0VSVklDRURJUi8kU0VSVklDRSIgXSB8fCBbIC14ICIkU1lTVEVNQ1RMIiAtYSAtZiAiJFNZU1RF TURTRVJWSUNFRElSLyR7U0VSVklDRX0uc2VydmljZSIgXSB8fAogCQlmYXRhbCAiJFNFUlZJQ0U6 IFVucmVjb2duaXplZCBzZXJ2aWNlIgogfQogCkBAIC02OSw3ICs3MSwxMyBAQCBjYXNlICIkMSIg aW4KIAkJc2hpZnQKIAkJY2hlY2tfc2VydmljZQogCQljZCAvIHx8IGV4aXQKLQkJIiRTRVJWSUNF RElSLyRTRVJWSUNFIiAiJEAiCisJCWlmIFsgLXggIiRTWVNURU1DVEwiIF07IHRoZW4KKwkJCWlm IFstZiAiJFNZU1RFTURTRVJWSUNFRElSLyR7U0VSVklDRX0uc2VydmljZSIgLW8gLXggIiRTRVJW SUNFRElSLyRTRVJWSUNFIiBdOyB0aGVuCisJCQkJZXhlYyAkU1lTVEVNQ1RMICIkQCIgJHtTRVJW SUNFfS5zZXJ2aWNlCisJCQlmaQorCQllbGlmIFsgLXggIiRTRVJWSUNFRElSLyRTRVJWSUNFIiBd OyB0aGVuCisJCQkiJFNFUlZJQ0VESVIvJFNFUlZJQ0UiICIkQCIKKwkJZmkKIAkJZXhpdCAkPwog CQk7OwogZXNhYwo= 4832 2011-03-06 02:12:02 +0300 2011-03-14 20:41:04 +0300 proposed patch service-systemd.diff text/plain 970 ldv ZGlmZiAtLWdpdCBhL3NlcnZpY2Uvc2VydmljZSBiL3NlcnZpY2Uvc2VydmljZQppbmRleCAyZTVi NDIxLi41NzdmYWU5IDEwMDY0NAotLS0gYS9zZXJ2aWNlL3NlcnZpY2UKKysrIGIvc2VydmljZS9z ZXJ2aWNlCkBAIC04LDcgKzgsMTAgQEAgUEFUSD0vc2JpbjovdXNyL3NiaW46L2JpbjovdXNyL2Jp bgogZXhwb3J0IFBBVEgKIAogVkVSU0lPTj0ic2VydmljZSB2ZXJzaW9uIDAuOTEtYWx0IgotU0VS VklDRURJUj0iL2V0Yy9pbml0LmQiCitTRVJWSUNFRElSPS9ldGMvaW5pdC5kCitTWVNURU1DVEw9 L2Jpbi9zeXN0ZW1jdGwKK1NZU1RFTURfU0VSVklDRV9ESVI9L2xpYi9zeXN0ZW1kL3N5c3RlbQor U1lTVEVNRF9DR1JPVVBfRElSPS9zeXMvZnMvY2dyb3VwL3N5c3RlbWQKIFNFUlZJQ0U9CiBPUFRJ T05TPQogCkBAIC0zNiw2ICszOSw5IEBAIGNoZWNrX3NlcnZpY2UoKQogCQlmYXRhbCAiJFNFUlZJ Q0U6IEludmFsaWQgc2VydmljZSBuYW1lIgogCiAJWyAteCAiJFNFUlZJQ0VESVIvJFNFUlZJQ0Ui IF0gfHwKKwlbIC14ICIkU1lTVEVNQ1RMIiAtYSBcCisJICAtZCAiJFNZU1RFTURfQ0dST1VQX0RJ UiIgLWEgXAorCSAgLWYgIiRTWVNURU1EX1NFUlZJQ0VfRElSLyRTRVJWSUNFLnNlcnZpY2UiIF0g fHwKIAkJZmF0YWwgIiRTRVJWSUNFOiBVbnJlY29nbml6ZWQgc2VydmljZSIKIH0KIApAQCAtNjks NyArNzUsMTEgQEAgY2FzZSAiJDEiIGluCiAJCXNoaWZ0CiAJCWNoZWNrX3NlcnZpY2UKIAkJY2Qg LyB8fCBleGl0Ci0JCSIkU0VSVklDRURJUi8kU0VSVklDRSIgIiRAIgotCQlleGl0ICQ/CisJCWlm IFsgLXggIiRTWVNURU1DVEwiIC1hIC1kICIkU1lTVEVNRF9DR1JPVVBfRElSIiBdOyB0aGVuCisJ CQlleGVjICIkU1lTVEVNQ1RMIiAiJEAiICIkU0VSVklDRS5zZXJ2aWNlIgorCQllbHNlCisJCQll eGVjICIkU0VSVklDRURJUi8kU0VSVklDRSIgIiRAIgorCQlmaQorCQlleGl0CiAJCTs7CiBlc2Fj Cg== 4836 2011-03-14 20:41:04 +0300 2011-03-14 20:41:04 +0300 proposed change service-systemd.diff text/plain 1346 ldv ZGlmZiAtLWdpdCBhL3NlcnZpY2Uvc2VydmljZSBiL3NlcnZpY2Uvc2VydmljZQppbmRleCAyZTVi NDIxLi5kNDIyNTcxIDEwMDY0NAotLS0gYS9zZXJ2aWNlL3NlcnZpY2UKKysrIGIvc2VydmljZS9z ZXJ2aWNlCkBAIC04LDcgKzgsMTAgQEAgUEFUSD0vc2JpbjovdXNyL3NiaW46L2JpbjovdXNyL2Jp bgogZXhwb3J0IFBBVEgKIAogVkVSU0lPTj0ic2VydmljZSB2ZXJzaW9uIDAuOTEtYWx0IgotU0VS VklDRURJUj0iL2V0Yy9pbml0LmQiCitTRVJWSUNFRElSPS9ldGMvaW5pdC5kCitTWVNURU1DVEw9 L2Jpbi9zeXN0ZW1jdGwKK1NZU1RFTURfU0VSVklDRV9ESVI9L2xpYi9zeXN0ZW1kL3N5c3RlbQor U1lTVEVNRF9DR1JPVVBfRElSPS9zeXMvZnMvY2dyb3VwL3N5c3RlbWQKIFNFUlZJQ0U9CiBPUFRJ T05TPQogCkBAIC0zMCwxMyArMzMsMzAgQEAgdXNhZ2UoKQogCVsgLW4gIiQxIiBdICYmIGV4aXQg IiQxIiB8fCBleGl0CiB9CiAKK3N5c3RlbWRfc3RhdHVzPQorc3lzdGVtZF9pc19hY3RpdmUoKQor eworCWlmIFsgLXogIiRzeXN0ZW1kX3N0YXR1cyIgXTsgdGhlbgorCQlbIC14ICIkU1lTVEVNQ1RM IiAtYSBcCisJCSAgLWQgIiRTWVNURU1EX0NHUk9VUF9ESVIiIF0gJiYKKwkJbW91bnRwb2ludCAt cSAiJFNZU1RFTURfQ0dST1VQX0RJUiIKKwkJc3lzdGVtZF9zdGF0dXM9JD8KKwlmaQorCXJldHVy biAkc3lzdGVtZF9zdGF0dXMKK30KKwogY2hlY2tfc2VydmljZSgpCiB7CiAJWyAtbiAiJHtTRVJW SUNFIyMqLyp9IiBdIHx8CiAJCWZhdGFsICIkU0VSVklDRTogSW52YWxpZCBzZXJ2aWNlIG5hbWUi CiAKLQlbIC14ICIkU0VSVklDRURJUi8kU0VSVklDRSIgXSB8fAotCQlmYXRhbCAiJFNFUlZJQ0U6 IFVucmVjb2duaXplZCBzZXJ2aWNlIgorCVsgLXggIiRTRVJWSUNFRElSLyRTRVJWSUNFIiBdICYm IHJldHVybiAwCisKKwlbIC1mICIkU1lTVEVNRF9TRVJWSUNFX0RJUi8kU0VSVklDRS5zZXJ2aWNl IiBdICYmCisJCXN5c3RlbWRfaXNfYWN0aXZlICYmCisJCXJldHVybiAwCisKKwlmYXRhbCAiJFNF UlZJQ0U6IFVucmVjb2duaXplZCBzZXJ2aWNlIgogfQogCiBpZiBbICQjIC1lcSAwIF07IHRoZW4K QEAgLTY5LDcgKzg5LDExIEBAIGNhc2UgIiQxIiBpbgogCQlzaGlmdAogCQljaGVja19zZXJ2aWNl CiAJCWNkIC8gfHwgZXhpdAotCQkiJFNFUlZJQ0VESVIvJFNFUlZJQ0UiICIkQCIKLQkJZXhpdCAk PworCQlpZiBzeXN0ZW1kX2lzX2FjdGl2ZTsgdGhlbgorCQkJZXhlYyAiJFNZU1RFTUNUTCIgIiRA IiAiJFNFUlZJQ0Uuc2VydmljZSIKKwkJZWxzZQorCQkJZXhlYyAiJFNFUlZJQ0VESVIvJFNFUlZJ Q0UiICIkQCIKKwkJZmkKKwkJZXhpdAogCQk7OwogZXNhYwo=