pcp
[Top] [All Lists]

pcp updates: kenj merge, qa, pmpost

To: PCP Mailing List <pcp@xxxxxxxxxxx>
Subject: pcp updates: kenj merge, qa, pmpost
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Wed, 24 Jul 2013 07:21:38 -0400 (EDT)
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <960497723.1259965.1374664843095.JavaMail.root@xxxxxxxxxx>
Reply-to: Nathan Scott <nathans@xxxxxxxxxx>
Thread-index: 9rLRa55HkJyP2deGrjBLzpGZQAgx+Q==
Thread-topic: pcp updates: kenj merge, qa, pmpost
Changes committed to git://oss.sgi.com/pcp/pcp.git dev

 GNUmakefile                    |   11 ++
 debian/pcp.postinst.tail       |   57 +++++++++---
 man/man1/pmatop.1              |   10 +-
 man/man1/pmcd.1                |    2 
 man/man1/pmdagluster.1         |    6 -
 man/man1/pmpost.1              |   31 ++----
 man/man1/pmstat.1              |    2 
 qa/115                         |   16 ++-
 qa/262                         |    6 -
 qa/262.linux                   |   16 +--
 qa/300                         |    5 -
 qa/504                         |   18 ++--
 qa/994                         |   88 +++++++++++++++++++
 qa/994.out                     |    1 
 qa/common.filter               |    3 
 qa/group                       |    1 
 qa/src/GNUlocaldefs            |    4 
 qa/src/mkpermslist             |  183 +++++++++++++++++++++++++++++++++++++++++
 qa/src/permslist               |   14 +++
 src/libpcp/src/secureconnect.c |    5 -
 src/pmcd/GNUmakefile           |    2 
 src/pmcd/rc_pmcd               |    8 +
 src/pmdas/mysql/GNUmakefile    |    2 
 src/pmie/GNUmakefile           |    9 --
 src/pmie/rc_pmie               |   20 +++-
 src/pmlogger/GNUmakefile       |    6 -
 src/pmlogger/pmlogger_check.sh |    4 
 src/pmlogger/pmlogger_daily.sh |   11 ++
 src/pmpost/pmpost.c            |   39 ++++++++
 src/pmproxy/GNUmakefile        |    2 
 src/pmwebapi/GNUmakefile       |    2 
 31 files changed, 488 insertions(+), 96 deletions(-)

commit faa61c3d9672574d4a24d535e1356c8615d4b982
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Wed Jul 24 21:19:02 2013 +1000

    Install the PMDA cache dir as part of the packaging
    
    Some QA tests recently started to notice a missing cache
    directory for persistent indom files.  Like other critical
    PCP directories, we should be installing this and setting
    permissions appropriately.
    
    Also fixed a recently added typo in the toplevel makefile.

commit c85b6654ff19c8d57ecaca1bee081db3b23f1309
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Wed Jul 24 21:15:47 2013 +1000

    Really fix qa/300 this time - was assuming a host containing "bozo"
    
    After an earlier attempt to fix this from kenj, when something must
    have switched from host bozo to bozo-laptop, test qa/300 now passes
    on any host.  Had me *very* confused for awhile since the test also
    creates a bozo PMDA - a nice little red herring!
    
    Create a seq.full file now, as I needed it to debug this (although
    it was obvious in hindsight, still a good idea to keep this output).

commit 7eeb89d6ef2b13f8b7c35695d1119f1433b8df10
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Wed Jul 24 21:12:06 2013 +1000

    Fix pmpost for use by the unprivileged pcp account
    
    As pmpost may nowadays be invoked by pmie running as "pcp",
    we need to do a better job of creating the noticeboard file
    in terms of filesystem permissions.  This change ensures it
    is created owned by root, and writable by group "pcp" (iow,
    $PCP_GROUP).
    
    Long-gone are the days when pmpost ran setuid root, so the
    man page is also updated to remove those references and to
    document this new behaviour.

commit 383b4657d8e68ad9d8a135a3b7bfdbf678df0ad5
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Wed Jul 24 19:57:10 2013 +1000

    Handle SASL mechanisms passing back debug statements
    
    Have observed for the first time now some SASL mechanisms
    passing back debug/trace statements, in the regular set of
    exchanges.  If we do not have tracing enabled (pmDebug and
    DBG_TRACE_AUTH), we must suppress these messages.

commit e8a0447c7adb6819c9fa2291e3a0426babcaf933
Merge: 2d71fb2 f5a283a
Author: Nathan Scott <nathans@xxxxxxxxxx>
Date:   Wed Jul 24 17:38:40 2013 +1000

    Merge branch 'dev' of git://oss.sgi.com/kenj/pcp into dev

commit f5a283ac64a748916aea7ea9cc41f53558b80db7
Merge: 9913a42 5e80530
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 12:18:05 2013 +1000

    Merge branch 'dev' of git://oss.sgi.com/pcp/pcp into dev
    
    Conflicts:
        qa/group
    
    Just collision in line ordering within the file.

commit 9913a427ce2c9a60150007cb74b07681ede6815f
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 10:14:03 2013 +1000

    qa/common.filter - handle changed waiting...pmie msg from rc script

commit 2b420834fa508964bdd46a0bbd3303e5f0a4488a
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 10:12:08 2013 +1000

    qa/262 - cleanup
    
    - remove white space dependency
    - handle "units" after numbers from pmstat
    - remove proc PMDA dependency

commit 225bc166f87c1e4c5115ee8d6b5e65f0ba7e1992
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 09:55:37 2013 +1000

    pmie rc script - Waiting ... PMIE -> Waiting ... pmie
    
    Lowercase pmie, just for consistency with the other rc scripts.

commit 399857b6a06c7a8fbadbc7e089da01747dec7378
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 09:54:30 2013 +1000

    qa/504 - $PCP_PMIECONTROL_PATH care and feeding
    
    1. don't create it, die if it does not exist
    2. restore mode, owner and group once you're done

commit cfeddf0f15341e8990e1dd4ba5af6bea8f049bb8
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 09:53:03 2013 +1000

    qa/115 - $PCP_PMIECONTROL_PATH care and feeding
    
    1. don't create it, die if it does not exist
    2. restore mode, owner and group once you're done

commit c55eee4bbf1ccc9a10aa2ca2921aef24c652f155
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 09:50:59 2013 +1000

    qa/994 (new) - check non-default artifacts from packages
    
    Compares mode, owner and group between makefiles and installed
    systems ... see src/permslist for makefile inventory and
    src/mkpermslist is the script for recreating thus.

commit 5d99e3073d9c868eaa13730815ee8a90f36db166
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 09:40:17 2013 +1000

    Non-default permissions artifacts and Debian packaging
    
    This commit tries to enforce consistency between the build,
    the Debian packages and the PCP run-time.
    
    In general terms, if the directory mode is 775 or the file mode is
    664, then we expect the ownership to be pcp:pcp.  There are some
    execeptions but these are noted with large comments.  To enforce
    this the makefiles need to explicitly set the mode, owner and
    group for the non-default artifacts.  The Debian pcp.postinst
    script needs to do _exactly_ the same thing as the settings from
    the makefile are stripped by dh_fixperms in the build.  And at
    run time we sometimes have to create or recreate things on the fly
    and the owner, group and mode here have to play by the same rules.
    
    Finally when we smack these files in qa, we need to be sure
    to restore the owner, group and mode to match the makefiles.
    The new qa/994 checks a lot of this.  The first batch of QA
    changes are in the next commit.
    
    If this survives Debian QA stress testing, I'll be making the same
    (philosophical) changes for the other packaging options we support,
    one by one.

commit bf406b2df6e9b4cedab8ee25fe14b73d9a108628
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 09:08:35 2013 +1000

    assorted man page typo fixes (thanks lintian)

commit a2bd6d4f5ffd5459afe10667c46257ec5e791489
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Wed Jul 24 08:52:40 2013 +1000

    pmdagluster - man page typo fix (thanks lintian)

commit 4cd5b87342126c7c8f7b164c873e9612fa98bd5c
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Tue Jul 23 21:26:05 2013 +1000

    pmie - packaging changes
    
    Cleanup the makefile for the non-default permissions artifacts and
    make the Debian packaging match the makefile and make the run-time
    (optional) directory creation match the makefile.

commit 0d831264f1ccf986e15a70beea871919c592b03a
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Tue Jul 23 16:37:11 2013 +1000

    pmlogrewrite config for mysql pmda ...
    
    oops, forgot to add migrate.conf to $LSRCFILES so the builds will work.

commit 375c5fd612744158b63285434c9b86d14af89304
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date:   Tue Jul 23 16:32:37 2013 +1000

    $PCP_RUN_DIR - packaging changes
    
    For most packages, $PCP_RUN_DIR is included in the pcp package,
    but for Debian and cases where /var/run is a mounted filesystem
    it may not exist, so in all of the associated run-time places
    we need to conditionall create or re-create it (with the same mode,
    owner and group as in the packaged case) before it is used.

<Prev in Thread] Current Thread [Next in Thread>
  • pcp updates: kenj merge, qa, pmpost, Nathan Scott <=