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:50:18 -0500
Cc: pcp <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <55787861.3040109@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> <55787861.3040109@xxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0
On 06/10/2015 12:48 PM, David Smith wrote:
> 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.
> ====

Ah, looks like you already found this one.

-- 
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>