Bug 2862 - flow ID is unknown value in operational store. (e.g. id: #UF*200-3) 95/21995/1
authorAnil Vishnoi <vishnoianil@gmail.com>
Fri, 5 Jun 2015 13:08:48 +0000 (18:38 +0530)
committerAnil Vishnoi <vishnoianil@gmail.com>
Fri, 5 Jun 2015 15:46:28 +0000 (15:46 +0000)
commita739f8bc8e265d62cd38e5b85e99f06a95070d66
treec88dc0033aaf987a31e936d5865702005406b3f0
parent12de492caa3d5ab518808520a146a3d33f86aca3
Bug 2862 - flow ID is unknown value in operational store. (e.g. id: #UF*200-3)

Fix for old Helium code base.
This issue only occurs when user don't set cookie value while installing the flow
and that's valid flow.

Cookie is used in flow comparison for the applications using match extensions
in their flow body. As of now openflowplugin don't use match extensions
in flow comparison, that can create a scenario where more then one stored flow
can match to any stats flow, if stored flows differ only by match extension.
Once match extensions are part of flow comparison, we should remove cookie
from flow comparison.

Cookie is an optional field, so user might not set it, but if switch
get flow without cookie value , it will use 0 as a default cookie value
and return cookie=0 when openflowplugin fetch the flow stats from switch.
In this scenario flow comparison will fail. This fix make sure that
if user didn't set cookie value while flow installation, skip the cookie
comparison while augmenting flow stats.

Change-Id: Ia156b70b08569ccd573ba07a549093fa1af3a82f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
applications/statistics-manager/src/main/java/org/opendaylight/openflowplugin/applications/statistics/manager/impl/helper/FlowComparatorFactory.java