[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#4770) monitoringslapd.sdf patch

ghenry@suretecsystems.com wrote:
> Full_Name: Gavin Henry
> Version: 2.3.30
> OS: FC6
> URL: http://www.suretecsystems.com/our_docs/admin-guide-monitoring.patch
> Submission from: (NULL) (
> Hi all,
> Monitoring section updated. Please review and provide feedback.
> Thanks,
> Gavin.
> (http://www.suretecsystems.com/our_docs/admin-guide-monitoring.patch)
At the moment the only comment I have is regarding back-monitor itself, 
not the doc: I question the decision to define the monitor info 
attributes as operational instead of user attributes. Requiring the use 
of "+" to return all of the monitored info seems pretty unfriendly, 
particularly since it causes the return of actual operational attributes 
that are completely irrelevant to the purpose of monitoring. I.e., the 
monitor info should be considered to be dynamically generated user 
attributes, not operational attributes.

   -- Howard Chu
   Chief Architect, Symas Corp.  http://www.symas.com
   Director, Highland Sun        http://highlandsun.com/hyc
   OpenLDAP Core Team            http://www.openldap.org/project/