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.