-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[ws-manager] Refactor connectToWorkspaceDaemon #11661
Conversation
Why do we need to revert #11517? I ask because if we revert it, after all, we have to wait until ws-damon pod running, don't we? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
We updated ws-daemon with the CI image. |
Related to #11664 |
I don't understand what you mean. :( |
I am sorry. I updated the ws-daemon daemonset image using the werft image from this PR.
It was bad. That said, we need to review the use of a pool for the connections. |
Sorry for being annoying, but for posterity, why was that change bad? |
Description
Reverts #11517 and improve error messages
Release Notes
Werft options: