-
Notifications
You must be signed in to change notification settings - Fork 190
*: upgrade build go version, add more build direction #71
Conversation
we need to upgrade go version in JenkinsCI >= 1.11.4 to pass CI, what do you think of it? @GregoryIan @csuzhangxc |
Let's just upgrade pingcap/SRE's |
TiDB already upgraded to Go 1.12 (pingcap/tidb#9299). |
Hi, today I and guanzhongyang just upgrade tidb to go1.12 in base image, If need, I thank it is time to upgrade to 1.12 for dm/lightning/binlog |
yeah I prefer Go 1.12.
|
1.12 has incorrect checksums. It would help contributors if pingcap moved everything to a version with correct checksumes. |
hi @gregwebs Did you mean |
/run-all-tests |
|
Yes! |
Sorry, I read this wrong and you are moving to 1.12 which has correct checksums, sounds good. |
go module cache has been added in container image in JenkinsCI, so we add |
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.
LGTM
LGTM |
What problem does this PR solve?
Support DM to build with higher version of Go. Go pre-1.11.4 version may have incorrect checksum in modules. ref: golang/go#29278
What is changed and how it works?
go.sum
go vet
to supportgo 1.12
Check List
Tests