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 <[email protected]> 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
0001-Parsing-pattern-was-wrong-so-updated.patch
Description: Binary data
test.py
Description: Binary data
_______________________________________________ linaro-validation mailing list [email protected] http://lists.linaro.org/mailman/listinfo/linaro-validation
