Skip to content

CI/GHA: switch to OBS criu repo, bring in criu 3.16#3222

Merged
cyphar merged 5 commits intoopencontainers:masterfrom
kolyshkin:criu-obs
Sep 24, 2021
Merged

CI/GHA: switch to OBS criu repo, bring in criu 3.16#3222
cyphar merged 5 commits intoopencontainers:masterfrom
kolyshkin:criu-obs

Conversation

@kolyshkin
Copy link
Copy Markdown
Contributor

Apparently (https://criu.org/index.php?title=Packages&diff=5197&oldid=5196) criu PPA repo, used for GHA CI, is replaced with the OBS repo (already used by Dockerfile), so let's use it.

While at it, modify the Dockefile to

  • update the repo URL to use Debian_11 (which was not available before);
  • fix the apt-key deprecation warning from Dockerfile.

See individual commits for more details.

This brings criu 3.16 to the table! 🎉

The Debian_11 was not available in this repo at the time when commit 24d318b
was made, so we had to use Debian_10 URL for Debian 11 (apparently without any
consequences).

Now Debian_11 is available, so let's switch to it.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This fixes

> Warning: apt-key is deprecated. Manage keyring files in trusted.gpg.d instead (see apt-key(8)).

Apparently, "the internets" disagree with the above, instead suggesting
using /usr/share/keyrings and a signed-by= declaration in sources.list.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This will bring criu 3.16, which is available from OBS but not (yet?) PPA.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This was supposed to be added temporarily, but slipped into the final
commit.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
As commits 120f740 and a587180 were added independently,
criu-dev go version was left at 1.16.x. Fix this.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin kolyshkin requested a review from thaJeztah September 23, 2021 20:36
@kolyshkin kolyshkin marked this pull request as ready for review September 23, 2021 20:36
@kolyshkin kolyshkin added this to the 1.1.0 milestone Sep 23, 2021
@kolyshkin
Copy link
Copy Markdown
Contributor Author

This is CI only (i.e. no changes other than CI) and is needed

@kolyshkin kolyshkin mentioned this pull request Sep 23, 2021
Copy link
Copy Markdown
Member

@cyphar cyphar left a comment

Choose a reason for hiding this comment

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

LGTM.

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