Skip to content

[1.3] criu: Add time namespace to container config after checkpoint/restore#4705

Merged
rata merged 1 commit intoopencontainers:release-1.3from
kolyshkin:1.3-4696
Apr 2, 2025
Merged

[1.3] criu: Add time namespace to container config after checkpoint/restore#4705
rata merged 1 commit intoopencontainers:release-1.3from
kolyshkin:1.3-4696

Conversation

@kolyshkin
Copy link
Copy Markdown
Contributor

Backport of #4696 to release-1.3.


Since v3.14, CRIU always restores processes into a time namespace to prevent backward jumps of monotonic and boottime clocks. This change updates the container configuration to ensure that runc exec launches new processes within the container's time namespace.

Fixes #2610

(cherry picked from commit b68cbdf)

Since v3.14, CRIU always restores processes into a time namespace to
prevent backward jumps of monotonic and boottime clocks. This change
updates the container configuration to ensure that `runc exec` launches
new processes within the container's time namespace.

Fixes opencontainers#2610

Signed-off-by: Andrei Vagin <avagin@gmail.com>
(cherry picked from commit b68cbdf)
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Copy link
Copy Markdown
Member

@rata rata left a comment

Choose a reason for hiding this comment

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

LGTM

@rata
Copy link
Copy Markdown
Member

rata commented Apr 2, 2025

@kolyshkin I've changed the branch protection rules in 1.3 to match release-1.2.

It had the "Require branches to be up to date before merging" option set, that didn't allow me to merge this. I think that option might work if we enable the auto-merge thing, but unsure if guthub will rebase automatically or what.

@rata rata merged commit fa44e9d into opencontainers:release-1.3 Apr 2, 2025
34 checks passed
@rata rata mentioned this pull request Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/checkpoint-restore backport/1.3-pr A backport PR to release-1.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants