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/