Re: глобальные директивы error_log и pid

[Maxim Dounin]

Hello!

On Sat, Nov 13, 2010 at 05:51:42PM +0200, Gena Makhomed wrote:

> On 13.11.2010 16:51, Maxim Dounin wrote in nginx at nginx.org list:
> 
> >> Ideally I would like Nginx to always write to the error log file as
> >> specified in the config, and never to the default error log file,
> >> but it seems that the codebase does not allow this
> >> and assumes logging facilities to be available
> >> even before the config file is loaded.
> 
> > Writing early errors to error_log as specified in config is not
> > possible as we haven't (yet) parsed the config.
> 
> это можно так сделать, чтобы писать early errors
> только в тот error_log, который указан в конфиге.
> 
> > It would be good to have something like '-e' switch instead,
> > similar to '-p' for prefix (actually, I even have it in my TODO
> > since 0.7.53, but ENOTIME).
> 
> примерно так сейчас Apache и делает:
> 
> # httpd -h 2>&1 | grep error
>   -e level           : show startup errors of level (see LogLevel)
>   -E file            : log startup errors to file
> 
> но это не очень красивое решение, потому что инофрмация тут дублируется,
> да и с pid-файлом будут аналогичные проблемы, тогда еще один ключ надо?

С pid-файлом таких проблем быть не может - он создаётся уже после 
заверешения парсинга конфига.

> я придумал более красивое решение:
> 
> 1. nginx определяет имя конфигурационного файла из параметра -с
>    или используется built-in значение, если ключ -c не определен.
> 
> 2. вызывается helper-функция, которая парсит конфигурационный файл,
>    считывая только несколько глобальных директив: error_log и pid,
>    если эти две директивы встретились в конфиге, парсинг прекращается.
>    директива include также игнорируется, как и возможные ошибки разбора.

Вопрос на самом деле очень простой: если мы хотим куда-то 
логировать ошибки открытия/парсинга конфига - то должны откуда-то ещё 
получить место для логирования этих ошибок.

Сейчас это "откуда-то ещё" - --error-log-path из бинарника (и 
префикс из бинарника или из ключа -p, если путь в --error-log-path 
относительный).

Предлагаемое решение в качестве "откуда-то ещё" сделать 
дополнительный sloppy-parser конфига мне кажется крайне 
сомнительным.

[...]

> 5. добавляется всего один параметр командной строки -G
> 
>   -G directive  : get global directive from configuration file
> 
> это необходимо для того, чтобы можно было seamless использовать
> один бинарник nginx для запуска нескольких независимых instances:
> 
> pidfile=`nginx -G pid`
> 
> и дальше init-script знает, какой pid file отвечает этому экземпляру.
> причем, вся конфигурация nginx находится только в одном месте - конфиге.

Just a side note: это не имеет ну никакого отношения к проблеме 
error_log'а.

Сейчас nginx не умеет "провернуть фарш назад", т.е. достать 
исходную строку как она была в конфиге.  А даже если бы и умел: 
для pid'а в общем случае нужна не исходная строка, а то во что она 
превратилась после применения префикса.

Так что сделать подобную функциональность универсальной - будет 
нетривиально, фактически нужно каждую директиву учить себя 
доставать обратно.

С другой стороны - вот именно для pid'а может быть и имеет смысл 
сделать.

Maxim Dounin