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

ci: Don't look for the C version of virtiofsd (x86_64) #4792

Conversation

fidencio
Copy link
Member

For some reason it was not caught by the CI before, but the install_qemu
script looks for the virtiofsd in the location where it should be if
built and installed via QEMU.

This is not needed anymore, at least for x86_64, as we're using the rust
version of virtiofsd.

Fixes: #4791

Signed-off-by: Fabiano Fidêncio fabiano.fidencio@intel.com

@fidencio
Copy link
Member Author

/test

@fidencio fidencio added the no-backport-needed Changed do not need to be applied to an older branch / repository label May 16, 2022
@katacontainersbot katacontainersbot added the size/small Small and simple task label May 16, 2022
For some reason it was not caught by the CI before, but the install_qemu
script looks for the virtiofsd in the location where it should be if
built and installed via QEMU.

This is not needed anymore, at least for x86_64, as we're using the rust
version of virtiofsd.

Fixes: kata-containers#4791

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
@fidencio fidencio force-pushed the wip/fix-breakage-after-virtiofsd-switch branch from 0a9a36e to 44bb10c Compare May 16, 2022 19:24
@fidencio
Copy link
Member Author

/test

@fidencio
Copy link
Member Author

/retest-arm

1 similar comment
@fidencio
Copy link
Member Author

/retest-arm

@fidencio
Copy link
Member Author

ARM CI failure, unrelated to this PR.

23:56:54 error execution phase preflight: [preflight] Some fatal errors occurred:
23:56:54 	[ERROR Port-6443]: Port 6443 is in use

/retest-arm

@fidencio
Copy link
Member Author

/retest-arm

@fidencio fidencio merged commit a0e593f into kata-containers:main May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport-needed Changed do not need to be applied to an older branch / repository size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CIs are breaking due to leftover code looking for the C version of virtiofsd (x86_64)
6 participants