koji

#2964 Unable to build docker images in CentOS 8.3+
Closed: Fixed by tkopecek. Opened by alexi.

Closed: Fixed
Reopen Issue

As described in BZ1923971, libguestfs switched its json-parsing library from YAJL to Jansson. Unfortunately, that causes a crash when Koji tries to build a docker image due to a symbol conflict with the json-c library (a dependency of dnf).

In practical terms, this means we're unable to build docker images on CentOS 8.3 and higher.

A simple, if not very elegant, workaround for this issue could look like this:

diff --git a/builder/kojid b/builder/kojid
index ba75e52e..a74fb705 100755
--- a/builder/kojid
+++ b/builder/kojid
@@ -49,6 +49,10 @@ from gzip import GzipFile
 from optparse import SUPPRESS_HELP, OptionParser
 import Cheetah.Template
+try:
+    import guestfs
+except:
+    pass
 import dnf
 import librepo
 import requests

This issue was made more difficult to diagnose due to #2963, as Koji was unable to detect the failure in libguestfs and instead it would go into a loop.

Note, simple internal workaround is to bypass importing dnf on image-only builders (it is needed only for createrepo channel). We've to import it in non-breaking order.

Metadata Update from @tkopecek:
- Custom field Size adjusted to None

So how would you suggest we do this instead?

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.27

@alexi What about #2993?

createrepo and image channels shouldn't be mixed on one machine anyway

Why not? We do mix them, so I guess this solution would still not work for us?

For me the reason is that createrepo needs ro access to /mnt/koji while image builders can live without it. So, it is safety/security least permission policy, no real obstacle.

I'll try your approach if import guestfs first is enough.

I've tried a few cases and it seems that it works. Updated PR.

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.26.1 (was: 1.27)

Metadata Update from @jcupova:
- Issue tagged with: testing-ready

Metadata Update from @jcupova:
- Issue untagged with: testing-ready
- Issue tagged with: no_qe

Commit 5fca74c9 fixes this issue

Commit 4b458ac0 fixes this issue

This issue has been migrated to Fedora Forge:
https://forge.fedoraproject.org/koji/koji/issues/2964

Please continue any further discussion there.
Metadata
None

no_qe

None
None
Low
None
Related Pull Requests
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.