Skip to content

Change ping/pong mechanism to update node status since callbacks are …#593

Merged
bartvanb merged 1 commit intomainfrom
bugfix/node-status-without-callback
Mar 13, 2023
Merged

Change ping/pong mechanism to update node status since callbacks are …#593
bartvanb merged 1 commit intomainfrom
bugfix/node-status-without-callback

Conversation

@bartvanb
Copy link
Copy Markdown
Member

…not fully supported by python socketio.

Now, sending a ping from each node to the server every minute instead. The ping leads to updated 'last_seen' and status 'online'. Additionally, the server has a new thread that checks every minute (+ a few seconds) which online nodes have not pinged and puts them to offline.

Fix #535

…not fully supported by python socketio.

Now, sending a ping from each node to the server every minute instead. The ping leads to updated 'last_seen' and status 'online'. Additionally, the server has a new thread that checks every minute (+ a few seconds) which online nodes have not pinged and puts them to offline.
Copy link
Copy Markdown
Contributor

@frankcorneliusmartin frankcorneliusmartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥇

@bartvanb bartvanb merged commit 9234737 into main Mar 13, 2023
@bartvanb bartvanb deleted the bugfix/node-status-without-callback branch March 13, 2023 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Node status ping/pong doesn't work when deployed

2 participants