Skip to content

Conversation

chrismeyersfsu
Copy link
Member

@chrismeyersfsu chrismeyersfsu commented Mar 27, 2024

SUMMARY

Fix websocket relay.

Fixes #15038

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API
AWX VERSION
devel
ADDITIONAL INFORMATION

@fosterseth
Copy link
Member

fixes #15038

@TheRealHaoLiu TheRealHaoLiu changed the title Without this conn.notifies() blocks forever Fix websocket relay. Without this conn.notifies() blocks forever Mar 27, 2024
@TheRealHaoLiu TheRealHaoLiu changed the title Fix websocket relay. Without this conn.notifies() blocks forever Fix websocket relay. Set autocommit so conn.notifies() does not blocks forever Mar 27, 2024
@TheRealHaoLiu TheRealHaoLiu merged commit 0b5e59d into ansible:devel Mar 27, 2024
djyasin pushed a commit to djyasin/awx that referenced this pull request Sep 16, 2024
…s forever (ansible#15043)

Without autocommit conn.notifies() blocks forever
djyasin pushed a commit to djyasin/awx that referenced this pull request Nov 11, 2024
…s forever (ansible#15043)

Without autocommit conn.notifies() blocks forever
djyasin pushed a commit to djyasin/awx that referenced this pull request Jul 22, 2025
…s forever (ansible#15043) (ansible#6588)

Without autocommit conn.notifies() blocks forever

Co-authored-by: Chris Meyers <[email protected]>
djyasin pushed a commit to djyasin/awx that referenced this pull request Jul 22, 2025
* Revert "[Backport][4.5] Fix wsrelay connection leak (ansible#6589)"

This reverts commit 090350d.

* Revert "Fix websocket relay. Set autocommit so conn.notifies() does not blocks forever (ansible#15043) (ansible#6588)"

This reverts commit 3f3b764.

* Revert "[4.5][backport] Fix wsrelay FATAL and not responsive after db disconnect (ansible#6584)"

This reverts commit 5c87889.

* Allow wsrelay to crash and prevent FATAL

- sleep before crashing to prevent rapid restart
- log after sleep because logging require DB connection and that can potentially cause another unhandled exception
- lower startsec for wsrelay to 1 (default) to prevent FATAL

* Backport ansible#14879
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.

[24.1.0] Job output not loading -- status not updating
3 participants