Bug 227771
Summary: | FHS: use sysconfdir (/etc) as config file location | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Retired] 389 | Reporter: | Rich Megginson <rmeggins> | ||||||||||||
Component: | Directory Server | Assignee: | Rich Megginson <rmeggins> | ||||||||||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Viktor Ashirov <vashirov> | ||||||||||||
Severity: | medium | Docs Contact: | |||||||||||||
Priority: | medium | ||||||||||||||
Version: | 1.0.4 | ||||||||||||||
Target Milestone: | --- | ||||||||||||||
Target Release: | --- | ||||||||||||||
Hardware: | All | ||||||||||||||
OS: | All | ||||||||||||||
Whiteboard: | |||||||||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||||||||
Doc Text: | Story Points: | --- | |||||||||||||
Clone Of: | Environment: | ||||||||||||||
Last Closed: | 2015-12-07 16:56:15 UTC | Type: | --- | ||||||||||||
Regression: | --- | Mount Type: | --- | ||||||||||||
Documentation: | --- | CRM: | |||||||||||||
Verified Versions: | Category: | --- | |||||||||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||||||||
Embargoed: | |||||||||||||||
Bug Depends On: | |||||||||||||||
Bug Blocks: | 152373, 240316, 427409 | ||||||||||||||
Attachments: |
|
Description
Rich Megginson
2007-02-08 00:09:30 UTC
Created attachment 147682 [details]
diffs
There were also two other problems:
1) repl-monitor-cgi has a dependency on Cgi.pm. The core fedora-ds package
does not support cgis, so we should remove it. This also causes rpm to error
during installation.
2) When creating the directories to use for the instance, if the directory
already exists, we should chown it anyway, just to make sure it has the correct
ownerships and permissions.
Created attachment 147685 [details]
new diffs
Looks good to me. Looks good. Created attachment 147690 [details]
cvs commit log
Reviewed by: nhosoi (Thanks!)
Files: see diff
Branch: HEAD
Fix Description: After much deliberation, we have decided that it is ok that
our dynamic config files are under /etc/fedora-ds/slapd-instance.
So the config_dir will be /etc/fedora-ds/slapd-instance and the security and
schema files will go there as well. Since the FHS is ambiguous about this
issue, and it will be very confusing if the configuration files are not under
/etc, and there are some agents (webmin, cfengine) that do "dynamically" modify
config files under /etc, this outweighs any considerations about having the
server using it's config file like an "ascii database".
In addition, the presence of repl-monitor-cgi causes rpm to complain, and since
we only support CGIs in the Admin Server, this file has been removed from the
core fedora-ds package.
Platforms tested: RHEL4, FC6
Flag Day: no
Doc impact: no
Created attachment 147916 [details]
diffs to add instconfigdir
Looks good to me, too. When checking in, could you please make sure it builds with the spec file fedora-ds/<platform>/fedora-ds.spec for the nightly build? Thanks. Created attachment 147933 [details]
cvs commit log
Reviewed by: nkinder, nhosoi, prowley (Thanks!)
Files: see diff
Branch: HEAD
Fix Description: I've added a new configure switch: --with-instconfigdir. This
switch will allow the user to specify a different location to store the dynamic
instance specific config files rather than the default
$sysconfdir/$package_name (e.g. /etc/fedora-ds). This is the directory which
will contain the slapd-instance directories which contain the instance specific
config, schema, and security files. Even though the user could override this
with ds_newinst.pl ([slapd] section config_dir), we needed to be able to set
the default so that the user would not have to remember to do this every time,
and so that packagers could set a reasonable default value for their platform.
Platforms tested: FC6, RHEL4
Flag Day: no
Doc impact: no
Verified that are config files are under /etc now. We have since changed the path a bit from the path this bug used though. We are using a path of /etc/dirsrv/slapd-foo. Marking as VERIFIED. |