Overview

Request 755694 accepted

No description set
Loading...

Dominique Leuenberger's avatar

That change can barely be correct - let alone with the work in progress to make /etc used only for admin managed config files and no system files.


Richard Brown's avatar

Indeed, transactional-updates use /usr/lib/tmpfiles.d all the time and it's exactly where packages SHOULD be storing their config.. Adding myself as reviewer to block this submission that would be worse than the status quo for transactional-updates in my view.


Ignaz Forster's avatar

As the package is not installing any tmpfiles in the spec file I assume those are generated / installed dynamically by the application at runtime. Is that correct? If so putting them into /etc would imho be fine.


William Brown's avatar

Correct - these are generated when an instance is created by dscreate, and have per-instance named and specific files that are generated with tmpfiles. That's why we needed to change to /etc as /usr/ is not writable during normal operation in transactional server.


Richard Brown's avatar

Well that's a quirky use case, but I guess it makes "sense" (as much sense as an LDAP server can ever make ;))


Dominique Leuenberger's avatar

If they are auto-generated and volatile: maybe /run would make even more sense for the usecase?


William Brown's avatar

They are not volatile "per boot" as much as "per instance". Directory Server allows multiple named instances on a single server so slapd-localhost, slapd-test, slapd-blah. Each one has a unique database, schema, ports. And they all need tmpfiles created, but they need the "instance name" ie the "localhost, test, blah" somewhere in it.

So they are long-lived, but "admin created". That's why they go into /etc.

Request History
William Brown's avatar

firstyear created request


Factory Auto's avatar

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

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Staging Bot's avatar

staging-bot added openSUSE:Factory:Staging:adi:10 as a reviewer

Being evaluated by staging project "openSUSE:Factory:Staging:adi:10"


Staging Bot's avatar

staging-bot accepted review

Picked "openSUSE:Factory:Staging:adi:10"


Dominique Leuenberger's avatar

dimstar_suse added RBrownSUSE as a reviewer

Please add your 3 cents


Richard Brown's avatar

RBrownSUSE accepted review


Dominique Leuenberger's avatar

dimstar accepted review


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:adi:10 got accepted.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:adi:10 got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:adi:10 got accepted.

openSUSE Build Service is sponsored by