Bug 31477

Summary:	Содержит включенный по умолчанию вызов из cron
Product:	Sisyphus	Reporter:	Vitaly Lipatov <lav>
Component:	vnstat	Assignee:	Nikolay A. Fetisov <naf>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	mike, naf
Version:	unstable
Hardware:	all
OS:	Linux

Description Vitaly Lipatov 2015-11-12 14:56:07 MSK

В пакете поставляется /etc/cron.d/vnstat, который каждые 5 минут начинает вызывать vnstat для сборка статистики.
Я бы предложил закомментировать в нём по умолчанию строку.
Да и все проверки бы убрал, оставив только вызов /usr/bin/vnstat -u
вместо
([ -x /usr/bin/vnstat ] && [ `/bin/ls /var/lib/vnstat/ | /bin/wc -l` -ge 1 ] && /usr/bin/vnstat -u)

Comment 1 Nikolay A. Fetisov 2015-11-13 19:09:08 MSK

Выключенного вызова там как раз нет - он именно включён.

А смысл его отключать?

Сейчас в процессе установки vnstat создаёт базы для имеющихся на момент установки
интерфейсов и далее начинает считать трафик. Вызовы его для обновления данных 
лёгкие и нагрузку на машину не создают. Если надо - можно отключить,
оставив vnstat только под просмотр текущего трафика ручными вызовами.
При обновлении пакета это сохранится.

По проверкам - в текущем виде нужны, в отсутствии баз запуск vnstat на 
обновлении даст ошибку.


Т.е., единственный вариант, когда смысл отключить cron-задачу есть - это 
если ставить ещё и vnstat-server. Хотя и там дублирующие обновления по
cron'у работе демона не мешают.

Comment 2 Vitaly Lipatov 2015-11-15 16:54:17 MSK

> Сейчас в процессе установки vnstat создаёт базы для имеющихся на момент
установки интерфейсов и далее начинает считать трафик
По моим наблюдением он берёт названия интерфейсов из конфига, куда прибито eth0.

> можно отключить, оставив vnstat только под просмотр текущего трафика ручными вызовами.
Так я и предлагаю по умолчанию отключить. Я ставлю инструмент, а он начинает по крону запускаться.
У меня логика простая: запуск по cron не многим отличается от запуска сервиса. А у нас в полиси есть, что сервис не должен стартовать по умолчанию.

> По проверкам - в текущем виде нужны, в отсутствии баз запуск vnstat на 
обновлении даст ошибку.
Особенно проверка наличия бинарника :)

Ну и пусть даёт ошибку, откуда возьмётся его запуск без баз?

Comment 3 Vitaly Lipatov 2016-04-06 11:31:08 MSK

Как сделано в RHEL. Всё в одном пакете vnstat. Правило для крон закомментировано:

# cat /etc/cron.d/vnstat
MAILTO=root
# to enable interface monitoring via vnstat remove comment on next line
# */5 * * * *  vnstat /usr/sbin/vnstat.cron

Сейчас у нас нелепая ситуация, когда я ставлю vnstat для того, чтобы _иногда_ посмотреть на трафик, а он начинает мусорить в логи cron.
И приходится ставить и выключать. Может быть, компромиссный вариант — это выделить в отдельный пакет правило для cron?

> Сейчас в процессе установки vnstat создаёт базы для имеющихся на момент
> установки интерфейсов
А нельзя ли заодно Interface выставлять в /etc/vnstat.conf ?

Comment 4 Repository Robot 2018-08-25 17:18:33 MSK

vnstat-1.18-alt1 -> sisyphus:

Sat Aug 25 2018 Nikolay A. Fetisov <naf@altlinux> 1.18-alt1
- New version
- Move cron script to -server package and disable it by default (Closes: 31477)
- Add vnstat-update script for databases and configuration updates