I was looking for following job and i think it looks good. https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node-flow-services-only-beryllium/
I merged both the patches. Thank for the patch Shigeru/Hideyuki and to be patient with the delays in reviews. Thanks Anil On Fri, Jul 22, 2016 at 7:03 PM, Tai, Hideyuki <[email protected]> wrote: > Hi Anil and other OFP project committers, > > > > Well, the patch for the stable/beryllium branch is also ready to be merged. > > > > https://git.opendaylight.org/gerrit/#/c/42213/ (stable/beryllium) > > > > I would like you to merge the patch before cutting off a release candidate > for the Beryllium SR3, that is, before Sunday. > > > > > > I kicked the CSIT-only-beryllium tests on the patch. > > > https://jenkins.opendaylight.org/releng/job/openflowplugin-patch-test-beryllium/4/ > > There were some failures on the CSIT, but all failures were not related to > the change which the patch (42213) introduces. > > > > Regards, > > Hideyuki Tai > > > > *From:* [email protected] [mailto: > [email protected]] *On Behalf Of *Tai, > Hideyuki > *Sent:* Friday, July 22, 2016 17:45 > *To:* Anil Vishnoi <[email protected]>; > [email protected] > *Cc:* Shigeru Yasuda <[email protected]> > > *Subject:* Re: [openflowplugin-dev] Requests for reviewing and merging > patches for Be SR3 > > > > Hi Anil and other OFP project committers, > > > > I think the patch for the master branch is read to be merged. > > Could you review and merge the following patch? > > > > https://git.opendaylight.org/gerrit/#/c/40429/ (For the master) > > > > (In the meantime, I'm checking the result of the CSIT-only-beryllium tests > on the patch <Gerrit 42213> for the stable/beryllium.) > > > > > > There are two points for Anil's requests regarding to the CSIT-only-boron. > > > > First, I've checked the result of the openflowplugin-patch-test-boron #15 > for the patch (40429) for the master branch. > > > https://jenkins.opendaylight.org/releng/job/openflowplugin-patch-test-boron/15/ > > > > There are 5 unsuccessful subproject builds for the test among the 11 > subprojects, and I've checked all the unsuccessful builds. > > And, I've confirmed that all failures on the unsuccessful builds are not > related to the patch (40429). > > > > > > Second, this is about the 4 failures on the > openflowplugin-csit-1node-flow-services-only-boron #518. > > > > These 4 tests sometimes fails even without the patch (40429). > > And, these tests didn't fail on the 541 build which also tested the patch > (40429). > > > https://jenkins.opendaylight.org/releng/job/openflowplugin-csit-1node-flow-services-only-boron/541/ > > > > These 4 tests checked operational DS of the opendaylight-inventory. > > Just reading the information in the DS failed. > > The REST API for that returned 404 where 200 was expected. > > That failures happened even before adding a group and a flow where the > patch (40429) brings changes. > > > > So I think the 4 failures on the #518 were not caused by the patch (40429). > > > > Best Regards, > > Hideyuki Tai > > > > *From:* Anil Vishnoi [mailto:[email protected] <[email protected]>] > > *Sent:* Thursday, July 21, 2016 20:04 > *To:* Tai, Hideyuki <[email protected]>; Luis Gomez Palacios < > [email protected]> > *Cc:* [email protected] > *Subject:* Re: [openflowplugin-dev] Requests for reviewing and merging > patches for Be SR3 > > > > 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: > Image removed by sender. 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. > > > > 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. > > > > 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
