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

fc: fix uncompressed dir path #5032

Merged
merged 3 commits into from Aug 16, 2022

Conversation

bergwolf
Copy link
Member

In the latest FC releases, the asset we download and decompress
has the following syntax:

release-vX.X.X-ARCH/firecracker-vX.X.X-ARCH

until v0.25 there was no ARCH in the folder name. Since we are updating
the supported FC version, this patch fixes the script that installs the
firecracker binary and the jailer.

Signed-off-by: Anastassios Nanos ananos@nubificus.co.uk

@katacontainersbot katacontainersbot added the size/tiny Smallest and simplest task label Aug 16, 2022
@bergwolf
Copy link
Member Author

/test

bergwolf added a commit to bergwolf/kata-containers that referenced this pull request Aug 16, 2022
These patches were backported from main branch:

05b2096 release: Adapt kata-deploy for 2.5.0
1b93015 build: Fix clh source build as normal user
01c889f runtime: Fix DisableSelinux config
59bd5c2 container: kill all of the processes in this container
22c005f nydus: upgrade nydus/nydus-snapshotter version
8220e54 runtime: add unlock before return in sendReq
4f0ca40 versions: Update Firecracker version to v1.1.0
da24fd8 clh: Don't crash if no network device is set by the upper layer
ed25d2c versions: Update Cloud Hypervisor to v25.0
dfc1413 action: extend commit message line limit to 150 bytes

Depends-on: github.com/kata-containers/tests#5032

Signed-off-by: Peng Tao <bergwolf@hyper.sh>
@fidencio
Copy link
Member

/retest-devmapper-qemu
/retest-ubuntu
/retest-arm

Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @bergwolf!

@katacontainersbot katacontainersbot added size/small Small and simple task and removed size/tiny Smallest and simplest task labels Aug 16, 2022
@bergwolf
Copy link
Member Author

/test-devmapper-qemu
/test-ubuntu
/test-arm

ananos and others added 3 commits August 16, 2022 16:48
In the latest FC releases, the asset we download and decompress
has the following syntax:

release-vX.X.X-ARCH/firecracker-vX.X.X-ARCH

until v0.25 there was no ARCH in the folder name. Since we are updating
the supported FC version, this patch fixes the script that installs the
firecracker binary and the jailer.

Fixes: kata-containers#4967
Signed-off-by: Anastassios Nanos <ananos@nubificus.co.uk>
This PR skips the nydus integration test to allow unblock the kata CI
until the issue is resolved.

Fixes kata-containers#4948

Signed-off-by: Gabriela Cervantes <gabriela.cervantes.tellez@intel.com>
Enable all debug flags to show more log for nydus test.

Depends-on: github.com/kata-containers/kata-containers#4695

Fixes: kata-containers#4947

Signed-off-by: liubin <liubin0329@gmail.com>
@bergwolf
Copy link
Member Author

/test-devmapper-qemu

@bergwolf
Copy link
Member Author

/test-ubuntu
/test-arm

@bergwolf
Copy link
Member Author

/test-clh-devmapper
/test-firecracker
/test-vfio

@bergwolf
Copy link
Member Author

/test-devmapper-clh

@bergwolf
Copy link
Member Author

/test-firecracker-ubuntu

@bergwolf
Copy link
Member Author

/test-fc

@bergwolf
Copy link
Member Author

/retest-vfio

1 similar comment
@bergwolf
Copy link
Member Author

/retest-vfio

Copy link
Contributor

@cmaf cmaf left a comment

Choose a reason for hiding this comment

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

thanks @bergwolf

@GabyCT GabyCT merged commit 7bdd998 into kata-containers:stable-2.5 Aug 16, 2022
@bergwolf bergwolf deleted the backport-stable-2.5 branch August 17, 2022 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants