ср, 15 авг. 2018 г. в 23:12, Валентин Бартенев <vb...@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
