Skip to content

ci: don't fail CI if criu-dev test fails#5012

Merged
rata merged 1 commit into
opencontainers:mainfrom
kolyshkin:criu-dev-ignore-fails
Dec 5, 2025
Merged

ci: don't fail CI if criu-dev test fails#5012
rata merged 1 commit into
opencontainers:mainfrom
kolyshkin:criu-dev-ignore-fails

Conversation

@kolyshkin

Copy link
Copy Markdown
Contributor

In view of recent criu-dev failure (see checkpoint-restore/criu#2781, checkpoint-restore/criu#2819), let's not fail the required "all-done" job when criu-dev tests fail.

@kolyshkin
kolyshkin marked this pull request as ready for review November 13, 2025 18:13
@kolyshkin
kolyshkin force-pushed the criu-dev-ignore-fails branch from 6296856 to 247faf5 Compare November 13, 2025 18:13
@kolyshkin

Copy link
Copy Markdown
Contributor Author

@opencontainers/runc-maintainers any opinions?

@rata
rata enabled auto-merge December 4, 2025 13:27

@rata rata left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

Not breaking runc CI for criu-dev makes sense to me. Also, criu could add some of these tests to their CI if they wanted to.

Restarted the failed check and enabled auto-merge. Hopefully it will be merged soon :)

@rata

rata commented Dec 4, 2025

Copy link
Copy Markdown
Member

CI still failing :(

In view of recent criu-dev failure, let's not fail the
required "all-done" job when criu-dev tests fail.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@lifubang
lifubang force-pushed the criu-dev-ignore-fails branch from 247faf5 to 5407cfe Compare December 5, 2025 02:54
@rata
rata merged commit ee8f6b6 into opencontainers:main Dec 5, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants