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]] Sent: Thursday, July 21, 2016 20:04 To: Tai, Hideyuki <[email protected]<mailto:[email protected]>>; Luis Gomez Palacios <[email protected]<mailto:[email protected]>> Cc: [email protected]<mailto:[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 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]<mailto:[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]<mailto:[email protected]> https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev -- Thanks Anil
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
