Merge branch 'master' of ssh://gerrit.onlab.us:29418/onos-next
Showing
1 changed file
with
2 additions
and
17 deletions
| ... | @@ -197,14 +197,7 @@ public class LinkManager | ... | @@ -197,14 +197,7 @@ public class LinkManager |
| 197 | checkNotNull(linkDescription, LINK_DESC_NULL); | 197 | checkNotNull(linkDescription, LINK_DESC_NULL); |
| 198 | checkValidity(); | 198 | checkValidity(); |
| 199 | 199 | ||
| 200 | - ConnectPoint src = linkDescription.src(); | ||
| 201 | - ConnectPoint dst = linkDescription.dst(); | ||
| 202 | - // if we aren't master for the device associated with the ConnectPoint | ||
| 203 | - // we probably shouldn't be doing this. | ||
| 204 | 200 | ||
| 205 | -// if (deviceService.getRole(dst.deviceId()) != MastershipRole.MASTER) { | ||
| 206 | -// return; | ||
| 207 | -// } | ||
| 208 | LinkEvent event = store.createOrUpdateLink(provider().id(), | 201 | LinkEvent event = store.createOrUpdateLink(provider().id(), |
| 209 | linkDescription); | 202 | linkDescription); |
| 210 | if (event != null) { | 203 | if (event != null) { |
| ... | @@ -232,11 +225,7 @@ public class LinkManager | ... | @@ -232,11 +225,7 @@ public class LinkManager |
| 232 | public void linksVanished(ConnectPoint connectPoint) { | 225 | public void linksVanished(ConnectPoint connectPoint) { |
| 233 | checkNotNull(connectPoint, "Connect point cannot be null"); | 226 | checkNotNull(connectPoint, "Connect point cannot be null"); |
| 234 | checkValidity(); | 227 | checkValidity(); |
| 235 | - // if we aren't master for the device associated with the ConnectPoint | 228 | + |
| 236 | - // we probably shouldn't be doing this. | ||
| 237 | - if (deviceService.getRole(connectPoint.deviceId()) != MastershipRole.MASTER) { | ||
| 238 | - return; | ||
| 239 | - } | ||
| 240 | log.info("Links for connection point {} vanished", connectPoint); | 229 | log.info("Links for connection point {} vanished", connectPoint); |
| 241 | // FIXME: This will remove links registered by other providers | 230 | // FIXME: This will remove links registered by other providers |
| 242 | removeLinks(getLinks(connectPoint)); | 231 | removeLinks(getLinks(connectPoint)); |
| ... | @@ -246,11 +235,7 @@ public class LinkManager | ... | @@ -246,11 +235,7 @@ public class LinkManager |
| 246 | public void linksVanished(DeviceId deviceId) { | 235 | public void linksVanished(DeviceId deviceId) { |
| 247 | checkNotNull(deviceId, DEVICE_ID_NULL); | 236 | checkNotNull(deviceId, DEVICE_ID_NULL); |
| 248 | checkValidity(); | 237 | checkValidity(); |
| 249 | - // if we aren't master for the device associated with the ConnectPoint | 238 | + |
| 250 | - // we probably shouldn't be doing this. | ||
| 251 | - if (deviceService.getRole(deviceId) != MastershipRole.MASTER) { | ||
| 252 | - return; | ||
| 253 | - } | ||
| 254 | log.info("Links for device {} vanished", deviceId); | 239 | log.info("Links for device {} vanished", deviceId); |
| 255 | removeLinks(getDeviceLinks(deviceId)); | 240 | removeLinks(getDeviceLinks(deviceId)); |
| 256 | } | 241 | } | ... | ... |
-
Please register or login to post a comment