ср, 15 авг. 2018 г. в 23:12, Валентин Бартенев <vbart@nginx.com>:
> On Wednesday 15 August 2018 22:46:08 Илья Шипицин wrote:
> [..]
> > если в лог упала ошибка, можно же как-то сделать, чтобы эту же ошибку
> > транслировать в код выхода reload-а ?
> >
>
> reload это команда "kill -s HUP" и у нее код выхода - это успешная
> отправка сигнала.
>
> Можете попытаться доработать rc-скрипт и добавить там парсинг лога,
> но непонятно сколько времени ждать. Обработка конфигурации может
> занять доли секунды, а может занимать минуты.
>
> Цель же такой доработки сомнительна. Посмотреть лог после
> изменение конфигурации, даже если оно прошло успешно - это
> вообще важно. Там могут возникать ошибки в рамках обработки
> запросов новой конфигурацией.
>
> Это плохо когда администраторы не смотрят в логи.
>
у нас эти штуки робот делает.
проверку успешности в reload в робот заложили, искусственный интеллект
"посмотри логи" нет.
я понимаю про отправку сигнала. я про другое. у nginx есть четко
обозначенная ошибка (в приведенном примере),
он пишет ее в лог. сигналы это ведь не единственный IPC механизм. можно
ведь задействовать другой механизм,
который бы позволил тому, что делает reload, понять, что все пошло не так ?
грубо, вы предлагаете администратору посмотреть лог. ок, пусть его смотрит
сам nginx, когда отправляет сигнал ?
отправил, понял, что кроме как по логу нельзя понять, ошибка или нет,
пошел, посмотрел лог, и потом уже вернул статус.
зачем на плечи человека перекладывать то, что может сделать машина
>
> В NGINX Unit такой трудности нет, т.к. он управляется через HTTP
> интерфейс и там любая операция с конфигурацией - это запрос, а
> её результат это ответ. Посылать же сигнал, а затем парсить
> лог - выглядит как костыль.
>
> --
> Валентин Бартенев
> _______________________________________________
> nginx-ru mailing list
> nginx-ru@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-ru
_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru
