Skip to content

Dockerfile: pin criu version and build from source#44086

Open
crazy-max wants to merge 1 commit intomoby:masterfrom
crazy-max:criu-src
Open

Dockerfile: pin criu version and build from source#44086
crazy-max wants to merge 1 commit intomoby:masterfrom
crazy-max:criu-src

Conversation

@crazy-max
Copy link
Member

@crazy-max crazy-max commented Sep 2, 2022

related to #43529

- What I did

pin criu for better reproducibility and build from source.
will solve issues we have with flakiness using the opensuse mirror
but also being able to build on other platforms:

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: CrazyMax [email protected]

@crazy-max crazy-max force-pushed the criu-src branch 2 times, most recently from 9d36239 to 98f888c Compare September 3, 2022 20:09
@crazy-max crazy-max requested a review from thaJeztah September 4, 2022 16:07
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah added this to the v-next milestone Sep 4, 2022
@thaJeztah
Copy link
Member

@tianon PTAL (sorry, looks like we're going back-and-forth on pre-built vs compile, but these package-repos are not very stable)

@cpuguy83
Copy link
Member

:sadpanda:

I really don't want to have to build this from source every time. Yeah it gets cached, but I don't always have the build cached.
It would be really nice to add this as an image to ghcr or something that we can just reference in the Dockerfile.

prabhav-thali added a commit to linux-on-ibm-z/moby that referenced this pull request Sep 19, 2022
@neersighted
Copy link
Member

I would like to see this (or some other approach) merged as it's bit me a few times -- can we reach consensus on this approach?

@thaJeztah
Copy link
Member

@cpuguy83 is that a "strong" no, or would you be ok with having this as an (intermediate?) solution?

@crazy-max also mentioned that the package is not available for armhf and riscv (but criu itself does support it? albeit still WIP); ideally that would be fixed in the SUSE repo of course.

While discussing; I was also looking where we actually use CRIU; I think this was for the integration tests, but it looks like the actual tests were broken, and currently skipped. For fun, I opened #44495 to see if they actually work.

pin criu for better reproducibility and build from source.
will solve issues we have with flakiness using the opensuse mirror
but also being able to build on other platforms

Signed-off-by: CrazyMax <[email protected]>
@crazy-max crazy-max force-pushed the criu-src branch 2 times, most recently from 202f248 to 759b6d7 Compare November 18, 2022 15:43
@rumpl rumpl modified the milestones: 24.0.0, v-future Apr 24, 2023
@kolyshkin
Copy link
Contributor

I'm against doing that. This increases test time, creates some extra build logs, and in general is a waste of resources, especially since this PR builds the same criu version over and over.

@thaJeztah thaJeztah added the area/checkpoint Related to (experimental) checkpoint/restore (CRIU) label Mar 4, 2024
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.

6 participants