Message info
 
To:netmod@ietf.org From:Martin Bjorklund Subject:[netmod] review of draft-ietf-netmod-routing-cfg-02 Date:Thu, 29 Mar 2012 08:40:30 +0200 (CEST)
 

Hi,

Here are my comments on draft-ietf-netmod-routing-cfg-02.

o 4.1

Each router instance in the core routing data model represents a
(logical) router whose configuration and operation is independent
of other router instances.

Why do we need this extra list (/routing/router)? Can you
examplify when more than one list entry would be used?


o 4.1

Logical network interfaces must be assigned to a router instance

What is a "Logical network interface"?
draft-ietf-netmod-interfaces-cfg does not use this term.

o 4.2

s/destination-prefix/dest-prefix/

OR the other way around in the data model.


o 4.3

You say that at least two routing tables MUST be present.

Can we change the name of a routing table to be:

type union {
type enumeration {
enum fib;
enum main;
}
type string;
}

This makes it clear which table is the fib and which is main.

Also, on my linux box, I can see the FIB, but what about main? Is
it always present?


o 4.4

Each routing protocol instance is connected to exactly one
routing table.

Where is this "exactly one" property in the data model?
"connected-routing-tables" has a list, so its seems a routing
protocol can be connected to 0 or more tables?


o 4.4

By default, every routing protocol instance SHOULD be connected
to the main routing table.

I do not understand this sentence. What does it mean in terms of
the configuration?


o 4.4

Every router instance MUST contain exactly one instance of the
"direct" pseudo-protocol.

What does this mean?


o 4.4.1

s/It is recommended/It is RECOMMENDED/

o leaf cur-hop-limit {
type uint8;
default "64";

Is this default really correct? It doesn't match the text:

The default should be set to the value specified in IANA
Assigned Numbers that was in effect at the time of
implementation.

I suggest you remove the default statement, and change the text to
something like

If not explicitly configured, the value to be used in
the Cur Hop Limit field should be the value specified in IANA
Assigned Numbers that was in effect at the time of
implementation.


/martin
_______________________________________________
netmod mailing list
netmod@ietf.org
https://www.ietf.org/mailman/listinfo/netmod