Re: Add Install/Remove Command Line Switches

----- Original Message -----
> On 20/02/17 22:13, Marko Myllynen wrote:
> > Hi,
> >
> > What do you think about this patch? Every now and then someone asks for
> > command line switches to select the proper mode (*) for PMDAs, let's
> > add them so that people won't need spend time discovering "echo c |
> > ./Install" :) Also add standard --help/--version switches while at it.
> >
> > *) While the patch allows defining the proper mode for a PMDA, reading
> > the man pages, the Programmer's Guide, and the code in pmdaproc.sh
> > leaves me still unclear what/how this PMDA monitor mode is supposed to
> > be / work. PCPIntro(1) and others describe monitors being tools like
> > pmlogger, pmrep, pmval, etc. but don't explain how a PMDA could work in
> > such mode. Clarifications on man pages would be much appreciated.
> 
> monitor mode comes from a time before the "distributed PMNS" when you
> needed to install a local PMNS to make metrics available for clients
> (aka monitors).  For a long time now, the PMNS has been delivered via
> pmcd or archives, so I cannot think of a reason why anyone would need or
> want to choose "monitor" mode.

It also handles pmchart (monitor) config file installs, for some PMDAs.
But these could be moved to always-installed (or dropped entirely, in
some cases):

$ find src -name \*.pmchart
src/pmdas/apache/Apache.pmchart
src/pmdas/cisco/Cisco.pmchart
src/pmdas/shping/shping.CPUTime.pmchart
src/pmdas/shping/shping.RealTime.pmchart
src/pmdas/sample/Sample.pmchart
src/pmdas/weblog/Web.Alarms.pmchart
src/pmdas/weblog/Web.Allservers.pmchart
src/pmdas/weblog/Web.Perserver.Bytes.pmchart
src/pmdas/weblog/Web.Perserver.Requests.pmchart
src/pmdas/weblog/Web.Requests.pmchart
src/pmdas/weblog/Web.Volume.pmchart
src/pmdas/sendmail/Sendmail.pmchart

> Since there are only a handful of people on the planet that remember
> these early days of PCP, I think we could safely drop the whole thing
> ... this would cut some code out of pmdaproc.sh, remove the confusion
> and trigger a massive QA re-base (almost 100 tests would need to be
> reviewed and some changed and all the .out files remade), but that would
> be a once-off effort.

+1

cheers.

--
Nathan

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links:

You receive all messages sent to this group.

View/Reply Online (#15231): https://groups.io/g/pcp/message/15231
View All Messages In Topic (3): https://groups.io/g/pcp/topic/4404906
Mute This Topic: https://groups.io/mt/4404906?uid=174580
New Topic: https://groups.io/g/pcp/post
-=-=-
pcp mailing list
[email protected]
https://groups.io/g/pcp/messages
-=-=-
Change Your Subscription: https://groups.io/g/pcp/editsub?uid=174580
Group Home: https://groups.io/g/pcp
Contact Group Owner: [email protected]
Terms of Service: https://groups.io/static/tos
Unsubscribe: https://groups.io/g/pcp/leave/354243/563757577/xyzzy
-=-=-=-=-=-=-=-=-=-=-=-