Статистика postfix на bash для zabbix

TEMP_RECEIVED=$(/bin/grep «bytes received» /tmp/postfix/temp5 | awk '{print $1}')

Я бы промолчал, если б у вас не появилось что-то новенькое. Хрен с ним, что эта классика давно обсосана насчёт идиотизма grep|awk, когда правильно просто:

awk '/bytes received/  {print $1}' /tmp/postfix/temp5

Если по делу, то ваши TEMP_RECEIVED могут быть пустыми и потому нельзя сравнивать как -eq, хотя можно, но предварительно проверить на пусто и установить в 0:

TEMP_RECEIVED=${TEMP_RECEIVED:-0}

полностью справедливые замечания!

Работе с bash, sed, awk, и прочим-прочим системным утилитам я учусь, и применяю те средства, которые мне оказались знакомы. В идеале я и хочу написать то, что будет работать без всяких лишних вызовов, это будет после достижения базовой функциональности.

но почему у вас /bin/grep, но просто awk ?

Где-то ранее столкнулся с тем, что grep не был обнаружен, дописал /bin/grep и все заработало. awk же был добавлен позже, видимо, в тот момент времени, ему уже не нужен был полный путь.

Надо пересилить себя и научиться обходиться без них.

пытаюсь. отягощается тем, что пересиливать нужно, если знаешь как. а то получается приложение сил - но выходит как у меня - с кучей временных файлов.

пытаюсь. отягощается тем, что пересиливать нужно, если знаешь как. а то получается приложение сил - но выходит как у меня - с кучей временных файлов.

Мониторинг же у вас будет в цикле, потому не надо запоминать старое значение во временном файле. Впрочем, если хотите, по получению сигналов SIGTERM/SIGINT/etc можно и записывать. Это будет хороший пилотаж.

ResPF у вас лишняя переменная, правильнее как-то так:

#!/bin/bash

##Old data of Received
BYTES_R=
read BYTES_R 2> /dev/null < /tmp/postfix/postfix.bytes.received
BYTES_R=${BYTES_R:-0}

# включаем запоминание во временный файл при сигнале
trap 'echo $BYTES_R > /tmp/postfix/postfix.bytes.received; exit 1' SIGHUP SIGINT SIGQUIT SIGTERM

while true; do
 ##Get new values 
 BYTES_RN=$(/usr/sbin/pflogsumm -h 0 -u 0 -d today --bounce_detail=0 --deferral_detail=0 --reject_detail=0 --smtpd_warning_detail=0 --no_no_msg_size /var/log/mail.log | awk 'BEGIN {sum=0} /bytes received/  {sum+=$1} END {print sum}')
 ##получаем abs(разницы)
 BYTES_R=$((BYTES_RN - BYTES_R < 0 ? BYTES_R - BYTES_RN : BYTES_RN - BYTES_R)) 
 # что-то делаем ещё
 ......
done

ResPF у вас лишняя переменная

Возможно, что нет, так как наблюдаемых параметров несколько, всего 7.

Разве не проще получить весь вывод, а потом его разбирать на кусочки, чем вызывать полный вывод 7 раз с последующим разбором?

Разве не проще получить весь вывод, а потом его разбирать на кусочки, чем вызывать полный вывод 7 раз с последующим разбором?

Да, но это нечестный приёмчик. Вы привели скрипт без намёка, что оно вам вообще нужно. Но и тут переменная не нужна. Если вывод как param=value, то вот как-то так:

        while read l; do
                tag=${l%%=*}
                val=${l#*=}
                case "${tag^^}" in
                 param1) param1=$val;;
                 param2) param2=$val;;
                esac
        done << EOF
$(/usr/sbin/pflogsumm ..... /var/log/mail.log)
EOF

Спасибо, буду разбирать ваш пример. Надеюсь я пойму синтаксис и смогу его применить осознанно