-
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
[loadgen] Support workspace classes #10409
Conversation
c3f0c08
to
c911d58
Compare
@@ -112,7 +117,7 @@ type WsmanExecutor struct { | |||
|
|||
// StartWorkspace starts a new workspace | |||
func (w *WsmanExecutor) StartWorkspace(spec *StartWorkspaceSpec) (callDuration time.Duration, err error) { | |||
ctx, cancel := context.WithTimeout(context.Background(), 120*time.Second) | |||
ctx, cancel := context.WithTimeout(context.Background(), 240*time.Second) |
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.
non-blocking:
@aledbf wdyt of this timeout? I assume it matches your expectation, but wanted to double check.
dev/loadgen/prod-benchmark.yaml
Outdated
ideImage: eu.gcr.io/gitpod-core-dev/build/ide/code:commit-ff263e14024f00d0ed78386b4417dfa6bcd4ae2f | ||
waitForRunning: "600s" | ||
waitForStopping: "600s" | ||
successRate: 0.80 | ||
successRate: 0.95 |
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.
Honest question, why not 100%?
What does that rationale or use case look like, when it is >= 95, but, < 100?
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.
Hey @Furisto , just the one question, solid work amigo! Good call sharing the Loom video, too.
c911d58
to
0e98af8
Compare
@kylos101 PTAL |
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.
馃憤
Description
Related Issue(s)
None
How to test
You can actually now just read the documentation 馃槈
Release Notes