Feature #2426
closedndn-autoconfig: Use face query operation to request multicast faces only
100%
Description
The current code requests list of all faces and then filters multicast faces.
Face query offloads filtering operation to NFD and simplifies code of ndn-autoconfig.
Files
Updated by Junxiao Shi almost 10 years ago
- Subject changed from (ndn-autoconfig) Use face query operation to request udp4 multicast faces only to ndn-autoconfig: Use face query operation to request udp4 multicast faces only
- Estimated time set to 2.00 h
Updated by Davide Pesavento almost 10 years ago
Why udp*4* only? Even if multicast udp6 is not supported by NFD at the moment, it might be in the future.
Updated by Junxiao Shi almost 10 years ago
- Subject changed from ndn-autoconfig: Use face query operation to request udp4 multicast faces only to ndn-autoconfig: Use face query operation to request multicast faces only
- Description updated (diff)
Why udp*4* only? Even if multicast udp6 is not supported by NFD at the moment, it might be in the future.
Face query operation can only support one set of conditions.
Querying "(udp4 or udp6) and multicast" requires two query operations "udp4 and multicast" and "udp6 and multicast".
HubDiscovery protocol just says "a multicast face", so the correct condition is just "multicast".
Updated by Alex Afanasyev about 9 years ago
- Status changed from New to In Progress
- Assignee set to Alex Afanasyev
Updated by Alex Afanasyev almost 9 years ago
- Target version changed from v0.4 to v0.5
Updated by Alex Afanasyev almost 9 years ago
- Status changed from In Progress to New
Updated by Junxiao Shi about 8 years ago
- Tracker changed from Task to Feature
- Status changed from New to In Progress
- Assignee changed from Alex Afanasyev to Junxiao Shi
- Target version changed from v0.5 to v0.6
Updated by Junxiao Shi about 8 years ago
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
https://gerrit.named-data.net/3349 moves ndn-autoconfig
to nfd::tools::autoconfig
namespace, to be consistent with nfdc.
https://gerrit.named-data.net/3350 changes MulticastDiscovery
to use FaceQueryDataset
.
Updated by Junxiao Shi about 8 years ago
- File 20161124171320.tgz 20161124171320.tgz added
- Status changed from Code review to In Progress
Updated by Junxiao Shi about 8 years ago
- Blocked by Task #3867: autoconfig: move from NFD repository added
Updated by Junxiao Shi about 8 years ago
- Project changed from NFD to ndn-tools
- Category deleted (
Tools) - Target version deleted (
v0.6) - % Done changed from 100 to 50
IntegrationTests are failing. I'll investigate.
Updated by Junxiao Shi about 8 years ago
- File 20161127163918.tgz 20161127163918.tgz added
Updated by Junxiao Shi about 8 years ago
- Project changed from ndn-tools to NFD
- Category set to Tools
- Status changed from In Progress to Code review
- Target version set to v0.6
- % Done changed from 50 to 100
IntegrationTests are failing due to a limitation of Emulab integ runner. The previous code filters faces by "UDP with IPv4 multicast address" condition; https://gerrit.named-data.net/3350 uses link-type=multi-access
face query filter, which includes both UDP and Ethernet multicast faces. The Emulab integ runner has blocked UDP multicast with iptables, but does not block Ethernet multicast.
#1712 came just in time to offer a blacklist for Ethernet multicast faces. I manually executed sudo infoedit -f /usr/local/etc/ndn/nfd.conf -s face_system.ether.blacklist.subnet -v 155.98.0.0/16
on each host, and then the HUB discovery test scenario is passing.
I'll work on a solution on Emulab integ runner side. In the meanwhile, this one is ready to go.
Updated by Junxiao Shi about 8 years ago
- File 20161206211724.tgz 20161206211724.tgz added
Emulab integ runner is fixed.
https://github.com/yoursunny/ndn-cxx-breaks/commit/a307ad9f42f5c09e28d478c118655a08fc658de8
Updated by Junxiao Shi about 8 years ago
- Status changed from Code review to Closed
Updated by Davide Pesavento almost 6 years ago
- Blocked by deleted (Task #3867: autoconfig: move from NFD repository)