On Mon, Aug 29, 2016 at 09:36:44AM -0400, Wietse Venema wrote:

> Robert Dahlem:
> > Because it does no harm to anything existing I propose that attached
> > patch gets into the distribution.
> 
> It works with multi-instance names. What is the result for the stock
> master.cf file, which has entries as shown below?
> 
>     submission inet n       -       n       -       -       smtpd
>        -o syslog_name=postfix/submission
>        ...
>     smtps     inet  n       -       n       -       -       smtpd
>        -o syslog_name=postfix/smtps
>        ...

The patch correctly groups these with the plain "postfix/<daemon>" names
below:

> This logs process names as: postfix/submission/smtpd, postfix/smtps/smtpd.
> 
> Sorry, I don't have time to parse the script.

Provided that users avoid more exotic forms of secondary service
instance syslog_name which are not of the form
<postfix-instance>/<service_tag>, the collate script will figure
out that these secondary service instances are associated with the
same base Postfix instance.

Given that the OP has tested the suggested (more general second
form of the) regexp, I think it can go into the bundled auxiliary
script.

Perhaps I should attempt to auto-detect the time stamp format?
That's the main wart that prevents the script from being more
broadly applicable.

Apart from ISO date/time and the traditional syslog format,
are there any other forms in wide use?

-- 
        Viktor.

Reply via email to