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

nydus: upgrad to v2.2.0 #6408

Merged
merged 2 commits into from
Mar 28, 2023
Merged

nydus: upgrad to v2.2.0 #6408

merged 2 commits into from
Mar 28, 2023

Conversation

jongwu
Copy link
Contributor

@jongwu jongwu commented Mar 3, 2023

upgrad nydus version to v2.2.0 thus it can pass the test on arm64

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Mar 3, 2023
When update the nydusd to 2.2, the argument "--hybrid-mode" cause
the following error:

thread 'main' panicked at 'ArgAction::SetTrue / ArgAction::SetFalse is defaulted'

Maybe we should remove it to upgrad nydusd

Fixes: kata-containers#6407
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
Use the latest nydus, we may let nydus work on arm64.

Fixes: kata-containers#6407
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
jongwu added a commit to jongwu/tests that referenced this pull request Mar 4, 2023
jongwu added a commit to jongwu/tests that referenced this pull request Mar 4, 2023
jongwu added a commit to jongwu/tests that referenced this pull request Mar 4, 2023
@liubin
Copy link
Member

liubin commented Mar 14, 2023

/test

@jongwu
Copy link
Contributor Author

jongwu commented Mar 23, 2023

/test-arm-snap

@jongwu
Copy link
Contributor Author

jongwu commented Mar 28, 2023

There is a bug in nydusd 2.2.0 on Arm but get fixed now. We can merge this first and wait for the next release of nydus then upgrad again. Thus I can build latest nydus to test kata + nydus on Arm.

WDYT? @liubin

@liubin liubin merged commit 75987aa into kata-containers:main Mar 28, 2023
@liubin
Copy link
Member

liubin commented Mar 28, 2023

There is a bug in nydusd 2.2.0 on Arm but get fixed now. We can merge this first and wait for the next release of nydus then upgrad again. Thus I can build latest nydus to test kata + nydus on Arm.

WDYT? @liubin

Sounds good to me.

@jongwu
Copy link
Contributor Author

jongwu commented Mar 28, 2023

Thanks! @liubin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants