Skip to content

build: upgrade to go1.10.3 #27597

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

Merged
merged 1 commit into from
Jul 15, 2018
Merged

build: upgrade to go1.10.3 #27597

merged 1 commit into from
Jul 15, 2018

Conversation

benesch
Copy link
Contributor

@benesch benesch commented Jul 15, 2018

go1.10 has a bug in its test2json tool that causes test names with colons in
them to be misparsed (golang/go#23920). This was causing the nightly stress
issue poster to incorrectly post dozens of issues about tests that had not
actually failed. Upgrade the builder to go1.10.3, which includes a fix for the
test2json tool.

Fix #27498.

Release note: None

@benesch benesch requested a review from a team July 15, 2018 17:01
@cockroach-teamcity
Copy link
Member

This change is Reviewable

go1.10 has a bug in its test2json tool that causes test names with colons in
them to be misparsed (golang/go#23920). This was causing the nightly stress
issue poster to incorrectly post dozens of issues about tests that had not
actually failed. Upgrade the builder to go1.10.3, which includes a fix for the
test2json tool.

Fix cockroachdb#27498.

Release note: None
Copy link
Member

@tbg tbg left a comment

Choose a reason for hiding this comment

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

Big fat lgtm. Thanks!

@benesch
Copy link
Contributor Author

benesch commented Jul 15, 2018

TFTR! Really excited to not have to clean up 50+ incorrect stress failures tonight. :)

bors r=tschottdorf

craig bot pushed a commit that referenced this pull request Jul 15, 2018
27597: build: upgrade to go1.10.3 r=tschottdorf a=benesch

go1.10 has a bug in its test2json tool that causes test names with colons in
them to be misparsed (golang/go#23920). This was causing the nightly stress
issue poster to incorrectly post dozens of issues about tests that had not
actually failed. Upgrade the builder to go1.10.3, which includes a fix for the
test2json tool.

Fix #27498.

Release note: None

Co-authored-by: Nikhil Benesch <[email protected]>
@craig
Copy link
Contributor

craig bot commented Jul 15, 2018

Build succeeded

@craig craig bot merged commit 45ec4be into cockroachdb:master Jul 15, 2018
@benesch benesch deleted the go1.10.3 branch July 17, 2018 22:58
@knz knz mentioned this pull request Aug 27, 2018
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.

testing: teamcity status reporting from tests has been garbage lately
3 participants