Skip to content

/ go

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

Jump to bottom

x/tools/gopls: investigate regtest performance on linux-arm builder #43355

Open
findleyr opened this issue Dec 23, 2020 · 3 comments
Open

x/tools/gopls: investigate regtest performance on linux-arm builder #43355

findleyr opened this issue Dec 23, 2020 · 3 comments
Labels
Tools gopls
Milestone
gopls/unplanned

Comments

@findleyr
Copy link
Contributor

@findleyr findleyr commented Dec 23, 2020 •
edited

The gopls regtests are slow, particularly on openbsd and darwin due to (we believe) kernel locking behavior. For this reason we've excluded them when testing with -short on several builders where they regularly time out.

But they usually run relatively quickly on Linux, so it's surprising (to me at least) that they are timing out on the new linux-arm builder. This is probably worth investigating at some point.
@gopherbot gopherbot added this to the Unreleased milestone Dec 23, 2020
@findleyr findleyr modified the milestones: Unreleased, gopls/unplanned Dec 23, 2020
@gopherbot
Copy link

@gopherbot gopherbot commented Dec 23, 2020

Change https://golang.org/cl/280092 mentions this issue: gopls/internal/regtest: skip a some new builders where regtests time out
gopherbot pushed a commit to golang/tools that referenced this issue Dec 23, 2020
@findleyr
gopls/internal/regtest: skip a some new builders where regtests time out
3e0a2b7 
Updates golang/go#42789
Updates golang/go#43355

Change-Id: Ic344237e492b08c30c4d86db6a6a39a561e2d27b
Reviewed-on: https://go-review.googlesource.com/c/tools/+/280092
Run-TryBot: Robert Findley <rfindley@google.com>
Trust: Robert Findley <rfindley@google.com>
Reviewed-by: Rebecca Stambler <rstambler@golang.org>
gopls-CI: kokoro <noreply+kokoro@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
@dmitshur
Copy link
Member

@dmitshur dmitshur commented Dec 23, 2020 •
edited

Do you know about the regtest performance on the linux-arm-aws builder?

The linux-arm builder is not new, has a known issue and is known to be slow (we disabled trybots because of many problems). The linux-arm-aws builder is newer (#41867) and has more resources allocated, so you may get a better signal about gopls behavior on linux/arm from it.

CC @cagedmantis.
@findleyr
Copy link
Contributor Author

@findleyr findleyr commented Dec 23, 2020

Ah I see, I probably only noticed the linux-arm failure due to recent awareness of other failures (particularly openbsd).

I can try out the linux-arm-aws builder in 2021.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Tools gopls
Projects
None yet
Milestone
gopls/unplanned
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@dmitshur @gopherbot @findleyr
You can’t perform that action at this time.