Skip to content

Update postfix_mailvolume#1633

Open
alexxroche wants to merge 3 commits into
munin-monitoring:masterfrom
alexxroche:master
Open

Update postfix_mailvolume#1633
alexxroche wants to merge 3 commits into
munin-monitoring:masterfrom
alexxroche:master

Conversation

@alexxroche
Copy link
Copy Markdown

Enable this plugin to additionally (optionally) read from the SystemD Journal

Tested on AlmaLinux 9.4

Enable this plugin to additionally (optionally) read from the SystemD Journal

Tested on AlmaLinux 9.4
Comment thread plugins/node.d/postfix_mailvolume Outdated
Comment thread plugins/node.d/postfix_mailvolume
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 10832442130

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.022%

Totals Coverage Status
Change from base Build 10714903398: 0.0%
Covered Lines: 1579
Relevant Lines: 2255

💛 - Coveralls

@sokai
Copy link
Copy Markdown

sokai commented Nov 4, 2025

Hey, may I kindly ask what status this PR has? – I came here, because I switched to systemd-journald and missing postfix_mailvolume stats (= values always are 0.00) now.
(I thought, based on the plugin doc, journald is still supported by defaulting to env.logfile syslog …)
ty + KR

@kenyon
Copy link
Copy Markdown
Member

kenyon commented Nov 4, 2025

@sokai this was already done in #1634, can you see if current master works for you?

@sokai
Copy link
Copy Markdown

sokai commented Feb 13, 2026

@kenyon Thanks for your quick reply and sry for my out of time reaction …

I'm on Debian (bookworm) ATM on a production system, so testing out the master (branch) is out of scope, sry. So I'll wait until it arrives at the Debian packages …
KR

Little update: munin-plugins-core at boorkworm is at 2.0.73 (from 2023). I now installed munin-* from bookworm-backportsbut v2.0.76 also is too old (to have the fix included) …

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants