Overview

Request 504997 superseded

needs to go with SR#504984


Dominique Leuenberger's avatar

This one seems to be responsible for quite some weird fallouts - likely some configure scripts now no longer find one of the files they are looking for (as update-ca-certificates -f no longer runs on the build system)

See for example build fails of

docker
mono-core
libqca2
libqca-qt5

CC @kukuk who made the change; the build system is not systemd managed and has does not fire the services (init=/.build/build)

The best course might be to still call update-ca-certificates if -f /.buildenv


Ludwig Nussel's avatar

I'd rather go for something like test -w /var/lib/ca-certificates


Dominique Leuenberger's avatar

Sounds like a reasonable option too; but that would mean on a running system with the service enabled, it is running twice: in the post script and then triggered by the service again


Ludwig Nussel's avatar

yes and no. If no ca-certificates change the service wouldn't trigger and even if it would most hook script won't regenerate their output. That's why the %post calls update-ca-certificates with the -f option to force regenerate all output files.


Thorsten Kukuk's avatar

Hm, /var/lib/ca-certificates is always writeable if we update ca-certificates, even in transactional-update case. But the result may not be correct, since not all certificates are accessible (/etc).


Dominique Leuenberger's avatar

But those would be corrected on next boot, when the oneshot service comes up and re-generates the files, no? (and reboot is needed for transactional updates in any case)


Ludwig Nussel's avatar

or we add that test directly to update-ca-certificates


Dominique Leuenberger's avatar

Any consensus on the way forward here?


Thorsten Kukuk's avatar

Yes, see bsc#1045942, transactional-update has the needed changes in Factory, waiting for Ludwig.


Dominique Leuenberger's avatar

Backlog, until a fix can be prepared to not break build system setups

Request History
Ludwig Nussel's avatar

lnussel created request

needs to go with SR#504984


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto added factory-repo-checker as a reviewer

Please review build success


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Yuchen Lin's avatar

maxlin_factory set openSUSE:Factory:Staging:F as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:F"


Yuchen Lin's avatar

maxlin_factory accepted review

Picked openSUSE:Factory:Staging:F


Dominique Leuenberger's avatar

dimstar accepted review


Factory Repo Checker's avatar

factory-repo-checker reopened review

ca-certificates is still building for repository openSUSE_Factory


Factory Repo Checker's avatar

factory-repo-checker accepted review

Builds for repo openSUSE:Factory:Staging:F/standard


Dominique Leuenberger's avatar

dimstar_suse approved review

Removing from openSUSE:Factory:Staging:F, re-evaluation needed


Dominique Leuenberger's avatar

dimstar_suse accepted review

Removing from openSUSE:Factory:Staging:F, re-evaluation needed


Dominique Leuenberger's avatar

dimstar_suse added factory-staging as a reviewer

Requesting new staging review


Ludwig Nussel's avatar

lnussel superseded request

superseded by 515015

openSUSE Build Service is sponsored by