Code review comment for lp:~pundiramit/lava-android-test/dalvikvm

Revision history for this message
Amit Pundir (pundiramit) wrote :

On 8 April 2013 19:58, Antonio Terceiro <email address hidden> wrote:

> On Mon, Apr 08, 2013 at 09:58:23AM -0000, Amit Pundir wrote:
> > On 3 April 2013 08:01, Antonio Terceiro <email address hidden>
> wrote:
> >
> > > On Tue, Apr 02, 2013 at 09:07:26AM -0000, Amit Pundir wrote:
> > > > I did not know how to use lava-test-shell into LAVA_TEST_PLAN back
> then
> > > so I pushed the test in both formats. I learned how to use that from
> > > Yongqin yesterday.
> > > >
> > > > Antonio, It is OK if you skip this one.
> > >
> > > This was already merged, but I think we should remove it. Can you
> please
> > > confirm that you guys are not submitting jobs that rely on having this
> > > in lava-android-test?
> > >
> >
> > Hi Antonio,
> >
> > I'm not yet able to test run this dalvik test as a lava-test-shell on
> LAVA
> > production server
> >
> https://android-build.linaro.org/builds/~linaro-android-restricted/pundiramit-test-0/
> .
> > For the time being I suggest we keep it in lava-android-test as well.
> I'll
> > let you know if when to remove this test from lava-android-test. Sounds
> > good?
>
> OK. Can you share what is the problem you have with running it with
> lava-test-shell?
>

So LAVA dashboard for a job running dalvik test as lava-android-test
https://validation.linaro.org/lava-server/dashboard/streams/private/team/linaro/android-daily/bundles/aded4ec3fb533f5982a68cab2c7731a2e73fbb86/c0ef53ec-b9a9-44a6-bdc0-1ebe2c4b20d5/
clearly
indicates the PASS/FAIL status of the test but a similar job to run dalvik
test through lava-test-shell doesn't show PASS/FAIL status of the test
https://validation.linaro.org/lava-server/dashboard/streams/private/team/linaro/android-daily/bundles/f281eb99d4eb493b1f2f603f45517f44ffe0600e/

I now see that the lava-test-shell script to run dalvik test doesn't have
execute permissions set. I'll ask Senthil to fix this.

> --
> Antonio Terceiro
> Software Engineer - Linaro
> http://www.linaro.org
>
>
> https://code.launchpad.net/~pundiramit/lava-android-test/dalvikvm/+merge/154872
> You are the owner of lp:~pundiramit/lava-android-test/dalvikvm.
>

« Back to merge proposal