Skip to content

fix: fix #972 by increasing file descriptors for filtermail, and reduce destination concurrency#971

Merged
hpk42 merged 1 commit into
mainfrom
fix158
May 14, 2026
Merged

fix: fix #972 by increasing file descriptors for filtermail, and reduce destination concurrency#971
hpk42 merged 1 commit into
mainfrom
fix158

Conversation

@hpk42
Copy link
Copy Markdown
Contributor

@hpk42 hpk42 commented May 14, 2026

No description provided.

@link2xt
Copy link
Copy Markdown
Contributor

link2xt commented May 14, 2026

This references wrong issue, the issue is in another repo: #972

@ccclxxiii ccclxxiii self-requested a review May 14, 2026 19:16
default_transport = lmtp-filtermail:inet:[127.0.0.1]:{{ config.filtermail_lmtp_port_transport }}
lmtp-filtermail_initial_destination_concurrency=10000
lmtp-filtermail_destination_concurrency_limit=10000
lmtp-filtermail_initial_destination_concurrency=500
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would not change this, this effectively reverts b05e268 and will result in head of line blocking that the change was fixing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, i reverted. i guess the file descriptor increase needs to suffice then.

Restart=always
RestartSec=30
User=vmail
LimitNOFILE=524288
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hpk42 consider parameterizing this value if we continue this path via chatmail.ini, otherwise this should work.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's all going to change eventually when chatmail/filtermail#141 gets implemented, so wouldn't expose anything as operator config right now.

@hpk42 hpk42 changed the title fix: fix #158 by increasing file descriptors for filtermail, and reduce destination concurrency fix: fix #972 by increasing file descriptors for filtermail, and reduce destination concurrency May 14, 2026
@hpk42 hpk42 force-pushed the fix158 branch 2 times, most recently from 2e9db06 to 484fb93 Compare May 14, 2026 19:20
Copy link
Copy Markdown
Contributor

@link2xt link2xt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit message is now wrong

@link2xt
Copy link
Copy Markdown
Contributor

link2xt commented May 14, 2026

Also related fix for filtermail issue, but this will need a new filtermail release: chatmail/filtermail#159

@hpk42 hpk42 merged commit a266ffd into main May 14, 2026
8 checks passed
@hpk42 hpk42 deleted the fix158 branch May 14, 2026 20:40
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.

3 participants