Bug 31477 - Содержит включенный по умолчанию вызов из cron
Summary: Содержит включенный по умолчанию вызов из cron
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: vnstat (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Nikolay A. Fetisov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-12 14:56 MSK by Vitaly Lipatov
Modified: 2018-08-25 17:18 MSK (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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