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

Disconnect debug #10831

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Disconnect debug #10831

wants to merge 5 commits into from

Conversation

mattkrick
Copy link
Member

@mattkrick mattkrick commented Feb 8, 2025

logs to debug the disconnect problems

Testing this in an on-demand env I couldn't reproduce the bug where the presence wasn't found.
Perhaps the previous PR fixed it where the gql executor shuts down more gracefully.
Or the PR where the keepAlive is closed out on websocket closure.
will leave this PR sitting out a bit longer to see if we can catch the server messing up again.
I may have to test on staging instead of on demand since it may be linked to the number of executors/web servers.

Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
@github-actions github-actions bot added the size/s label Feb 8, 2025
@rafaelromcar-parabol
Copy link
Contributor

I may have to test on staging instead of on demand since it may be linked to the number of executors/web servers.

You can increase the number of both components on any on-demand-envs! You just need to change the replicaCount in the values.yaml and apply the release 😸 On Staging we have 3 web-servers and 3 gql-executors

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

Successfully merging this pull request may close these issues.

2 participants