Loading...
Request History
jengelh created request
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
anag+factory set openSUSE:Factory:Staging:E as a staging project
Being evaluated by staging project "openSUSE:Factory:Staging:E"
anag+factory accepted review
Picked "openSUSE:Factory:Staging:E"
dimstar accepted review
dimstar_suse accepted review
Staging Project openSUSE:Factory:Staging:E got accepted.
dimstar_suse approved review
Staging Project openSUSE:Factory:Staging:E got accepted.
dimstar_suse accepted request
Staging Project openSUSE:Factory:Staging:E got accepted.
creating and packaging the directory for the user could have been a nice aid; possibly putting a README (hoping sssd would only load *.conf files) with some starting points on how to configure it
Could the example be packaged as %doc, so that users have at least some reference ?
Shipping empty directories in /etc, while not as bad as shipping files in /etc, is still a cosmetic nuisance as far as the container efforts go. You never if the admin resets/replaces /etc in some form or fashion, and so rpm would be racing with the admin in possibly reinstating the directory everytime (if it can — is readonly-/etc a thing yet?). So I think it is better to not any /etc files, regular or directory, with rpms.
Read-only /etc is not anything I heard about so far. Admins are supposed to configure their systems
Even pam installs /etc/pam.d for the users to know where to put their config. The dir-structure being there helps the users guide to how to do the configuration
Hm, yes. But also:
man sssd.conf
-> EXAMPLES. What do you think about that?basically anything giving the users some hints / starting points sounds like a good thing to offer
One important thing (which the template sssd.conf just offered) is the fact that sssd.conf MUST be mode 0600 - or sssd refuses to start