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/TSDR-52 tsdr <p>In NodeConnectorStatisticsChangeHandler a TDRMetricRecord list is built with direct array[] access from the FlowCapableNodeConnectorStatistics supplied.  In some cases, a FlowCapableNodeConnectorStatistics instance is supplied without bytes and packet metrics.  This fix enables building the TSDRMetricRecord with or without the bytes and packet metrics.  It was failing when FlowCapableNodeConnectorStatistics did not supply the bytes and packet metrics.</p> <p>If this effects downstream processing, then that should probably be fixed to accomodate this change.  The failing code was padding the TSDRMetricsRecord with a timestamp and no data.  If downstream needs this then we should probably fix that as there is no data associated with the timestamp.</p> <p> </p> <p> </p> <p> </p> TSDR-52

Accept metrics list when bytes and packet metrics are missing

Bug Medium Resolved Done Scott Melton Scott Melton Sat, 14 Apr 2018 02:33:55 +0000 Mon, 16 Apr 2018 02:35:27 +0000 Sat, 14 Apr 2018 02:39:09 +0000 0 2 0 minutes 0 minutes 1 hour <p>Fix was to properly build the TSDRMetricsRecord list when bytes and packet metrics are missing from the supplied FlowCapableNodeConnectorStatistics instance.  The list builder assumed that if the other met4rics were present then the bytes and packet data was there too.  This may indicate that the FlowCapableNodeConnectorStatistics isn't being built completely when the bytes and packet data is unavailable. </p> <p>Opened <a href="https://jira.opendaylight.org/browse/TSDR-53" title="Investigate FlowCapableNodeConnectorStatistics building" class="issue-link" data-issue-key="TSDR-53"><del>TSDR-53</del></a> to investigate if FlowCapableNodeConnectorStatistics is being created properly when bytes and packet data are missing or to fix all other downstream processing where bytes and packet data are expected, whichever is determined to be the right design.</p> <p>The failing test case below exposes NodeConnectorStatisticsChangeHandler is not checking for missing data returned from FlowCapableNodeConnectorStatistics before building the TSDRMetricsRecord.</p> <p> </p> <p> org.opendaylight.tsdr.datacollection.DataCollectionTest<br/> testNodeConnectorStatisticsChangeHandler(org.opendaylight.tsdr.datacollection.DataCollectionTest)  Time elapsed: 0.022 sec  &lt;&lt;&lt; ERROR!<br/> java.lang.ArrayIndexOutOfBoundsException: 8</p> <p><a href="https://jira.opendaylight.org/secure/ViewProfile.jspa?name=scottmelton" class="user-hover" rel="scottmelton">scottmelton</a> If this test is already fixed in master branch, then should this test also be cherry picked to Carbon since it is still supported? </p> <p>Yes, probably.  I checked in the exact same code that was in master into carbon  under a seperate patch.  Is there a need to cherry pick?  It is the exact code in both master and carbon?  I will apply fixes to master and cherry pick to carbon in the future.  I can redo that here if you think it is necessary?  Thanks for looking after this.</p> Development Rank 0|i03ddz: