Issue 3460 - Adding LDAP statistics script to contrib directory
Summary: Adding LDAP statistics script to contrib directory
Status: VERIFIED SUSPENDED
Alias: None
Product: OpenLDAP
Classification: Unclassified
Component: contrib (show other issues)
Version: unspecified
Hardware: All All
: --- normal
Target Milestone: ---
Assignee: OpenLDAP project
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-01-05 20:30 UTC by matty@daemons.net
Modified: 2021-08-03 17:47 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description matty@daemons.net 2005-01-05 20:30:45 UTC
Full_Name: Ryan Matteson
Version: 2.2.0
OS: Solaris
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (198.185.0.144)


Howdy,

I recently developed ldap-stats.pl, and would like to get it included with
future openldap releases. A sample ldap-stats.pl report is available here:

http://daemons.net/~matty/code/ldap-stats.txt

The Perl script that provides this report is available here:

http://daemons.net/~matty/code/ldap-stats.pl

I am hoping this utility will help fellow OpenLDAP admins. On a licensing note,
I am 100% impartial to which license is used.

Thanks for your consideration,
- Ryan


Comment 1 Howard Chu 2005-01-08 14:43:47 UTC
moved from Incoming to Contrib
Comment 2 Kurt Zeilenga 2005-03-15 02:06:46 UTC
Does this script work with 2.3?

What overlap does it have with the statslog script included in 2.3?

I recommend you change the license to OpenLDAP Public License.
This will allow your code to be used without significant
restriction in other contributions to the OpenLDAP Project.

Thanks, Kurt
Comment 3 Kurt Zeilenga 2005-03-15 02:07:47 UTC
changed notes
changed state Open to Feedback
Comment 4 matty@daemons.net 2005-04-08 16:32:50 UTC
Hi Kurt,

The statslog script seems to summarize the logfiles, but doesn't provide 
meaningful statistics (e.g., number of binds, number of connections, 
number of unindexed searches). You can view a sample ldap-stats.pl report 
here:

http://daemons.net/~matty/code/ldap-stats.txt

I updated the license to match the OpenLDAP license, and from what I can 
tell, it seems to work ok with 2.3beta.

Thanks for the feedback,
- Matty


From: Kurt Zeilenga <openldap-its@OpenLDAP.org>
To: matty@daemons.net
Subject: Re: Adding LDAP statistics script to contrib directory (ITS#3460)
Date: Tue Mar 15 02:06:46 2005

Does this script work with 2.3?

What overlap does it have with the statslog script included in 2.3?

I recommend you change the license to OpenLDAP Public License.
This will allow your code to be used without significant
restriction in other contributions to the OpenLDAP Project.

Thanks, Kurt

Comment 5 Kurt Zeilenga 2005-09-29 00:08:37 UTC
changed notes
changed state Feedback to Suspended
Comment 6 Kurt Zeilenga 2005-10-02 15:42:20 UTC
changed notes
Comment 7 Gavin Henry 2006-08-04 18:01:51 UTC
The script has now moved to:

 http://prefetch.net/code/ldap-stats.pl

-- 
Kind Regards,

Gavin Henry.
Managing Director.

T +44 (0) 1224 279484
M +44 (0) 7930 323266
F +44 (0) 1224 824887
E ghenry@suretecsystems.com

Open Source. Open Solutions(tm).

http://www.suretecsystems.com/


Comment 8 Quanah Gibson-Mount 2006-11-02 19:42:32 UTC
I'd like to see this script included as well.  There were two concern 
listed in the ITS:

#1) Awaiting new version with proper copyright notice.
#2) Need information about incorporated patches to ensure all who have IPR 
in this
script have made appropriate license grants.

I know #1 has been addressed.

As for #2, I grant such a release for the work I've done on the script.

Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a release 
for the work you've done with this script? :)

--Quanah

--
Quanah Gibson-Mount
Principal Software Developer
ITS/Shared Application Services
Stanford University
GnuPG Public Key: http://www.stanford.edu/~quanah/pgp.html

Comment 9 Todd Lyons 2006-11-02 20:07:04 UTC
On Thu, Nov 02, 2006 at 11:42:32AM -0800, Quanah Gibson-Mount wrote:

>I'd like to see this script included as well.  There were two concern 
>listed in the ITS:
>
>#1) Awaiting new version with proper copyright notice.
>#2) Need information about incorporated patches to ensure all who have IPR 
>in this
>script have made appropriate license grants.
>
>I know #1 has been addressed.
>
>As for #2, I grant such a release for the work I've done on the script.
>
>Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a release 
>for the work you've done with this script? :)

I do, though I don't think my work amounts to much more than one item
in a foreach loop.  But I do.
-- 
Regards...		Todd
when you shoot yourself in the foot, just because you are so neurally
broken that the signal takes years to register in your brain, it does
not mean that your foot does not have a hole in it.      --Randy Bush
Linux kernel 2.6.17-5mdv   3 users,  load average: 0.00, 0.04, 0.13

Comment 10 Peter Schober 2006-11-02 20:45:27 UTC
quanah et al.,

* Quanah Gibson-Mount <quanah@stanford.edu> [2006-11-02 20:42]:
> Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a
> release for the work you've done with this script? :)

go ahead.

actually I though about rewriting it not just once before (it's rather
spaghetti-ish, imho) but too many other things keep popping up.

regards,
-p.schober

-- 
peter.schober@univie.ac.at - vienna university computer center
Universitaetsstrasse 7, A-1010 Wien, Austria/Europe
Tel. +43-1-4277-14155, Fax. +43-1-4277-9140

Comment 11 daveh@coreng.com.au 2006-11-02 21:59:33 UTC
On Thu, 2 Nov 2006, Quanah Gibson-Mount wrote:

> Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a 
> release for the work you've done with this script? :)

Sure; fame at last :-)

-- 
Dave Horsfall  DTM  VK2KFU  daveh@ci.com.au  Ph: +61 2 9552-5509 (d) -5500 (sw)
Corinthian Eng'ng P/L, Ste 54 Jones Bay Whf, 26-32 Pirrama Rd, Pyrmont 2009, AU

Comment 12 Gavin Henry 2006-11-02 23:06:53 UTC
<quote who="quanah@stanford.edu">
> I'd like to see this script included as well.  There were two concern
> listed in the ITS:
>
> #1) Awaiting new version with proper copyright notice.
> #2) Need information about incorporated patches to ensure all who have IPR
> in this
> script have made appropriate license grants.
>
> I know #1 has been addressed.
>
> As for #2, I grant such a release for the work I've done on the script.
>
> Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a release
> for the work you've done with this script? :)

Do you mind if I do some clean up on it?

Mainly Perl::Critic and Perl::Tidy. There's a few tests missing and
assumptions been made with opening files and system commands etc.

I'll have a patch tomorrow.

You can take it or leave it ;-)

Gavin.

http://perlmonks.org/?node_id=386673
http://www.suretecsystems.com/

Comment 13 Quanah Gibson-Mount 2006-11-02 23:13:53 UTC

--On Thursday, November 02, 2006 11:07 PM +0000 ghenry@suretecsystems.com 
wrote:

> <quote who="quanah@stanford.edu">
>> I'd like to see this script included as well.  There were two concern
>> listed in the ITS:
>>
>> # 1) Awaiting new version with proper copyright notice.
>> # 2) Need information about incorporated patches to ensure all who have
>> # IPR
>> in this
>> script have made appropriate license grants.
>>
>> I know #1 has been addressed.
>>
>> As for #2, I grant such a release for the work I've done on the script.
>>
>> Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a release
>> for the work you've done with this script? :)
>
> Do you mind if I do some clean up on it?
>
> Mainly Perl::Critic and Perl::Tidy. There's a few tests missing and
> assumptions been made with opening files and system commands etc.
>
> I'll have a patch tomorrow.
>
> You can take it or leave it ;-)

I don't mind at all, just make sure you get the very latest version. ;)

--Quanah


--
Quanah Gibson-Mount
Principal Software Developer
ITS/Shared Application Services
Stanford University
GnuPG Public Key: http://www.stanford.edu/~quanah/pgp.html

Comment 14 peter@adpm.de 2006-11-03 17:19:15 UTC
Hi,

On Thursday, 2. November 2006 20:42, Quanah Gibson-Mount wrote:
> Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a release
> for the work you've done with this script? :)

Granted !

Peter

-- 
Peter Marschall
peter@adpm.de

Comment 15 daveh@coreng.com.au 2006-11-06 00:06:42 UTC
> Do you mind if I do some clean up on it?
> 
> Mainly Perl::Critic and Perl::Tidy. There's a few tests missing and
> assumptions been made with opening files and system commands etc.

You might also highlight the fact that it seems to require at least 5.8 
(due to the "qw()" lists).

I also don't like hard-wiring "/usr/bin/perl", as the system Perls are 
rarely up to date.  My favourite technique is:

#! /usr/bin/env perl -w

which finds Perl from $PATH.

Or just hard-wire "/usr/local/bin/perl" and let the sysadmin sort it 
out...

-- 
Dave Horsfall  DTM  VK2KFU  daveh@ci.com.au  Ph: +61 2 9552-5509 (d) -5500 (sw)
Corinthian Eng'ng P/L, Ste 54 Jones Bay Whf, 26-32 Pirrama Rd, Pyrmont 2009, AU

Comment 16 Gavin Henry 2006-11-06 07:58:23 UTC
<quote who="daveh@ci.com.au">
>> Do you mind if I do some clean up on it?
>>
>> Mainly Perl::Critic and Perl::Tidy. There's a few tests missing and
>> assumptions been made with opening files and system commands etc.
>
> You might also highlight the fact that it seems to require at least 5.8
> (due to the "qw()" lists).

Well, according to the method I always use to check that,
Perl::MinimumVersion, we get:

 perlver ldap-stats.pl


   --------------------------------------------
 | file          | explicit | syntax | external |
 | -------------------------------------------- |
 | ldap-stats.pl | ~        | v5.6.0 | n/a      |
 | -------------------------------------------- |
 | Minimum explicit version : ~                 |
 | Minimum syntax version   : v5.6.0            |
 | Minimum version of perl  : v5.6.0            |
   --------------------------------------------


>
> I also don't like hard-wiring "/usr/bin/perl", as the system Perls are
> rarely up to date.  My favourite technique is:
>
> #! /usr/bin/env perl -w
>
> which finds Perl from $PATH.
>
> Or just hard-wire "/usr/local/bin/perl" and let the sysadmin sort it
> out...

Sounds good. env method is best I suppose, but there are others as listed at:

http://perldoc.perl.org/perlrun.html#DESCRIPTION

Almost finished that cleanup. It's taken a bit longer than expected.

Thanks.

-- 
Kind Regards,

Gavin Henry.
Managing Director.

T +44 (0) 1224 279484
M +44 (0) 7930 323266
F +44 (0) 1224 824887
E ghenry@suretecsystems.com

Open Source. Open Solutions(tm).

http://www.suretecsystems.com/

Comment 17 frank.swasey@uvm.edu 2006-11-20 19:55:17 UTC
I'm slow reading my email (well 18 days isn't so bad really... ahem...)

On 11/2/06 2:42 PM, quanah@stanford.edu wrote:
> As for #2, I grant such a release for the work I've done on the script.
> 
> Peter S, Dave, Peter M, Frank, and Todd, do you also grant such a release 
> for the work you've done with this script? :)

Yes, I do grant this release.

-- 
Frank Swasey                    | http://www.uvm.edu/~fcs
Sr Systems Administrator        | Always remember: You are UNIQUE,
University of Vermont           |    just like everyone else.
   "I am not young enough to know everything." - Oscar Wilde (1854-1900)
Comment 18 Quanah Gibson-Mount 2007-04-01 23:04:19 UTC
changed notes
Comment 19 Kurt Zeilenga 2007-04-01 23:18:39 UTC
changed notes
Comment 20 OpenLDAP project 2014-08-01 21:03:26 UTC
In discussion
IPR review required