Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

version: add new docker version entry for agent test on arm #3057

Merged
merged 1 commit into from
Nov 25, 2020

Conversation

jongwu
Copy link
Contributor

@jongwu jongwu commented Nov 11, 2020

make proto in agent test on arm64 will fail when using docker 18.06
and using docker 19.03 can avoid that failure. So a new entry of
docker version for agent on arm64 is added.

Signed-off-by: Jianyong wu jianyong.wu@arm.com
Fixes: #3056

@jodh-intel @devimc

jongwu added a commit to jongwu/agent that referenced this pull request Nov 11, 2020
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: kata-containers#861
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
jongwu added a commit to jongwu/agent that referenced this pull request Nov 11, 2020
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: kata-containers#861
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
jongwu added a commit to jongwu/agent that referenced this pull request Nov 11, 2020
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: kata-containers#861
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
@devimc
Copy link

devimc commented Nov 11, 2020

thanks @jongwu , but

INFO: Running 'static_check_versions' function
versions.yaml
  219:9     error    wrong indentation: expected 6 but found 8  (indentation)

@pmores
Copy link

pmores commented Nov 11, 2020

lgtm

make proto in agent test on arm64 will fail when using docker 18.06
and using docker 19.03 can avoid that failure. So a new entry of
docker version for agent on arm64 is added.

Signed-off-by: Jianyong wu <jianyong.wu@arm.com>
Fixes: kata-containers#3056
jongwu added a commit to jongwu/agent that referenced this pull request Nov 12, 2020
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: kata-containers#861
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
jongwu added a commit to jongwu/agent that referenced this pull request Nov 17, 2020
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: kata-containers#861
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
jongwu added a commit to jongwu/agent that referenced this pull request Nov 17, 2020
"make proto" will fail on arm64 using docker 18.06. This bug will be
gone if using docker 19.03. so upgrade docker before "make proto"
for arm64.

Depends-on: github.com/kata-containers/runtime#3057
Fixes: kata-containers#861
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
@jongwu
Copy link
Contributor Author

jongwu commented Nov 18, 2020

/test

@codecov
Copy link

codecov bot commented Nov 18, 2020

Codecov Report

Merging #3057 (27c558e) into master (dcd2dc3) will increase coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3057      +/-   ##
==========================================
+ Coverage   50.28%   50.44%   +0.16%     
==========================================
  Files         120      120              
  Lines       15841    15917      +76     
==========================================
+ Hits         7966     8030      +64     
- Misses       6789     6796       +7     
- Partials     1086     1091       +5     

@jongwu
Copy link
Contributor Author

jongwu commented Nov 23, 2020

Hello @jodh-intel @lifupan, any comments?

@jongwu
Copy link
Contributor Author

jongwu commented Nov 25, 2020

hello @jodh-intel @devimc , is it time to merge this PR?

Copy link

@devimc devimc left a comment

Choose a reason for hiding this comment

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

thanks @jongwu

@devimc devimc merged commit 806cc56 into kata-containers:master Nov 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

make proto in agent will fail when using docker 18.06 on arm64
4 participants