https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/BGPCEP-700 bgpcep <p><a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/14/odl1_karaf.log.gz" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/14/odl1_karaf.log.gz</a></p> <p>2017-10-12 09:58:26,502 | ERROR | ntLoopGroup-13-8 | BGPDocumentedException | 271 - org.opendaylight.bgpcep.bgp-parser-api - 0.9.0.SNAPSHOT | Error = WELL_KNOWN_ATTR_MISSING<br/> org.opendaylight.protocol.bgp.parser.BGPDocumentedException: Well known mandatory attribute missing: ORIGIN<br/> at org.opendaylight.protocol.bgp.parser.impl.message.BGPUpdateMessageParser.checkMandatoryAttributesPresence(BGPUpdateMessageParser.java:173)<span class="error">&#91;272:org.opendaylight.bgpcep.bgp-parser-impl:0.9.0.SNAPSHOT&#93;</span><br/> at org.opendaylight.protocol.bgp.parser.impl.message.BGPUpdateMessageParser.parseMessageBody(BGPUpdateMessageParser.java:146)<span class="error">&#91;272:org.opendaylight.bgpcep.bgp-parser-impl:0.9.0.SNAPSHOT&#93;</span><br/> at org.opendaylight.protocol.bgp.parser.impl.message.BGPUpdateMessageParser.parseMessageBody(BGPUpdateMessageParser.java:48)<span class="error">&#91;272:org.opendaylight.bgpcep.bgp-parser-impl:0.9.0.SNAPSHOT&#93;</span><br/> at org.opendaylight.protocol.bgp.parser.spi.pojo.SimpleMessageRegistry.parseBody(SimpleMessageRegistry.java:31)<span class="error">&#91;273:org.opendaylight.bgpcep.bgp-parser-spi:0.9.0.SNAPSHOT&#93;</span><br/> at org.opendaylight.protocol.bgp.parser.spi.AbstractMessageRegistry.parseMessage(AbstractMessageRegistry.java:63)<span class="error">&#91;273:org.opendaylight.bgpcep.bgp-parser-spi:0.9.0.SNAPSHOT&#93;</span><br/> at org.opendaylight.protocol.bgp.rib.impl.BGPByteToMessageDecoder.decode(BGPByteToMessageDecoder.java:50)<span class="error">&#91;277:org.opendaylight.bgpcep.bgp-rib-impl:0.9.0.SNAPSHOT&#93;</span><br/> at io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:411)<span class="error">&#91;14:io.netty.codec:4.1.8.Final&#93;</span><br/> at io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:248)<span class="error">&#91;14:io.netty.codec:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:363)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:349)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:341)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:293)<span class="error">&#91;14:io.netty.codec:4.1.8.Final&#93;</span><br/> at io.netty.handler.codec.ByteToMessageDecoder.fireChannelRead(ByteToMessageDecoder.java:280)<span class="error">&#91;14:io.netty.codec:4.1.8.Final&#93;</span><br/> at io.netty.handler.codec.ByteToMessageDecoder.callDecode(ByteToMessageDecoder.java:396)<span class="error">&#91;14:io.netty.codec:4.1.8.Final&#93;</span><br/> at io.netty.handler.codec.ByteToMessageDecoder.channelRead(ByteToMessageDecoder.java:248)<span class="error">&#91;14:io.netty.codec:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:363)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:349)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.fireChannelRead(AbstractChannelHandlerContext.java:341)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.DefaultChannelPipeline$HeadContext.channelRead(DefaultChannelPipeline.java:1334)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:363)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.AbstractChannelHandlerContext.invokeChannelRead(AbstractChannelHandlerContext.java:349)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.DefaultChannelPipeline.fireChannelRead(DefaultChannelPipeline.java:926)<span class="error">&#91;19:io.netty.transport:4.1.8.Final&#93;</span><br/> at io.netty.channel.epoll.AbstractEpollStreamChannel$EpollStreamUnsafe.epollInReady(AbstractEpollStreamChannel.java:1018)<span class="error">&#91;20:io.netty.transport-native-epoll:4.1.8.Final&#93;</span><br/> at io.netty.channel.epoll.EpollEventLoop.processReady(EpollEventLoop.java:394)<span class="error">&#91;20:io.netty.transport-native-epoll:4.1.8.Final&#93;</span><br/> at io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:299)<span class="error">&#91;20:io.netty.transport-native-epoll:4.1.8.Final&#93;</span><br/> at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:858)<span class="error">&#91;16:io.netty.common:4.1.8.Final&#93;</span><br/> at io.netty.util.concurrent.DefaultThreadFactory$DefaultRunnableDecorator.run(DefaultThreadFactory.java:144)<span class="error">&#91;16:io.netty.common:4.1.8.Final&#93;</span><br/> at java.lang.Thread.run(Thread.java:748)<span class="error">&#91;:1.8.0_141&#93;</span></p> <p>Operating System: All<br/> Platform: All</p> BGPCEP-700

Well known mandatory attribute missing: ORIGIN

Bug Verified Done Tomas Markovic Claudio David Gasparini Mon, 16 Oct 2017 07:35:54 +0000 Tue, 20 Jul 2021 10:33:42 +0000 Tue, 27 Mar 2018 05:05:12 +0000 Bugzilla Migration Fluorine Nitrogen Carbon Oxygen BGP 0 5 <p>Hi Claudio, could you please give us the steps to reproduce this issue?</p> <p>thanks in advance,</p> <p>Issue is seen in the test, the link is attached. <br/> Therefore you will need to investigate which part of the test is triggering.<br/> Check if the error if from our side or from the test, etc..</p> <p>Regards,</p> <p>I merged this CSIT patch:</p> <p><a href="https://git.opendaylight.org/gerrit/#/c/64424/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/64424/</a></p> <p>still, seems like we could handle hte BGPDocumentedException a little<br/> cleaner than throwing the whole trace. Can we get a more clear/concise<br/> log.error() message instead?</p> <p>Hi Jamo, Which was the point of the patch merged? I don't see that it fixed anything<span class="error">&#91;0&#93;</span>, <br/> neither I see any attachment/prove that patch was tested against sandbox or any other place. <br/> Commit message is unclear, and it doesn't say what is fixing and how.</p> <p>Regards, <br/> Claudio</p> <p><span class="error">&#91;0&#93;</span> <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/19/odl1_karaf.log.gz" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/19/odl1_karaf.log.gz</a></p> <p>I just saw this bug was updated, and when I looked I realized I never responded. sorry.</p> <blockquote> <p>Hi Jamo, Which was the point of the patch merged? I don't see that it fixed anything<span class="error">&#91;0&#93;</span>, <br/> neither I see any attachment/prove that patch was tested against sandbox or any other place. <br/> Commit message is unclear, and it doesn't say what is fixing and how.</p> <p>Regards, <br/> Claudio</p> <p><span class="error">&#91;0&#93;</span><a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/19/odl1_karaf.log.gz" class="external-link" target="_blank" rel="nofollow noopener">https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/19/odl1_karaf.log.gz</a></p></blockquote> <p>I don't know/remember the point of the patch. Maybe Yrineu can comment? I think I merged it<br/> assuming it was resolve the Exception, but it has not, now that I have checked a <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/38/odl1_karaf.log.gz" class="external-link" target="_blank" rel="nofollow noopener">recent job</a></p> <p>I do suspect the patch was actually helpful though, as you can see the output of the robot<br/> step using the string that was updated in the patch does not have funky <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/38/robot-plugin/log.html.gz#s1-s5-t4-k3-k3" class="external-link" target="_blank" rel="nofollow noopener">characters being returned </a><br/> like <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/19/log.html.gz#s1-s5-t4-k3-k3" class="external-link" target="_blank" rel="nofollow noopener">this one does </a></p> <p>Hey Jamo Luhrsen, when I sent that patch I saw that we was requesting to an incomplete URL, once I changed it, I ran the tests locally and has worked fine. The same when I sent the patch to review.</p> <blockquote><p>Hey Jamo Luhrsen, when I sent that patch I saw that we was requesting to an incomplete URL, once I changed it, I ran the tests locally and has worked fine. The same when I sent the patch to review.</p></blockquote> <p>ok, <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=Yrineu+Rodrigues" class="user-hover" rel="Yrineu Rodrigues">Yrineu Rodrigues</a> Like I said, I think your patch was helpful. But the actual bug this is filed for:</p> <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">BGPDocumentedException: Well known mandatory attribute missing: ORIGIN</pre> </div></div> <p>is still happening.</p> <blockquote> <p>I do suspect the patch was actually helpful though, as you can see the output of the robot<br/> step using the string that was updated in the patch does not have funky <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/19/log.html.gz#s1-s5-t4-k3-k3" class="external-link" target="_blank" rel="nofollow noopener">characters being returned</a><br/> like <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/38/robot-plugin/log.html.gz#s1-s5-t4-k3-k3" class="external-link" target="_blank" rel="nofollow noopener">this one does</a></p></blockquote> <p>Regarding patch <a href="https://git.opendaylight.org/gerrit/#/c/64424/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/64424/</a> it has been reverted since. And if you check the scritp_uri_opt in both of those logs, they are the same, so it has not been fixed by this. I don't think it does anything useful.</p> <p>This ERROR actually comes from completely different test, bgpcep/bgpuser/ibgp_peer_lsp.robot, where we actually send WRONG routes, and expect BGP to reject them.</p> <p>from documentation</p> <blockquote> <p>TC1_Connect_BGP_Peer<br/> <span class="error">&#91;Documentation&#93;</span> Connect BGP peer with advertising the routes without mandatory params like LOC_PREF.</p></blockquote> <blockquote> <p>TC1_Check_Example_Bgp_Rib<br/> <span class="error">&#91;Documentation&#93;</span> Check RIB for not containig linkstate-route(s), because update messages were not good.</p></blockquote> <p> </p> <p>Indeed if we search by <a href="https://git.opendaylight.org/gerrit/#/q/topic:bug/9294+(status:open+OR+status:merged)" class="external-link" target="_blank" rel="nofollow noopener">topic</a>. I see 3 patches merged, one adding, second reverting what it was added few hours ago, and then a <a href="https://git.opendaylight.org/gerrit/#/c/64471/4/csit/suites/bgpcep/tcpmd5user/tcpmd5user.robot" class="external-link" target="_blank" rel="nofollow noopener">third</a> one </p> <p>which is updating other different test and different feature, and with a conversation that was still going on. </p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=jluhrsen" class="user-hover" rel="jluhrsen">jluhrsen</a> would it be possible to create some kind of rule, that some of the formal committers from the project which is been tested, has to review and approve it before tests are update. I can propose it on next TSC meeting if you agree or required.</p> <p>Basically my point is that those test represent the stability of the project, and PTLs cannot be checking everyday if someone changed the test, before </p> <p>validate the results of them. I'm really thankful <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=Yrineu+Rodrigues" class="user-hover" rel="Yrineu Rodrigues">Yrineu Rodrigues</a> for your contribution, but we need to be careful what changes are introduced to test.</p> <p>Otherwise we will ending thinking that we are testing one thing when indeed we are doing something different.</p> <p>Let me know what both of you think about this, and I'll close the bug as resolved later.</p> <p> </p> <p>Regards,, </p> <p> </p> <p> </p> <p>I think we have two separate conversations here.</p> <ol> <li><b>This bug</b><br/> This bug was filed by <a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=cdgasparini" class="user-hover" rel="cdgasparini">cdgasparini</a> and deals with the ugly Exception <div class="code panel" style="border-width: 1px;"><div class="codeContent panelContent"> <pre class="code-java">Error = WELL_KNOWN_ATTR_MISSING org.opendaylight.protocol.bgp.parser.BGPDocumentedException: Well known mandatory attribute missing: ORIGIN</pre> </div></div> <p>That has not been fixed, according to a <a href="https://logs.opendaylight.org/releng/jenkins092/bgpcep-csit-1node-gate-userfeatures-all-oxygen/38/odl1_karaf.log.gz" class="external-link" target="_blank" rel="nofollow noopener">recent job</a><br/> If the Error is expected (negative test case) then my suggestion (you can take it or leave it) <img class="emoticon" src="https://jira.opendaylight.org/images/icons/emoticons/smile.png" height="16" width="16" align="absmiddle" alt="" border="0"/> is to catch that Exception<br/> and print something more meaningful to a non-expert parsing the logs. If it's not expected, then I guess we still have the<br/> bug to fix. I was just chiming in on this bug in good faith. This project and features are totally out of my realm of expertise.</p></li> <li><b>CSIT changes</b><br/> If the problem we ended up with here is happening a lot, or even just every now and again, we can discuss<br/> a better way to ensure it happens less. We can discuss creating some formal rules, but let's make sure the<br/> problem is worth the overhead. However, if BGPCEP would like to have additional review coverage from the<br/> project's committers, we can try to implement that by culture. I can ask all int/test committers to make sure<br/> those patches are vetted by a bgpcep committer. In some of the other active projects we have (e.g. netvirt<br/> and genius) there are usually committers paying attention (reviewing and writing) CSIT patches. Even then<br/> we get issues creeping in that we have to clean up.<br/> Shall I request the int/test committers to ensure we get a bgpcep committer sign-off on those project's <br/> related CSIT patches?<br/> BTW, here is <a href="https://git.opendaylight.org/gerrit/c/68288/" class="external-link" target="_blank" rel="nofollow noopener">one </a></li> </ol> <p>Thanks</p> <blockquote> <p>If the Error is expected (negative test case) then my suggestion (you can take it or leave it) is to catch that Exception<br/> and print something more meaningful to a non-expert parsing the logs.</p></blockquote> <p>It is expected, I will have a look at possible solutions to clarify logs.</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=jluhrsen" class="user-hover" rel="jluhrsen">jluhrsen</a></p> <ol> <li><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=tomas.markovic" class="user-hover" rel="tomas.markovic">tomas.markovic</a> could you take care of this, thanks</li> <li>I agree with you that this will end creating more overhead, mostly on project with not many active committers. There is no need to create an specific rule only for BGPCEP, it was just a suggestion that in my IMHO could be useful  for all projects. I leave it to your expertise. I'm not fully familiar with merge process under int/test. We use to had to provide some bug where keep track, and sandbox link with run,  when Vratko was reviewing our patches. But I guess its not mandatory.</li> </ol> <p>Thank you to both of you for your time and contribution.</p> <p>Regards, </p> <p><a href="https://git.opendaylight.org/gerrit/#/c/70085/" class="external-link" target="_blank" rel="nofollow noopener">https://git.opendaylight.org/gerrit/#/c/70085/</a></p> Development External issue ID 9294 External issue URL Issue Type Rank 0|i02dwn: