Skip to content

Add apt dependencies for xen-sys CI tests#120

Merged
roypat merged 2 commits intorust-vmm:mainfrom
epilys:add-podman-for-xen-sys
Jan 15, 2025
Merged

Add apt dependencies for xen-sys CI tests#120
roypat merged 2 commits intorust-vmm:mainfrom
epilys:add-podman-for-xen-sys

Conversation

@epilys
Copy link
Member

@epilys epilys commented Jan 14, 2025

Install podman in order to be able to build and run containers inside
the CI container, and install binutils-aarch64-linux-gnu since the
aarch64-linux-gnu-objcopy binary is required to be able to create
aarch64 kernel ELFs for testing.

Signed-off-by: Manos Pitsidianakis manos.pitsidianakis@linaro.org

If --load is not provided to "docker build" command, the image only
remains in the local cache and is not available for use locally when
doing "docker run rustvmm/dev:$VERSION" like the README suggests.

Without this change using the image locally is not possible, the image
does not show up when doing `docker image ls` and the tag is not found
when doing `docker run <TAG>`. the `docker build` step prints this
warning: "WARNING: No output specified with docker-container driver.
Build result will only remain in the build cache."

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
Install podman in order to be able to build and run containers inside
the CI container, and install binutils-aarch64-linux-gnu since the
aarch64-linux-gnu-objcopy binary is required to be able to create
aarch64 kernel ELFs for testing.

Signed-off-by: Manos Pitsidianakis <manos.pitsidianakis@linaro.org>
@epilys epilys force-pushed the add-podman-for-xen-sys branch from 346e03b to a6859cd Compare January 15, 2025 07:26
@epilys
Copy link
Member Author

epilys commented Jan 15, 2025

CI has completed successfully, this can be merged

@roypat roypat merged commit b0fe0ef into rust-vmm:main Jan 15, 2025
3 checks passed
@epilys epilys deleted the add-podman-for-xen-sys branch January 15, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants