Skip to content

Fix wsrelay connection leak#15113

Merged
TheRealHaoLiu merged 1 commit into
ansible:develfrom
TheRealHaoLiu:fix-wsrelay-connection-leak
Apr 16, 2024
Merged

Fix wsrelay connection leak#15113
TheRealHaoLiu merged 1 commit into
ansible:develfrom
TheRealHaoLiu:fix-wsrelay-connection-leak

Conversation

@TheRealHaoLiu
Copy link
Copy Markdown
Member

SUMMARY
  • when re-establishing connection to db close old connection
  • re-initialize WebSocketRelayManager when restarting asyncio.run
  • log and ignore error in cleanup_offline_host (this might come back to bite us)
  • cleanup connection when WebSocketRelayManager crash
ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API
AWX VERSION

ADDITIONAL INFORMATION

- when re-establishing connection to db close old connection
- re-initialize WebSocketRelayManager when restarting asyncio.run
- log and ignore error in cleanup_offline_host (this might come back to bite us)
- cleanup connection when WebSocketRelayManager crash
@TheRealHaoLiu TheRealHaoLiu force-pushed the fix-wsrelay-connection-leak branch from 9515ae5 to 42c0e3a Compare April 16, 2024 16:49
@TheRealHaoLiu TheRealHaoLiu merged commit e873bb1 into ansible:devel Apr 16, 2024
@TheRealHaoLiu TheRealHaoLiu deleted the fix-wsrelay-connection-leak branch April 16, 2024 18:54
djyasin pushed a commit to djyasin/awx that referenced this pull request Sep 16, 2024
- when re-establishing connection to db close old connection
- re-initialize WebSocketRelayManager when restarting asyncio.run
- log and ignore error in cleanup_offline_host (this might come back to bite us)
- cleanup connection when WebSocketRelayManager crash
djyasin pushed a commit to djyasin/awx that referenced this pull request Nov 11, 2024
- when re-establishing connection to db close old connection
- re-initialize WebSocketRelayManager when restarting asyncio.run
- log and ignore error in cleanup_offline_host (this might come back to bite us)
- cleanup connection when WebSocketRelayManager crash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants