On Thu, Jul 21, 2016 at 9:32 PM, Luis Gomez <[email protected]> wrote:

>
> On Jul 21, 2016, at 8:04 PM, Anil Vishnoi <[email protected]> wrote:
>
> Hi Hideyuki,
>
> This patch looks good to me, but looks like CSIT test is failing on this
> patch. Did you run the CSIT-only-boron /CSIT-only-Beryllium test on these
> patches ?
>
>
> https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-flow-services-only-beryllium/
>
> https://jenkins.opendaylight.org/releng/view/openflowplugin/job/openflowplugin-csit-1node-flow-services-only-boron/
>
> Shiguru/Hideyuki, I looked at the first test CSIT job
> https://jenkins.opendaylight.org/releng/job/openflowplugin-patch-test-boron/12/
> : UNSTABLE (skipped)
>
> And it shows 4 failure for the *-only-boron job (
> openflowplugin-csit-1node-flow-services-only-boron
> <https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node-flow-services-only-boron/>[image:
> Unstable]#518
> <https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node-flow-services-only-boron/518/>)
> and these flows are related to flow/group operational flow. Can you please
> have a look at it and see if these are anyway related to your patch.
>
>
> This test above is for patch:
> https://git.opendaylight.org/gerrit/#/c/40429/ (not 42213)
>
​40429 is cherry-pick to master for 42213, so basically running the same
patch.​


>
> These errors [1] are normal and are due to CPqD not being stable since we
> changes openflowjava to accommodate project requirements last release.
>
> [1]
> https://logs.opendaylight.org/releng/jenkins092/openflowplugin-csit-1node-flow-services-only-boron/518/archives/log.html.gz
>
​okay.​


>
>
> Luis, i think none of us have access to trigger these CSIT job against the
> patch. I think shiguru and others tried to run test-openflowplugin, but
> looks like all the attempt failed (Except the first one) for some reason.
> So if we can get result of above two tests, we can merge the patch.
>
>
> The CI should be better now that we have migrated all csit jobs to public
> cloud, please retry test-openflowplugin and let me know if you have
> questions interpreting any of the results.
>
>
> On Thu, Jul 21, 2016 at 3:37 PM, Tai, Hideyuki <[email protected]>
> wrote:
>
>> Hi committers of the OpenFlow Plugin project,
>>
>> Could you review and merge the following patch for the bug 6073?
>>
>> https://git.opendaylight.org/gerrit/#/c/42213/
>>
>> We would like to get this change in the Beryllium SR3 which is cut off on
>> July 24.
>>
>> Best Regards,
>> Hideyuki Tai
>>
>> _______________________________________________
>> openflowplugin-dev mailing list
>> [email protected]
>> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
>>
>
>
>
> --
> Thanks
> Anil
>
>
>


-- 
Thanks
Anil
_______________________________________________
openflowplugin-dev mailing list
[email protected]
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Reply via email to