-
Notifications
You must be signed in to change notification settings - Fork 148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Support the asyncio debug mode and apply aiomonitor-ng #688
Conversation
the port number of aiomonitors changes due to better management of ports? |
Manager and agent's aiomonitor ports were overlapping when there are two or more worker processes in the manager (the actual aiomonitor port used by each worker process = the configured port number + |
- The "creation location" in the ps command output was wrong because 0.5.0 retrieved the other end of the stack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works as intended in my environment. LGTM.
I will make separate PRs for future updates of aiomonitor-ng. |
* feat: Add `[debug].{asyncio,enhanced-aiomonitor-task-info}` config flags in the manager, agent, and storage-proxy TOML files * fix: Prevent overlapping of manager/agent aiomonitor ports by changing the default ports to have more distant values (50100, 50200). * setup: Replace aiomonitor with aiomonitor-ng * feat: Add aiomonitor support to storage-proxy (default port: 50300) Backported-From: main Backported-To: 22.03
If the asyncio debug mode is enabled, it additionally keeps track of the stack trace of where the tasks are created though this behavior is undocumented.
Chaining of full task creation/cancellation tracebacks is handled in our modified aiomonitor.
This PR also fixes missing sample configurations for aiomonitor ports and changes the default value of them to avoid overlapping of the aiomonitor ports in manager and agent when they run in a same host (e.g., development setups).