Hello Patryk,

I partially reverted the change to allow to use both upper and lower cases: https://git.lavasoftware.org/lava/lava/merge_requests/588

Should be in the next release (2019.06)


Cheers

Le mar. 30 avr. 2019 à 14:58, Patryk Mungai Ndungu <patryk.mungai-ndungu.kx@renesas.com> a écrit :

Hello Remi, Milosz,

 

Thank you for the explanation. I think it was a useful feature, but its an easy enough fix from the user side as well.

 

Thanks,

Patryk

 

From: Remi Duraffort <remi.duraffort@linaro.org>
Sent: 30 April 2019 09:32
To: Milosz Wasilewski <milosz.wasilewski@linaro.org>
Cc: Patryk Mungai Ndungu <patryk.mungai-ndungu.kx@renesas.com>; lava-users@lists.lavasoftware.org
Subject: Re: [Lava-users] lava-test-case only handles lower case results strings

 

Hello Milosz,

 

 

I don't think, removing the default fixup dict was needed. Maybe we can reintroduce it if some users needs it.

 

 

Cheers

 

Le mar. 30 avr. 2019 à 10:26, Milosz Wasilewski <milosz.wasilewski@linaro.org> a écrit :

On Tue, 30 Apr 2019 at 09:19, Remi Duraffort <remi.duraffort@linaro.org> wrote:
>
> Hello,
>
> I'm surprised by such change as the corresponding source code hasn't been touched for more than a year.
> Anyway, I can confirmed that the result strings should be lower case "pass", "fail", "skip" or "unknown".

Isn't this caused by removal of default 'fixup' dictionary? I recall
talking about that recently.

milosz

>
>
> Thanks
>
> Le mer. 24 avr. 2019 à 12:15, Patryk Mungai Ndungu <patryk.mungai-ndungu.kx@renesas.com> a écrit :
>>
>> Hello,
>>
>> I have recently upgraded from 2018.11 to 2019.03 and have noticed that the results of a lot of the tests I have been running no longer got parsed correctly by LAVA. This was because I was sending the results using upper case results strings.
>> Eg. lava-test-case <test-case> PASS     opposed to        lava-test-case <test-case> pass
>>
>> This resulted in the following logs in the lava job:
>> Received signal: <TESTCASE> TEST_CASE_ID=ETH_T_001 RESULT=PASS
>> Bad test result: PASS
>>
>> Changing my results parsing script to only send lower case results strings fixed the issue, but was this restriction intended with the upgrade?
>>
>> Kind Regards,
>> Patryk
>>
>> _______________________________________________
>> Lava-users mailing list
>> Lava-users@lists.lavasoftware.org
>> https://lists.lavasoftware.org/mailman/listinfo/lava-users
>
>
>
> --
> Rémi Duraffort
> LAVA Team, Linaro
> _______________________________________________
> Lava-users mailing list
> Lava-users@lists.lavasoftware.org
> https://lists.lavasoftware.org/mailman/listinfo/lava-users


 

--

Rémi Duraffort

LAVA Team, Linaro



--
Rémi Duraffort
LAVA Team, Linaro