Skip to content

[smtp-relay] Nobble log ingrestion to allow metrics startup - #148

Open
djjudas21 wants to merge 3 commits into
mainfrom
146_smtp_metrics
Open

[smtp-relay] Nobble log ingrestion to allow metrics startup#148
djjudas21 wants to merge 3 commits into
mainfrom
146_smtp_metrics

Conversation

@djjudas21

Copy link
Copy Markdown
Owner

What this PR does / why we need it

At the moment it is not possible for the metrics exporter to read /var/log/maillog from the Postfix container without some additional engineering work on log rotation and possibly even shared volumes - discussion in #146 (comment)

As a workaround, this PR causes the metrics exporter to open a fake Postfix logfile from /dev/null, so it can start up without crashing. This means the metrics exporter will not have access to the Postfix event log (it still has access to the Postfix socket) so it will report a reduced set of metrics. At the moment I'm not sure what it will report.

It should still be possible to measure the queue length and ensure the exporter is working properly, but it might fail to report the number of messages transmitted, etc.

Which issue this PR fixes

Checklist

  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

@djjudas21

Copy link
Copy Markdown
Owner Author

@leleobhz what do you think about this PR? 🙂

@leleobhz

Copy link
Copy Markdown
Contributor

@djjudas21

I think this is a good start, but some logic for log rotating postfix logs or test if FIFO files are compatible with both postfix and kumina collector may be a next step.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[smtp-relay] monitoring enabled causes CrashLoopBackOff

2 participants