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

feat(api): console,vnc reconnect #553

Merged
merged 8 commits into from
Dec 5, 2024
Merged

Conversation

danilrwx
Copy link
Contributor

@danilrwx danilrwx commented Dec 4, 2024

Description

Graceful termination of websocket connection for serial console and vnc connections

Why do we need it, and what problem does it solve?

Add the ability to handle connection termination if another user connects to the console or vnc

What is the expected result?

In case of connection of another user, send websocket connection closing message

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

@danilrwx danilrwx marked this pull request as draft December 4, 2024 14:20
Signed-off-by: Daniil Antoshin <[email protected]>
Signed-off-by: Daniil Antoshin <[email protected]>
Signed-off-by: Daniil Antoshin <[email protected]>
Signed-off-by: Daniil Antoshin <[email protected]>
Signed-off-by: Daniil Antoshin <[email protected]>
@danilrwx danilrwx force-pushed the feat/api/console-vnc-reconnect branch from d590a89 to 6d18cc1 Compare December 5, 2024 11:02
@danilrwx danilrwx marked this pull request as ready for review December 5, 2024 11:02
@yaroslavborbat yaroslavborbat self-requested a review December 5, 2024 11:09
Signed-off-by: Daniil Antoshin <[email protected]>
yaroslavborbat
yaroslavborbat previously approved these changes Dec 5, 2024
Signed-off-by: Daniil Antoshin <[email protected]>
@danilrwx danilrwx merged commit b0501a9 into main Dec 5, 2024
12 checks passed
@danilrwx danilrwx deleted the feat/api/console-vnc-reconnect branch December 5, 2024 12:11
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.

3 participants