Skip to content

Conversation

@hanzei
Copy link
Contributor

@hanzei hanzei commented Jan 27, 2026

Summary

Add documentation note explaining the migration from pids_limit to mem_limit in Docker deployments.

Background

A customer experienced PostgreSQL connection failures due to hitting pids_limit: 100 in production. PostgreSQL creates one process per database connection, so this limit restricted deployments to approximately 90-95 concurrent connections before failing.

Changes

Added a note to the Docker deployment documentation explaining:

  • Why pids_limit was removed from the deployment configuration
  • The technical reason for the failures (PostgreSQL's one-process-per-connection model)
  • The recommended alternative (mem_limit)
  • Link to scaling documentation for guidance

Related PR

Add note explaining why pids_limit was removed in favor of mem_limit.
PostgreSQL's one-process-per-connection model caused production
failures with pids_limit: 100, restricting deployments to ~90-95
concurrent connections.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@hanzei hanzei added the 1: Dev Review Requires review by a core commiter label Jan 27, 2026
@hanzei hanzei requested a review from lieut-data January 27, 2026 14:45
@github-actions
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 610bab1

@hanzei hanzei requested a review from cwarnermm January 27, 2026 15:22
@hanzei hanzei added 2: Editor Review Requires review by an editor and removed 1: Dev Review Requires review by a core commiter labels Jan 27, 2026
Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thanks, @hanzei!

@cwarnermm cwarnermm added 3: Reviews Complete All reviewers have approved the pull request and removed 2: Editor Review Requires review by an editor labels Jan 27, 2026
@hanzei hanzei merged commit 538f868 into master Jan 27, 2026
6 checks passed
@hanzei hanzei deleted the pids_limit branch January 27, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3: Reviews Complete All reviewers have approved the pull request Guidance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants