ietf-wg-snac / draft-ietf-snac-simple

Automatically Connecting Stub Networks to Unmanaged Infrastructure
2 stars 5 forks source link

Sync constant names with RFC4861, reword text as appropriate #98

Open Abhayakara opened 4 weeks ago

Abhayakara commented 4 weeks ago

Currently e.g. RA_BEACON_INTERVAL is the same as MinRtrAdvInterval and MaxRtrAdvInterval. This is true of other constants as well. We should use the same constant names when we can, and just override the defaults as appropriate. Also don't use the term "Beacon." An RA beacon is a multicast RA. Update the text to clarify that.

EskoDijk commented 4 weeks ago

There was a final comment (by Fernando also) that it's fine in fact to define own constant names for values that a SNAC router has to set in a particular way.

E.g. we can define SNAC_MIN_RTR_INTERVAL and say in the draft that a SNAC router must set MinRtrAdvInterval := SNAC_MIN_RTR_INTERVAL.
But if we can avoid introducing such SNAC-specific name, that's also fine of course!