Skip to content

teamcity-trigger: run nightly stressrace with -p 1#56775

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:stress-no-parallel
Nov 24, 2020
Merged

teamcity-trigger: run nightly stressrace with -p 1#56775
craig[bot] merged 1 commit intocockroachdb:masterfrom
andreimatei:stress-no-parallel

Conversation

@andreimatei
Copy link
Copy Markdown
Contributor

We're currently running with -p 2. We're seeing the TeamCity machines
struggle with this - we're getting mysterious failures with logs spammed
with slow requests and stalled disks.

Fixes #56693
Fixes #56484
Fixes #56355

Tests found by searching for "disk slowness detected" and applying a
modicum of common sense.

Fixes #55404
Fixes #55418
Fixes #55420
Fixes #55421
Fixes #55423
Fixes #55574
Fixes #55627
Fixes #55799
Fixes #55884
Fixes #55930
Fixes #56037
Fixes #56106
Fixes #56172
Fixes #56274
Fixes #56402
Fixes #56702
Fixes #56704

Release note: None

@andreimatei andreimatei requested review from jlinder, nvb and tbg November 16, 2020 23:49
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor Author

@andreimatei andreimatei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @jlinder, @nvanbenschoten, and @tbg)


pkg/cmd/teamcity-trigger/main.go, line 112 at r1 (raw file):

		// Run race build. With run with -p 1 to avoid overloading the machine.
		noParallelism := 1
		opts["env.GOFLAGS"] = fmt.Sprintf("-race -parallel=%d", parallelism)

I've let this be higher on purpose...

Copy link
Copy Markdown
Collaborator

@jlinder jlinder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a reasonable approach to try. :lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @jlinder, @nvanbenschoten, and @tbg)

@andreimatei andreimatei requested a review from a team as a code owner November 24, 2020 01:49
We're currently running with -p 2. We're seeing the TeamCity machines
struggle with this - we're getting mysterious failures with logs spammed
with slow requests and stalled disks.

Fixes cockroachdb#56693
Fixes cockroachdb#56484
Fixes cockroachdb#56355

Tests found by searching for "disk slowness detected" and applying a
modicum of common sense.

Fixes cockroachdb#55404
Fixes cockroachdb#55418
Fixes cockroachdb#55420
Fixes cockroachdb#55421
Fixes cockroachdb#55423
Fixes cockroachdb#55574
Fixes cockroachdb#55627
Fixes cockroachdb#55799
Fixes cockroachdb#55884
Fixes cockroachdb#55930
Fixes cockroachdb#56037
Fixes cockroachdb#56106
Fixes cockroachdb#56172
Fixes cockroachdb#56274
Fixes cockroachdb#56402
Fixes cockroachdb#56702
Fixes cockroachdb#56704
Fixes cockroachdb#56717
Fixes cockroachdb#56794
Fixes cockroachdb#56889
Fixes cockroachdb#56891
Fixes cockroachdb#56946
Fixes cockroachdb#56990

Release note: None
@andreimatei
Copy link
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Nov 24, 2020

Build succeeded:

@craig craig bot merged commit 48f860b into cockroachdb:master Nov 24, 2020
@andreimatei andreimatei deleted the stress-no-parallel branch November 24, 2020 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment