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

Drag our CI to this decade #1771

Merged
merged 1 commit into from Oct 18, 2021
Merged

Conversation

pmatilai
Copy link
Member

@pmatilai pmatilai commented Sep 3, 2021

Been stuck on Fedora 32 for too long anyhow, but for debugedit we need
at least 35. Kick out a BDB remnant while at it.

@pmatilai pmatilai added the test Testsuite-related label Sep 3, 2021
@dmnks
Copy link
Contributor

dmnks commented Sep 7, 2021

OK, so this is strange. The dnf update command in the image fails with getaddrinfo() thread failed to start, see the CI logs below. I can reproduce this locally, but with Docker only. With Podman, it works fine.

@cgwalters
Copy link
Contributor

OK, so this is strange. The dnf update command in the image fails with getaddrinfo() thread failed to start, see the CI logs below. I can reproduce this locally, but with Docker only.

https://bugzilla.redhat.com/show_bug.cgi?id=1958474 probably.

@pmatilai
Copy link
Member Author

pmatilai commented Sep 8, 2021

But isn't that supposed to be fixed in F35 already?

(and btw, thanks guys for looking into it)

@wuyuoss
Copy link

wuyuoss commented Oct 14, 2021

Hey @pmatilai, wondering how does this CI upgrading from Fedora 32 to Fedora 35 go, any updates?
(we are working on rpmsign fsverity support which will depend on at least Fedora 35)

@pmatilai
Copy link
Member Author

So this is indeed strange. I just noticed that debugedit happens to be available on F34 too now, and on that version things pass just fine 😲

librepo is same on both, libcurl differs (7.76 vs 7.78 at the baseline I think), plus of course a million other things. F34 is plenty newer than F32 and testsuite passes, I take what I get...

Been stuck on Fedora 32 for too long anyhow, but for debugedit we need
at least 34. Kick out a BDB remnant while at it.
@pmatilai
Copy link
Member Author

Ookkay, this seems right on the money: https://pascalroeleven.nl/2021/09/09/ubuntu-21-10-and-fedora-35-in-docker/

Doesn't do much good for us though:

$ docker build --security-opt seccomp=unconfined -t rpm -f ci/Dockerfile .
ERRO[0000] Can't add file /home/runner/rpm/.git/objects/info/commit-graph to tar: io: read/write on closed pipe
ERRO[0000] Can't close tar writer: io: read/write on closed pipe
Error response from daemon: The daemon on this platform does not support setting security options on build

So unless Docker in Ubuntu (this really is quite mad 🤪 ) gets a fix for this issue, F34 is the last Fedora we can upgrade the CI to. We may better start packing... 🛄

@pmatilai pmatilai merged commit 20572c6 into rpm-software-management:master Oct 18, 2021
@pmatilai pmatilai deleted the ci35-pr branch February 2, 2022 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Testsuite-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants