Bug 245665

Summary:

dsalib still relies on server root

Product:

[Retired] 389

Reporter:

Nathan Kinder <nkinder>

Component:

Admin

Assignee:

Nathan Kinder <nkinder>

Status:

CLOSED CURRENTRELEASE

QA Contact:

Viktor Ashirov <vashirov>

Severity:

low

Docs Contact:

Priority:

low

Version:

1.1.0

CC:

nhosoi, rmeggins

Target Milestone:

---

Target Release:

---

Hardware:

All

OS:

Linux

Whiteboard:

Fixed In Version:

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2015-12-07 16:40:19 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:

240316, 427409

Attachments:

Description	Flags
CVS Diffs - First Pass	none
CVS Commit Message (comment 1 diffs)	none
CVS Diffs - Second Pass	none
CVS Commit Message (comment 3 diffs)	none

Description Nathan Kinder 2007-06-25 22:58:11 UTC

The dsa library in Admin Server still heavily relies on the concept of a server
root.  This library is used by the Directory Server cgi tasks.

The dsa library should try to get paths dynamically from the Directory Server
config when possible.  Other paths that are not in the config, such as the
location of command-line utilities, should be passed in as a define at
compilation time.

Comment 1 Nathan Kinder 2007-06-25 23:07:25 UTC

Created attachment 157833 [details]
CVS Diffs - First Pass

This is a first pass at cleaning up the paths in dsalib.  These changes make
the path getter functions use the Directory Server config values where
possible.

Comment 2 Nathan Kinder 2007-06-25 23:24:33 UTC

Created attachment 157834 [details]
CVS Commit Message (comment 1 diffs)

Checked in changes from comment 1.

Comment 3 Nathan Kinder 2007-06-26 17:51:14 UTC

Created attachment 157923 [details]
CVS Diffs - Second Pass

This pass eliminated the usage of a "server root" in the dsa library.

I also refactored some of the code that was referring to the "install root" as
it did not seem to really represent the directory that it refers to.  I renamed
it to "instance dir" and adjusted the getter function and variables
accordingly.  I needed to pass in the DS libdir as a define for being able to
find the instance dir in the code.

I also found that I was incorrectly using the lockdir config attribute to set
the rundir, which was causing problems for the start, stop, and restart CGIs. 
We don't have the rundir in cn=config by default, but we already have it set as
a PIDDIR define, so I opted to use that instead.  The start, stop, and restart
CGIs now work properly.

Comment 4 Nathan Kinder 2007-06-26 18:36:22 UTC

Created attachment 157931 [details]
CVS Commit Message (comment 3 diffs)

Checked in changes from comment 3.

Comment 5 Yi Zhang 2007-11-29 01:08:23 UTC

1. Based on discussion with Noriko. Since DS80 doesn't have the concept of
"ServerRoot". This bug is more relate to code clean up.

2. As in the past we have many cig script relate testing, and there is no
failure relate to "ServerRoot", this bug can be consider as verified.

Comment 6 Yi Zhang 2007-11-29 01:15:12 UTC

All Console functionalities depend on this lib. And console automation test will
rerun next week. If there are any broken function, I will file a new bug. For
now, this bug can be closed.