Hi Michael, We have merged one patch https://git.opendaylight.org/gerrit/#/c/69473/ which is moving to the new toString format and will take care of the failing TCs.
I have also raised one more patch https://git.opendaylight.org/gerrit/#/c/69486/ to remove the toString dependency on TCs and assert on values. Regards, Arun From: [email protected] [mailto:[email protected]] On Behalf Of Michael Vorburger Sent: Wednesday, March 14, 2018 3:23 PM To: openflowplugin-dev <[email protected]> Cc: [email protected]; release ([email protected]) <[email protected]> Subject: Re: [openflowplugin-dev] [release] Autorelease fluorine failed to build openflowplugin from openflowplugin Hello openflowpluguineans, and just FYI mdsal-dev: On Wed, Mar 14, 2018 at 2:42 AM, Jenkins <[email protected]<mailto:[email protected]>> wrote: Attention openflowplugin-devs, Autorelease fluorine failed to build openflowplugin from openflowplugin in build 16. Attached is a snippet of the error message related to the failure that we were able to automatically parse as well as console logs. Console Logs: https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/autorelease-release-fluorine/16 Jenkins Build: https://jenkins.opendaylight.org/releng/job/autorelease-release-fluorine/16/ Please review and provide an ETA on when a fix will be available. Thanks, ODL releng/autorelease team Failed tests: PacketOutConvertorTest.toPacketOutInputAllParmTest:176 expected:<PortNumber[ [_value=2]]> but was:<PortNumber[{_value=2}]> this looks like yet another impact of some toString() change which must have recently gone into mdsal... we dealt with something like this in genius a few days ago, and this may help you to fix it in the same way: https://git.opendaylight.org/gerrit/#/c/69375/1/itm/itm-impl/src/test/java/org/opendaylight/genius/itm/tests/xtend/ExpectedExternalTunnelObjects.xtend https://lists.opendaylight.org/pipermail/genius-dev/2018-March/002465.html but if I were you ideally I would not just fix the String but change the test to not rely on toString but to get that value (and perhaps and instanceof, if there are other things than PortNumber possible)? I'm happy to review such a change if you like. Tx, M. -- Michael Vorburger, Red Hat [email protected]<mailto:[email protected]> | IRC: vorburger @freenode | ~ = http://vorburger.ch<http://vorburger.ch/>
_______________________________________________ openflowplugin-dev mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev
