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.
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
The changes in comment#6 look fine.
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.