Skip to content

Conversation

@rst0git
Copy link
Contributor

@rst0git rst0git commented Oct 29, 2025

testing criu-dev as per discussion at checkpoint-restore/criu#2781 (comment)

1. Add status check support (same as in bats' run helper).

2. Add RUNC_PRE_CMD support (so we can use commands like taskset or
   timeout).

3. Drop sane_helper since the output of the command is shown in case of
   an error, and we show the command itself in runc wrapper (unless -N
   or ! is provided -- in this case the command is shown by bats,
   together with the error). This does not show the output of successful
   commands which IMO is a net positive since we are almost always
   interested in failed command output only.

4. Use the new functionality in cpu_affinity.bats and start.bats as a
   showcase (the test of refactoring is in a separate commit).

Signed-off-by: Kir Kolyshkin <[email protected]>
These places should check runc exit code but they don't.

Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Kir Kolyshkin <[email protected]>
@rst0git rst0git force-pushed the test-criu-dev branch 2 times, most recently from b31c22d to e23974b Compare October 30, 2025 08:39
@rst0git rst0git closed this Oct 30, 2025
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.

2 participants