Skip to content

Conversation

@Mitiaj
Copy link

@Mitiaj Mitiaj commented Nov 25, 2025

  • Add all missing arguments from latest, 4.2 version.

@adrianreber
Copy link
Member

Bumping to v8 seems unnecessary as we do not break API. At least from my point of view.

@Mitiaj Mitiaj force-pushed the feat/support-criu-v4 branch 2 times, most recently from 65ca398 to 6343237 Compare November 25, 2025 15:32
@adrianreber
Copy link
Member

Much better diff 😉

rpc/rpc.proto Outdated
optional bool stream = 73;
optional bool ghost_fiemap = 74;
optional criu_file_validation_method file_validation_method = 75 [default = BUILD_ID];
optional bool deprecated_ok = 76;
Copy link
Member

@rst0git rst0git Nov 25, 2025

Choose a reason for hiding this comment

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

Hmm, these fields are not in CRIU's rpc.proto file:
https://github.com/checkpoint-restore/criu/blob/master/images/rpc.proto

Adding them here would not make any difference because CRIU will not support them.

Copy link
Member

@rst0git rst0git Nov 25, 2025

Choose a reason for hiding this comment

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

There is an open PR for the stream field (checkpoint-restore/criu#2536).

Adding these fields to the rpc.proto file also requires changes in CRIU.

Copy link
Author

Choose a reason for hiding this comment

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

makes sense. let me remove these options

Downgrade package version to v7

Signed-off-by: Michal Voitiulevic <[email protected]>
@Mitiaj Mitiaj force-pushed the feat/support-criu-v4 branch from 6343237 to 4d24d4b Compare November 25, 2025 17:03
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.

3 participants