Skip to content
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

Websocket is not available or went away #138

Closed
tsawyer opened this issue Jun 13, 2023 · 3 comments · Fixed by #150
Closed

Websocket is not available or went away #138

tsawyer opened this issue Jun 13, 2023 · 3 comments · Fixed by #150
Labels
bug Something isn't working

Comments

@tsawyer
Copy link
Member

tsawyer commented Jun 13, 2023

Lots of nodes are shutdown or go away for valid or invalid reasons. Is there some way the Allmon3 daemon could be notified to attempt a connect when the web server requests access? I'm think having to restart the service is not the best practice to handle this common occurrence.

@tsawyer tsawyer added the enhancement New feature or request label Jun 13, 2023
@jxmx
Copy link
Member

jxmx commented Jun 13, 2023 via email

@jxmx
Copy link
Member

jxmx commented Jun 17, 2023

@tsawyer - Let me know if you're seeing this exact pattern in the your /var/log/daemon.log:

Jun 17 11:58:36 allmon3 allmon3[5505]: ERROR: (asl_allmon.ami_conn) connection failed to 172.17.16.137:5038: [Errno 111] Connection refused
Jun 17 11:58:36 allmon3 allmon3[5505]: INFO: (asl_allmon.allmon3_ws_status) connection reestablished after 1 retries
Jun 17 11:58:36 allmon3 allmon3[5505]: ERROR: (asl_allmon.ami_conn) asl_cmd_response() OSError: [Errno 9] Bad file descriptor
Jun 17 11:58:36 allmon3 allmon3[5505]: ERROR: (asl_allmon.allmon3_ws_status) received OSError: [Errno 9] Bad file descriptor; trying to reconnect
Jun 17 11:58:37 allmon3 allmon3[5505]: ERROR: (asl_allmon.ami_conn) [Errno 9] Bad file descriptor

Notably that a connection file, claims it was re-established, and then immediately again says it has a bad file descriptor. I think there's a race condition in the logic.

@jxmx jxmx added bug Something isn't working and removed enhancement New feature or request labels Jun 17, 2023
@jxmx
Copy link
Member

jxmx commented Jun 17, 2023

#142 is not related.

@jxmx jxmx mentioned this issue Jun 30, 2023
@jxmx jxmx closed this as completed in #150 Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants