pcp updates: nfs4.2 qa and libpcp_web fix

"Ken McDonell" <[email protected]>
Newsgroups gmane.comp.sysutils.pcp
Message-ID <[email protected]>
Changes committed to git://git.pcp.io/kenj/pcp master

Ken McDonell (2):
      src/libpcp_web/json_helpers.c: fix handling of 64-bit values on 32-bit platforms
      qa/232: adjust filter for recently tweaked nfs4.2 server metric indom

 qa/232                            |   12 +++++++++++-
 src/libpcp_web/src/json_helpers.c |    8 ++++----
 2 files changed, 15 insertions(+), 5 deletions(-)

Details ...

commit 044de09b97c2c812a9ddde7d83c04ba0b15f7204
Author: Ken McDonell <[email protected]>
Date:   Thu Nov 17 06:51:35 2016 +1100

    qa/232: adjust filter for recently tweaked nfs4.2 server metric indom

    Client-side done earlier, but these were missed.

    Also added a comment to the test to explain where the mysterious
    but apparently OK numbers 18, 22, 56 and (now) 71 come from in the
    _filter_dodgey_nfs_indom() method.

commit cf98b792f62631c31a8ec7542aa7869307ba2eab
Author: Ken McDonell <[email protected]>
Date:   Thu Nov 17 06:46:47 2016 +1100

    src/libpcp_web/json_helpers.c: fix handling of 64-bit values on 32-bit platforms

    Using long as the data type and calling strtol() won't work.
    Rather use the type __int64_t (to match the type of the .ll field in
    a pmAtomValue) and convert using strtoll().
    Ditto for unsigned 64-bit metrics.

    With these changes, qa/884 passes on both 32-bit and 64-bit platforms.

-=-=-=-=-=-=-=-=-=-=-=-
pcp mailing list
[email protected]
https://groups.io/g/pcp/messages
-=-=-
Groups.io Links:

You receive all messages sent to this group.

View/Reply Online (#14719): https://groups.io/g/pcp/message/14719
View All Messages In Topic (1): https://groups.io/g/pcp/topic/3146624
Mute This Topic: https://groups.io/mt/3146624?uid=174580
New Topic: https://groups.io/g/pcp/post

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
-=-=-=-=-=-=-=-=-=-=-=-
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.