Project

General

Profile

Bug #2783

ndn-link-control-helper does not seem to control the desired link

Added by 鹏飞 靳 over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Category:
helpers
Target version:
Start date:
05/04/2015
Due date:
% Done:

100%

Estimated time:

Description

When I use this helper in my experiment to simulate link failure, like "Simulator::Schedule(Seconds(10.0), ndn::LinkControlHelper::FailLinkByName, "R4", "R2");" I find this would fail all the link connected to R4, not just the link between R2 and R4. (The topo and result are shown in the attachment, I use broadcast strategy)

Link_Failure.png (16.2 KB) Link_Failure.png I want fail link between R2 and R4, but it seems like fail all the link connected to R4 鹏飞 靳, 05/04/2015 06:47 AM
ndn-simple-test.cc (3.2 KB) ndn-simple-test.cc scenario code 鹏飞 靳, 05/04/2015 05:51 PM
317

History

#1 Updated by Spyros Mastorakis over 3 years ago

  • Subject changed from ndn-link-control-helper can`t work well to ndn-link-control-helper does not seem to control the desired link

Hello,

may I have the code for your simulation scenario, so that I can reproduce the issue?

#2 Updated by 鹏飞 靳 over 3 years ago

The test code is in the attachment. I modified ndn-simple.cpp. At line 68-76 are the link control code. When I run it, failure the link between 2-4 and 4-2 shows different result. And the topo don`t have link 2-3, but when i fail the link between 2-3 and 3-2, I got different result too.

So i guess the link control function may failure all the link connected to the first node.

#3 Updated by Spyros Mastorakis over 3 years ago

  • Status changed from New to In Progress
  • Assignee set to Spyros Mastorakis

Thanks for the scenario. I will take a look at it and report back.

#4 Updated by Spyros Mastorakis over 3 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 90

I am sorry for the delay. There was indeed a bug in link control helper that was fixed.

Here is the commit fixing the bug:

http://gerrit.named-data.net/#/c/2070/

Thank you for reporting it!

#5 Updated by Alex Afanasyev over 3 years ago

  • Status changed from Code review to Resolved

Before closing, this issue requires a test case...

#6 Updated by Spyros Mastorakis over 3 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 90 to 100

Also available in: Atom PDF