Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

bench-bot is not responding to commands sometimes #83

Closed
joao-paulo-parity opened this issue Jan 27, 2022 · 1 comment · Fixed by #84
Closed

bench-bot is not responding to commands sometimes #83

joao-paulo-parity opened this issue Jan 27, 2022 · 1 comment · Fixed by #84
Assignees

Comments

@joao-paulo-parity
Copy link
Contributor

e.g. paritytech/polkadot#4777 (comment)

might be related to #68

or might be related to some other problem which is not visible from the logs at the moment #82

@joao-paulo-parity joao-paulo-parity self-assigned this Jan 28, 2022
@joao-paulo-parity
Copy link
Contributor Author

My current guess is related to the proxy we're using for receiving events (https://github.com/paritytech/devops/issues/1070). Specifically after we reconnect:

ERROR (server):                                                                                        
    type: "error"                                                                                      
INFO (server): Connected    

It seems that although the server says it's "Connected" again, we might no longer receive events.

Here's the trail of code sources leading up to this assumption:

If the guess is correct then we can try to restart the whole server when an error happens instead of reconnecting, but that implies a refactor in the server's setup.

This problem might originate from the library we're using, https://github.com/EventSource/eventsource, which does not appear to be well-maintained. If this bot were to be rewritten in another language as suggested by #54 then the error could end up being solved by using a (better and) different implementation of EventSource, e.g. in processbot we're using a different library (https://github.com/paritytech/parity-processbot/blob/2f8ffedb62a2abc1a3b50c983087e80c1d8677be/src/main.rs#L72).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant