Skip to content

bazci: stream test results#89845

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
rickystewart:bazci-streaming
Oct 18, 2022
Merged

bazci: stream test results#89845
craig[bot] merged 1 commit intocockroachdb:masterfrom
rickystewart:bazci-streaming

Conversation

@rickystewart
Copy link
Copy Markdown
Collaborator

@rickystewart rickystewart commented Oct 12, 2022

Have bazci start a build event service
to monitor the build, rather than parsing the bep log at the end of
the build. This gives us better behavior in CI, as test results can
"stream" instead of requiring us to wait until all tests have completed
to report.

Release note: None
Epic CRDB-15060

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@rickystewart rickystewart changed the title bazci: move the logic of posting github issues to bazci bazci: stream test results Oct 12, 2022
@rickystewart rickystewart force-pushed the bazci-streaming branch 3 times, most recently from b0e18f8 to 2a6a9b7 Compare October 17, 2022 14:52
@rickystewart rickystewart marked this pull request as ready for review October 17, 2022 15:37
@rickystewart rickystewart requested a review from a team as a code owner October 17, 2022 15:37
Have `bazci` start a [build event service](https://bazel.build/remote/bep)
to monitor the build, rather than parsing the `bep` log at the end of
the build. This gives us better behavior in CI, as test results can
"stream" instead of requiring us to wait until all tests have completed
to report.

Release note: None
@rickystewart
Copy link
Copy Markdown
Collaborator Author

bors r=healthy-pod

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 18, 2022

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Oct 18, 2022

Build succeeded:

@craig craig bot merged commit 081f108 into cockroachdb:master Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants