-
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] change workspace state tracing #10597
Conversation
@csweichel @mads-hartmann |
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.
Awesome! One step closer to better traces :)
Left two nits, hence the hold.
/hold
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.
Hi! 👋
I have no idea how to test this (also, the "how to test" instructions are empty) but I'm happy to rubber-stamp-approve the 2-lines removed from components/common-go/kubernetes/kubernetes.go
(co-owned by team WebApp).
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.
Awesome - thanks a lot for following up on this 🎉
All comments addressed. /unhold |
Description
As discussed in #10264 (comment)
Related Issue(s)
Fixes #10590
How to test
Release Notes
Documentation