Bug #2783
closed
ndn-link-control-helper does not seem to control the desired link
100%
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）
Files
| 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 | ||
| ndn-simple-test.cc (3.2 KB) ndn-simple-test.cc | scenario code |
Updated by Spyros Mastorakis over 9 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?
Updated by 鹏飞 靳 over 9 years ago
- File ndn-simple-test.cc ndn-simple-test.cc added
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.
Updated by Spyros Mastorakis over 9 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.
Updated by Spyros Mastorakis over 9 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!
Updated by Alex Afanasyev over 9 years ago
- Status changed from Code review to Resolved
Before closing, this issue requires a test case...
Updated by Spyros Mastorakis over 9 years ago
- Status changed from Resolved to Closed
- % Done changed from 90 to 100