Skip to content

fix: ensure listen websocket isn't opened for non-latest agents #2002

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

Merged
merged 2 commits into from
Jun 2, 2022

Conversation

coadler
Copy link
Contributor

@coadler coadler commented Jun 2, 2022

Exponential backoff is only enabled for if the websocket fails to open.
If the websocket is opened but immediately killed, the agent will try to
immediately reconnect. This is desireable in cases where coderd is being
replaced or network conditions cause the connection to die.

coadler added 2 commits June 2, 2022 14:46
Exponential backoff is only enabled if the websocket fails to open. If
the websocket is opened but immediately killed, the agent will try to
immediately reconnect. This is desireable in cases where coderd is being
replaced or network conditions cause the connection to die, but not for
permanent errors.
@coadler coadler force-pushed the colin/agent-latest-build branch from 1e00749 to 7355d57 Compare June 2, 2022 19:46
@coadler coadler merged commit 89dde21 into main Jun 2, 2022
@coadler coadler deleted the colin/agent-latest-build branch June 2, 2022 20:03
kylecarbs pushed a commit that referenced this pull request Jun 10, 2022
Exponential backoff is only enabled if the websocket fails to open. If
the websocket is opened but immediately killed, the agent will try to
immediately reconnect. This is desireable in cases where coderd is being
replaced or network conditions cause the connection to die, but not for
permanent errors.
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: Ensure agent websockets from outdated builds are rejected before websocket is opened
2 participants