Skip to content
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

update vendor to build on ARM64 #10805

Merged
merged 4 commits into from
Jun 17, 2019

Conversation

siddontang
Copy link
Member

What problem does this PR solve?

Build TiDB on ARM64

What is changed and how it works?

Update vendor

Check List

Tests

  • Manual test , start tidb-server on ARM64 successfully

@siddontang
Copy link
Member Author

seem the failed CI is not related to this change

PTAL @zhouqiang-cl

@shenli
Copy link
Member

shenli commented Jun 13, 2019

/run-all-tests

@zhouqiang-cl
Copy link
Contributor

go mod tidy failed

@siddontang
Copy link
Member Author

@zhouqiang-cl

Do you know how to fix it?

@zhouqiang-cl
Copy link
Contributor

@tiancaiamao PTAL

@codecov
Copy link

codecov bot commented Jun 17, 2019

Codecov Report

Merging #10805 into master will increase coverage by 0.0004%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             master     #10805        +/-   ##
================================================
+ Coverage   80.2873%   80.2878%   +0.0004%     
================================================
  Files           417        417                
  Lines         88451      88453         +2     
================================================
+ Hits          71015      71017         +2     
+ Misses        12231      12225         -6     
- Partials       5205       5211         +6

@siddontang
Copy link
Member Author

PTAL @shenli

Copy link
Member

@shenli shenli left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jackysp jackysp left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@winkyao winkyao left a comment

Choose a reason for hiding this comment

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

LGTM

@winkyao
Copy link
Contributor

winkyao commented Jun 17, 2019

/run-all-tests

@winkyao winkyao merged commit 464306b into pingcap:master Jun 17, 2019
crazycs520 added a commit to crazycs520/tidb that referenced this pull request Jul 9, 2019
@Deepak275
Copy link

@winkyao I need some help of yours :). So i am building the image for arm64 to host it in my private artifactory.. i have tried with v5.0.2 and v4.0.0 . Images is getting build but while running the container getting this standard_init_linux.go:178: exec user process caused “exec format error”. Looks like 5.0.2 version is not compatible for arm64, correct me if I am wrong. THankyou so much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants