Code Review / controller.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 8c9a1cf) | patch
Fix bug in Static Route configuration path 83/883/2
authorAlessandro Boch <aboch@cisco.com>
Thu, 15 Aug 2013 16:36:38 +0000 (09:36 -0700)
committerGerrit Code Review <gerrit@opendaylight.org>
Sat, 17 Aug 2013 04:12:40 +0000 (04:12 +0000)
commit53bf3293a370a093faa114402bcae1f8938c28c7
tree2c89002f4e49844eac2006a6d6771d5102c98dc3tree | snapshot
parent8c9a1cf398348546e6e5ba74a2012ed942a2fddbcommit | diff
Fix bug in Static Route configuration path

ISSUE: Multiple equivalent static route configurations with different names not detected
CHANGE: In addStaticRoute() perform proper equality check on existing StaticRoute objects
        (using available StaticRoute compareTo() method) instead of checking on StaticRouteConfig
        objects which can never be positive as same name config is detected at the beginning of method

Change-Id: I51d2e1a641f612f5e79eb11063a868739a275209
Signed-off-by: Alessandro Boch <aboch@cisco.com>
opendaylight/forwarding/staticrouting/src/main/java/org/opendaylight/controller/forwarding/staticrouting/internal/StaticRoutingImplementation.java diff | blob | history
opendaylight/forwarding/staticrouting/src/test/java/org/opendaylight/controller/forwarding/staticrouting/StaticRouteTest.java diff | blob | history
OpenDaylight controller