pcp
[Top] [All Lists]

Re: [pcp] JSON PMDA with indom cache changes

To: Nathan Scott <nathans@xxxxxxxxxx>
Subject: Re: [pcp] JSON PMDA with indom cache changes
From: David Smith <dsmith@xxxxxxxxxx>
Date: Wed, 10 Jun 2015 12:48:17 -0500
Cc: pcp <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <54437433.12427793.1433465221723.JavaMail.zimbra@xxxxxxxxxx>
References: <555DEF05.7030108@xxxxxxxxxx> <461406362.3814890.1432280874168.JavaMail.zimbra@xxxxxxxxxx> <55677BB7.3060805@xxxxxxxxxx> <1432466455.9615965.1433146915880.JavaMail.zimbra@xxxxxxxxxx> <2105731158.10292085.1433231032998.JavaMail.zimbra@xxxxxxxxxx> <55709952.9070209@xxxxxxxxxx> <54437433.12427793.1433465221723.JavaMail.zimbra@xxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
On 06/04/2015 07:47 PM, Nathan Scott wrote:
> Hi David,
> 
> ----- Original Message -----
>> [...]
>> When you say things are "non-deterministic" do you mean that sometimes
>> you see a pass? I certainly never saw this kind of behavior when I was
>> testing.
> 
> I always see it fail the same way on the 2 hosts I've run it so far.  I say
> "non-deterministic" because its produced different output for you when you
> initially saved the golden output for the test.
> 
> I imagine its to do with different hashing schemes across the different
> versions of python, and not a real problem in the new code.
> 
>> Hmm. Since we have fairly full control over commands here, we could do
>> something like the following (untested) patch. Basically we sort the
>> output of only the 'children json' dbpmda command
>>
> 
> *nod* - this is what Ken's suggesting too.
> 
>>
>> Note there are a couple of other places that would need the same treatment.
>>
> 
> Yeah, I think we'll need to go ahead & do this kind of thing.

On the pcpfans.git dsmith/dev branch, I've done this in the following
commit:

====
commit fc785142078eec77fe40926fb4fa6efd6c1e4ef3
Author: David Smith <dsmith@xxxxxxxxxx>
Date:   Tue Jun 9 12:36:06 2015 -0500

    Make the JSON PMDA qa test more robust.

    * qa/1052: Sort output of certain commands, to avoid problems with
      differing python internal ordering.
    * qa/1052.out: Updated.
====

-- 
David Smith
dsmith@xxxxxxxxxx
Red Hat
http://www.redhat.com
256.217.0141 (direct)
256.837.0057 (fax)

<Prev in Thread] Current Thread [Next in Thread>