Bug 30534 - Не работает под systemd
Summary: Не работает под systemd
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: tor (show other bugs)
Version: unstable
Hardware: all Linux
: P3 major
Assignee: Anton Farygin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-03 19:05 MSK by Evgenii Terechkov
Modified: 2014-12-22 16:29 MSK (History)
3 users (show)

See Also:

Attachments
Поправка пути до pid-файла (337 bytes, patch)
2014-12-03 19:08 MSK, Evgenii Terechkov 		no flags Details | Diff
конфиг для /etc/tmpfiles.d/ (30 bytes, text/plain)
2014-12-03 19:09 MSK, Evgenii Terechkov 		no flags Details
Юнит-файл (296 bytes, application/octet-stream)
2014-12-03 19:16 MSK, Evgenii Terechkov 		no flags Details
Ещё один вариант юнит-файла (498 bytes, text/plain)
2014-12-19 21:09 MSK, ruslandh 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Evgenii Terechkov 2014-12-03 19:05:28 MSK 
Из-за несовпадения пути к pid-файлу в содержимом /e/r/init.d/tor и LSB-заголовке SYSV-only сервис tor зависает при старте. Если поправить ошибку, pidfile не создаётся т.к. нет каталога /var/run/tor, приходится создавать /etc/tmpfiles.d/tor.conf.

Прилагаю минимальные изменения для работы в режиме совместимости с sysv. Но лучше конечно же написать нативный юнит-файл.
Comment 1 Evgenii Terechkov 2014-12-03 19:08:49 MSK 
Created attachment 6191 [details]
Поправка пути до pid-файла
Comment 2 Evgenii Terechkov 2014-12-03 19:09:24 MSK 
Created attachment 6192 [details]
конфиг для /etc/tmpfiles.d/
Comment 3 Evgenii Terechkov 2014-12-03 19:16:08 MSK 
Created attachment 6193 [details]
Юнит-файл

Вот такой юнит-файл у меня вроде работает.
Comment 4 Владимир Диденко 2014-12-04 09:19:25 MSK 
Может еще и обновить до 2.5.10 сразу ?
Comment 5 Владимир Диденко 2014-12-05 19:06:07 MSK 
В общем, я собрал 0.2.5.10 с поддержкой systemd - task #136044. Прошу заинтересованных потестировать, ну и поглядеть на изменения

http://git.altlinux.org/people/cow/packages/tor.git

Пока не решил, что делать с torify. Уже некоторое время это просто обертка над torsocks (вчера собрал в сизиф). В федоре поставили зависимость в tor на torsocks, чтобы torify сразу работал. С другой стороны torify не всем нужен. Так что вариантов несколько 

- оставить все как есть. Кто захочет доставит пакет ручками. torify сообщает пользователю, если он не нашел torsocks.
- поставить зависимость на torsocks как в федоре
- переместить torify в отдельный подпакет с зависимостью на torsocks - в федоре так было раньше, но потом вернули взад.
Comment 6 Владимир Диденко 2014-12-15 13:54:57 MSK 
Так как недовольных не нашлось, отправил #136044 в сизиф - прошу дать approve.
Comment 7 ruslandh 2014-12-19 21:09:01 MSK 
Created attachment 6205 [details]
Ещё один вариант юнит-файла

Ещё один вариант юнит-файла
Comment 8 Владимир Диденко 2014-12-19 21:48:08 MSK 
(В ответ на комментарий №7)
> Created an attachment (id=6205) [details]
> Ещё один вариант юнит-файла
> 
> Ещё один вариант юнит-файла

Чем не устраивает 

http://git.altlinux.org/people/cow/packages/?p=tor.git;a=blob;f=tor.systemd.service;h=f09096d77a3f3a6a8de6ffe808436fb516f5176a;hb=refs/heads/sisyphus

из таска #136044 ?

К приатаченному сервис файлу у меня есть замечания

1. Лишний пустой ExecStart
2. Ненужный "--defaults-torrc /usr/share/tor/defaults-torrc". В новой версии 
   2.5.10 этого файла уже нет. 
3. --runasdaemon 1 - неправильно для типа simple. См. 

https://trac.torproject.org/projects/tor/ticket/8368#comment:13

4. Из пункта 3 следует, что не нужен "--pidfile /var/run/tor/tor.pid"
Comment 9 ruslandh 2014-12-20 08:59:25 MSK 
Ок, вы правы, разобрался
Comment 10 Владимир Диденко 2014-12-22 16:29:38 MSK 
Задание прошло в сизиф. Закрываю.