Hi Senthil,

Please find the patch for the 0xbench.yaml with updated parsing pattern, that get passed.

yaml is placed @ http://staging.people.linaro.org/~harigopal.gollamudi/0xbench.yaml and tested this in validation server @
http://validation.linaro.org/lava-server/scheduler/job/50358

and also verified the pattern with the output, refer the attached sample script file 'test.py'

Thanks and Regards,
HariGopal G


On 26 March 2013 19:49, Senthil Kumaran <senthil.kumaran@linaro.org> wrote:
Hi Hari,

On Sat, 2013-03-23 at 00:52 +0530, Harigopal Gollamudi wrote:
> please find the attached patch. Kindly review this and update me

I went through your new patch. What I found was, the new pattern you
have put in does not relate to the old parse pattern!

-    pattern: "^(?P<test_case_id>*?)\\s+(?P<measurement>[\\d.]+)\\s
+(?P<units>\\S+)\\s+(?P<result>(pass|fail))\\s*$"
+    pattern: "^(?P<test_case_id>\\S*\\s*\\S*)\\s+(?P<measurement>\\d
+\.\\d+)\\s+(?P<units>\\S+)\\s+(?P<result>\\S+)$"

The new change is not the equivalent of the old and it is not correct (I
am concerned about the regex here). I request you to fix the above and
run a sample job in validation.linaro.org with the 0xbench yaml file.
Please include link to the sample run in your next email.

Thank You.
--
Senthil Kumaran
http://www.stylesen.org/
http://www.sasenthilkumaran.com/




--
Thanks and Regards,
HariGopal G
Android Engineer