salt: [BUG] Memory leak in master ProcessManager
Description We updated our infrastructure from 3002.6 to 3003 and observed a rather severe memory leak in the ProcessManager of the salt master. We have both OSS and SSE masters and could observe the same behavior on all of them.
Steps to Reproduce the behavior Issue is repeatable here and immediately visible after restarting salt-master (trend is visible after 5-15 minutes)
Screenshots

Versions Report
salt --versions-report
Salt Version:
Salt: 3003
Dependency Versions:
cffi: 1.14.1
cherrypy: unknown
dateutil: Not Installed
docker-py: Not Installed
gitdb: Not Installed
gitpython: Not Installed
Jinja2: 2.11.1
libgit2: 1.0.1
M2Crypto: 0.35.2
Mako: Not Installed
msgpack: 0.6.2
msgpack-pure: Not Installed
mysql-python: 1.3.12
pycparser: 2.14
pycrypto: Not Installed
pycryptodome: 3.10.1
pygit2: 1.2.1
Python: 3.6.8 (default, Nov 16 2020, 16:55:22)
python-gnupg: Not Installed
PyYAML: 3.13
PyZMQ: 17.0.0
smmap: Not Installed
timelib: Not Installed
Tornado: 4.5.3
ZMQ: 4.1.4
Salt Extensions:
sseape: 8.3.0+4
System Versions:
dist: centos 7 Core
locale: UTF-8
machine: x86_64
release: 3.10.0-1127.19.1.el7.x86_64
system: Linux
version: CentOS Linux 7 Core
About this issue
- Original URL
- State: closed
- Created 3 years ago
- Comments: 17 (16 by maintainers)
Commits related to this issue
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to dwoz/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to dwoz/salt by dwoz 3 years ago
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to saltstack/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to saltstack/salt by dwoz 3 years ago
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to Ch3LL/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to Ch3LL/salt by dwoz 3 years ago
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to saltstack/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to saltstack/salt by dwoz 3 years ago
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to xeacott/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to xeacott/salt by dwoz 3 years ago
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to xeacott/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to xeacott/salt by dwoz 3 years ago
- Add connect to IPCMessageSubscriber's async_methods Fixes #60049 by making sure an IPCMessageSubscriber that is wrapped by SyncWrapper has a connect method that runs the coroutine rather than returns... — committed to xeacott/salt by dwoz 3 years ago
- Add changelog for #60049 — committed to xeacott/salt by dwoz 3 years ago
- Merge freeze into master (#60396) * Merge 3002.6 bugfix changes (#59822) * Pass `CI_RUN` as an environment variable to the test run. This allows us to know if we're running the test suite under... — committed to saltstack/salt by Ch3LL 3 years ago
- Merge freeze into master (#60396) * Merge 3002.6 bugfix changes (#59822) * Pass `CI_RUN` as an environment variable to the test run. This allows us to know if we're running the test suite under... — committed to truzzon/salt by Ch3LL 3 years ago
- Merge 3003.3 into master (#60924) * Merge 3002.6 bugfix changes (#59822) * Pass `CI_RUN` as an environment variable to the test run. This allows us to know if we're running the test suite under... — committed to saltstack/salt by Ch3LL 3 years ago
If you install setproctitle python module it will list what most of the processes do. there are two or three that don’t have labels though. for troubleshooting memory leaks it is important to know which process is causing the problems.
Also since you are running 3002.2 the memory leak you are reporting will be vastly different then the one in this ticket. 3002.2 had it’s own memory issues. this ticket should be for 3003+