Changes committed to git://git.pcp.io/pcp master
Ken McDonell (6):
qa/1108: add extra diagnostics
qa/admin/pcp-daily: add some more tests to run for one of the
FreeBSD hosts
src/pmlogger/pmnewlog.sh: improve checking
src/pmcd/src/client.c: small change to new client connection logic
qa/119: stop pmmgr at the beginning and restart at the end
src/pmlogger/pmnewlog.sh: fix typo in last commit
qa/1108 | 3 +++
qa/119 | 7 +++++++
qa/admin/pcp-daily | 12 +++++++++---
src/pmcd/src/client.c | 20 +++++++++++++-------
src/pmlogger/pmnewlog.sh | 16 +++++++++++++++-
5 files changed, 47 insertions(+), 11 deletions(-)
Details ...
commit b002ddb7f2e2d7aa3814f321a3eff5d27502da55
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Fri Jul 8 16:27:42 2016 +1000
src/pmlogger/pmnewlog.sh: fix typo in last commit
commit 194af2bcc605bc477407bdfba1bac0fe255e8b83
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Fri Jul 8 16:26:22 2016 +1000
qa/119: stop pmmgr at the beginning and restart at the end
pmlogger processes from pmmgr that may, or may not, be there were
making this test non-deterministic.
commit cd6a6b97672d29f6cde37ff717ee9fe1a20298ab
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Fri Jul 8 10:07:46 2016 +1000
src/pmcd/src/client.c: small change to new client connection logic
In AcceptNewClient(), ECONNABORTED is a (sort of) expected result
from __pmAccept() ... at least it has been seen in QA on FreeBSD
platforms.
Also in the same area, if the error from __pmAccept() is not
expected we should just drop the client connection, not terminate
pmcd as was previously the case.
commit 58bf69753faeeb4109e6a0cbd9098ec3c41e909f
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Fri Jul 8 09:34:57 2016 +1000
src/pmlogger/pmnewlog.sh: improve checking
pmnewlog is supposed to restart _exactly_ one pmlogger process.
If it find more than one "target" processes it should report and quit.
This situation is most unlikely in production environments but has
been seen in QA, with some earlier test probably causing qa/1108 to
fail as a result ... this change should caue qa/1108 to fail with
more useful error messages.
commit 548e435c44c71b27d3d3a0029aef7c94441f88e0
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Fri Jul 8 09:34:22 2016 +1000
qa/admin/pcp-daily: add some more tests to run for one of the
FreeBSD hosts
commit 44b0b1e427807388b71adb0d76e990f1a4bffd38
Author: Ken McDonell <kenj@xxxxxxxxxxxxxxxx>
Date: Fri Jul 8 09:33:53 2016 +1000
qa/1108: add extra diagnostics
|