Project

General

Profile

Actions

Feature #2426

closed

ndn-autoconfig: Use face query operation to request multicast faces only

Added by Alex Afanasyev almost 10 years ago. Updated almost 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tools
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
2.00 h

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

20161124171320.tgz (584 KB) 20161124171320.tgz integ NFD 3350,2 Junxiao Shi, 11/24/2016 03:06 PM
20161127163918.tgz (588 KB) 20161127163918.tgz integ 3349,2 Junxiao Shi, 11/27/2016 11:46 AM
20161206211724.tgz (573 KB) 20161206211724.tgz integ 3350,5 Junxiao Shi, 12/06/2016 03:37 PM
Actions #1

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
Actions #2

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.

Actions #3

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".

Actions #4

Updated by Alex Afanasyev almost 9 years ago

  • Status changed from New to In Progress
  • Assignee set to Alex Afanasyev
Actions #5

Updated by Alex Afanasyev almost 9 years ago

  • Target version changed from v0.4 to v0.5
Actions #6

Updated by Alex Afanasyev almost 9 years ago

  • Status changed from In Progress to New
Actions #7

Updated by Junxiao Shi almost 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
Actions #8

Updated by Junxiao Shi almost 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.

Actions #9

Updated by Junxiao Shi almost 8 years ago

Actions #10

Updated by Junxiao Shi almost 8 years ago

  • Blocked by Task #3867: autoconfig: move from NFD repository added
Actions #11

Updated by Junxiao Shi almost 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.

Actions #13

Updated by Junxiao Shi almost 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.

Actions #15

Updated by Junxiao Shi almost 8 years ago

  • Status changed from Code review to Closed
Actions #16

Updated by Davide Pesavento almost 6 years ago

  • Blocked by deleted (Task #3867: autoconfig: move from NFD repository)
Actions

Also available in: Atom PDF