Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI fixes for Centos 7 and ALT Sisyphus #361

Merged
merged 2 commits into from
Dec 5, 2024
Merged

Conversation

vt-alt
Copy link
Contributor

@vt-alt vt-alt commented Dec 3, 2024

Description

Fix CI for Centos 7 and ALT Sisyphus.

How Has This Been Tested?

  • Centos fix on my GA.
  • It seems that flt.altlinux.org have 1 day delay behind actual repo. Thus, the fix will start to work after 24 hours.

@vt-alt vt-alt marked this pull request as draft December 4, 2024 20:53
ALT switched to numerical kernel flavours, thus no 'un-def' anymore.
Bring the new package 'kernel-headers-modules-mainline' that's supposed
to track the mainline kernels. If it fails bring 'latest1' kernel
headers. Bringing 'mainline' could fail during the merge window when
`x.y.1` stable is already released to the `x.y` flavour, so the
pseudo-flavour isn't longer 'mainline' but 'latest1' (note it ends with
`1`). This should not be the 'latest' as it provided by multiple
packages and naïve apt-get install will refuse to install.

Fixes: lkrg-org#360
Signed-off-by: Vitaly Chikunov <[email protected]>
Native GA container cannot handle `centos:7` anymore, and workarounds are
ugly. Just use native docker, which we have in cross workflow. (Even
though it will not be a cross build. Maybe it will be better renamed do
docker build.)

To support this `cross.yml` is reworked to run any command instead of
getting a simple list of packages from matrix.

Fixes: lkrg-org#359
Signed-off-by: Vitaly Chikunov <[email protected]>
@vt-alt vt-alt marked this pull request as ready for review December 4, 2024 21:48
@vt-alt
Copy link
Contributor Author

vt-alt commented Dec 4, 2024

OK All looks good today. (Except for unrelated mkosi failure fatal: unable to access 'https://git.launchpad.net/ubuntu/+source/debootstrap/': The requested URL returned error: 503)

@vt-alt
Copy link
Contributor Author

vt-alt commented Dec 4, 2024

If this all goes good we could even switch from non-wokring anymore Ubuntu mainline build/boot tests to ALT's mainline kernels, and even for aarch64. Perhaps, first add easy aarch64 build like this one here (in cross). Then when I get time could do docker boot tests.

@solardiz solardiz merged commit 54da909 into lkrg-org:main Dec 5, 2024
20 checks passed
@vt-alt
Copy link
Contributor Author

vt-alt commented Dec 6, 2024

I tested Docker-building for aarch64 fedora:rawhide, and it, curiously, took very very long in compare to ubuntu:jammy (same aarch64 docker build).

  • arm64v8/ubuntu:jammy took 5m 42s
  • quay.io/fedora/fedora:rawhide took 58m 57s

This is not usable.

ps. The patch for historical purposes.

diff --git a/.github/workflows/cross.yml b/.github/workflows/cross.yml
index d296ac10..d1c5789e 100644
--- a/.github/workflows/cross.yml
+++ b/.github/workflows/cross.yml
@@ -38,6 +38,9 @@ jobs:
                                  -e 's,^#baseurl=http://mirror\.centos\.org/centos/\$releasever,baseurl=https://vault.centos.org/7.9.2009,' \
                                 /etc/yum.repos.d/CentOS-Base.repo && \
                           yum install -y git file gcc make  kernel-devel kernel elfutils-libelf-devel
+                    - image: quay.io/fedora/fedora:rawhide
+                      platform: linux/arm64
+                      run: dnf install -y git file gcc make  elfutils-libelf-devel kernel-devel kernel
         steps:
             - uses: docker/setup-qemu-action@v2
             - uses: actions/checkout@v3

pps. time dnf install:

#6 1700.1 real	28m19.873s
#6 1700.1 user	26m26.723s
#6 1700.1 sys	2m21.983s

So this really was on cpu. I really don't understand why so extreme difference in comparison to ubuntu.

@solardiz
Copy link
Contributor

solardiz commented Dec 6, 2024

Thank you very much @vt-alt!

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