Bug 28102 - zabbix-agent: Необходимо обеспечить совместимость службы с systemd
Summary: zabbix-agent: Необходимо обеспечить совместимость службы с systemd
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: zabbix-agent (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Alexei Takaseev
QA Contact: qa-sisyphus
URL: http://www.altlinux.org/Systemd_Statu...
Keywords:
Depends on:
Blocks: 28008
  Show dependency tree
 
Reported: 2012-11-21 16:23 MSK by Mike
Modified: 2012-12-29 05:32 MSK (History)
2 users (show)

See Also:


Attachments
tmpfiles.d conf (39 bytes, text/plain)
2012-11-28 15:08 MSK, Mike
no flags Details
патч на юнит (319 bytes, patch)
2012-11-28 15:09 MSK, Mike
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mike 2012-11-21 16:23:16 MSK
Подготовить корректный unit-файл.

Подробности по ссылке: http://www.altlinux.org/Systemd_Status_P7
Comment 1 Alexei Takaseev 2012-11-23 09:23:01 MSK
Поддержка systemd добавлена
Comment 2 Mike 2012-11-23 13:31:59 MSK
Вы проверяли? 
В всех unit-файлах стоит Type=oneshot, что не правильно, т.к. systemd, будет ожидать прекращения работы сервиса, как факт его правильной работы.

В вашем же случае, вы останавливаете загрузку системы.
Поправьте, пожалуйста.
Comment 3 Alexei Takaseev 2012-11-23 16:11:38 MSK
unit-файлы взяты из Fedore с минимальными правками.

Проверил вручную, что zabbix_agentd, что zabbix_pgsql при запуске демонизируются самостоятельно, стопора загрузки быть не должно.
Comment 4 Mike 2012-11-23 18:04:21 MSK
(В ответ на комментарий №3)
> unit-файлы взяты из Fedore с минимальными правками.
> 
> Проверил вручную, что zabbix_agentd, что zabbix_pgsql при запуске
> демонизируются самостоятельно, стопора загрузки быть не должно.

Имена unit-файлов, должны иметь такое же имя, как у rc-скриптов запуска сервиса, иначе systemd сочтёт их за два разных сервиса, как сейчас:

# systemctl status zabbix_agentd.service
zabbix_agentd.service - LSB: the zabbix network monitor agent
          Loaded: loaded (/etc/rc.d/init.d/zabbix_agentd)
...

И 
# systemctl status zabbix-agent.service
zabbix-agent.service - ZABBIX Monitor Agent
          Loaded: loaded (/lib/systemd/system/zabbix-agent.service; disabled)

для systemd - это два разных сервиса.
Comment 5 Alexei Takaseev 2012-11-25 19:17:07 MSK
2.0.3-alt3
Comment 6 Mike 2012-11-28 15:08:02 MSK
Created attachment 5657 [details]
tmpfiles.d conf
Comment 7 Mike 2012-11-28 15:09:18 MSK
Created attachment 5658 [details]
патч на юнит
Comment 8 Mike 2012-11-28 15:10:17 MSK
У меня работает с данными файлами и настройками юнита. Сервис корректно запускается и останавливается.
Comment 9 Alexei Takaseev 2012-11-28 16:38:30 MSK
(В ответ на комментарий №6)
> Created an attachment (id=5657) [details]
> tmpfiles.d conf

Погодите, в пакете zabbix-common (на который по зависимостям ставится вместе со всеми другими zabbix-подпакетами) уже есть /etc/tmpfiles.d/zabbix.conf с содержимым:

D /var/run/zabbix 0755 zabbix zabbix -

разве его не достаточно?

исправления в unit файлы внесены
Comment 10 Mike 2012-11-28 18:13:34 MSK
(В ответ на комментарий №9)
> (В ответ на комментарий №6)
> > Created an attachment (id=5657) [details] [details]
> > tmpfiles.d conf
> 
> Погодите, в пакете zabbix-common (на который по зависимостям ставится вместе со
> всеми другими zabbix-подпакетами) уже есть /etc/tmpfiles.d/zabbix.conf с
> содержимым:
> 
> D /var/run/zabbix 0755 zabbix zabbix -
> 
> разве его не достаточно?
> 

Достаточно, просто zabbix-common не обновился, при обновлении zabbix-agent и я не заметил данного изменения.
Только, переложите файл из /etc в /lib, т.к. это полиси.
Comment 11 Mike 2012-11-28 18:17:47 MSK
думаю надо бы добавить в agent. зависимость на определённую версию common, в котором есть конфиг для tmpfiles.d
Comment 12 solo 2012-12-17 23:38:51 MSK
(В ответ на комментарий №11)
> думаю надо бы добавить в agent. зависимость на определённую версию common, в
> котором есть конфиг для tmpfiles.d

  На мой взгляд -- не строгую, вида:

Requires: zabbix-common >= V1.V2.V3-altR

где V1.V2.V3-altR версия и релиз пакета , начиная с которого присутствует конфиг для tmpfiles.d.
Comment 13 Alexei Takaseev 2012-12-29 05:32:34 MSK
Посмотрите, пожалуйста 2.0.4-alt2