Task #1407
closed
nfdc add subcommand
100%
Description
Implement add subcommand as a composite of "create" and "add-nexthop".
Command syntax:
nfdc add <Name> <FaceUri> [<Cost>]
This is equivalent to:
FACEID=`nfdc create <FaceUri>`
nfdc add-nexthop <Name> $FACEID <Cost>
Updated by Junxiao Shi over 10 years ago
nfdc create should write the FaceId to stdout.
A bash script can capture this FaceId and provide create + add-nexthop feature.
The philosophy of UNIX is each command only do one thing. Composite operation should use a script.
And this composite operation isn't so useful because prefix registration should use NRD.
Updated by Junxiao Shi over 10 years ago
- Subject changed from nfdc should have 'add' (and maybe 'remove' with aliases 'del' and 'delete') to nfdc add subcommand
- Description updated (diff)
- Assignee set to Hila Ben Abraham
- Priority changed from Normal to Low
- Estimated time set to 2.00 h
Updated by Alex Afanasyev over 10 years ago
- Priority changed from Low to Normal
Hila, what is the progress on this task? People are started to use nfd and lack of helper 'add' command confuses them... :(
Updated by Hila Ben Abraham over 10 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 50
Updated by Hila Ben Abraham over 10 years ago
- Status changed from In Progress to Code review
- % Done changed from 50 to 100
Updated by Syed Amin over 10 years ago
Alex, not sure but I think this "add" updates FIB directly; shouldn't it go through NRD controller? And the command should also have a provision of specifying flags.
Updated by Alex Afanasyev over 10 years ago
This should be extended (we still could have direct modification of FIB, but with a secret nfdc option). Can you propose the extended format for add?
Updated by Alex Afanasyev over 10 years ago
- Related to Task #1416: nfdc register/unregister subcommands added
Updated by Syed Amin over 10 years ago
I was waiting for this task to be finished, so that I can make registration related changes to nfdc. However, I've started from the under review commit now, as the behavior of add command needs to be modified in any case.
Updated by Junxiao Shi over 10 years ago
- Status changed from Code review to Closed