Brian,
Thank you for your detailed reviewed of this document in the framework on
INT-DIR.
I will refer to your review for my ballot next week.
Regards
-éric
-----Original Message-----
From: Int-dir <int-dir-boun...@ietf.org> on behalf of Brian Haberman via
Datatracker <nore...@ietf.org>
Reply-To: Brian Haberman <br...@innovationslab.net>
Date: Monday, 14 September 2020 at 20:51
To: "int-...@ietf.org" <int-...@ietf.org>
Cc: "last-c...@ietf.org" <last-c...@ietf.org>, "spring@ietf.org"
<spring@ietf.org>, "draft-ietf-spring-srv6-network-programming....@ietf.org"
<draft-ietf-spring-srv6-network-programming....@ietf.org>
Subject: [Int-dir] Intdir telechat review of
draft-ietf-spring-srv6-network-programming-18
Reviewer: Brian Haberman
Review result: On the Right Track
Section 3
- - - - - - -
The abbreviated description of the section is a bit confusing related to FIB
lookup. This text:
When an SRv6 SID is in the Destination Address field of an IPv6
header of a packet, it is routed through an IPv6 network as an IPv6
address.
Its processing is defined in [RFC8754] section 4.3 and reproduced
here as a reminder.
makes it sound like all FIB lookups are being done on SIDs whereas the text
in
8754, section 4.3 is much clearer that the lookups occur on IPv6 addresses
and
that some may be SIDs.
Section 3.1
- - - - - -
This document defines an SRv6 SID as consisting of LOC:FUNCT:ARG,
where a locator (LOC) is encoded in the L most significant bits of
the SID, followed by F bits of function (FUNCT) and A bits of
arguments (ARG). L, the locator length, is flexible, and an operator
is free to use the locator length of their choice. F and A may be
any value as long as L+F+A <= 128. When L+F+A is less than 128 then
the remainder of the SID MUST be zero.
Does a system outside of the SR Ingress Node need to discover L? If so, is
it
derived from seeing a FIB entry for LOC? How does the a system determine the
length of F and A? By comparing FIB entries for LOC and LOC:FUNCT (that is
what
I infer from section 3.2)? The parsing rules seem incomplete and can lead to
behavior that is non-deterministic. The same can be said for B:N.
What are the guidelines for choosing LOC (or B)? Does this come strictly
out of
the unicast address space? ULA space? Does this spec support LOC being
allocated out of multicast space?
The following text seems rather limiting:
The ARG value of a routed SID SHOULD remain constant among packets in
a given flow. Varying ARG values among packets in a flow may result
in different ECMP hashing and cause re-ordering.
If ARG needs to stay constant, does this limit the types of functions that
can
be implemented using this technique?
Section 3.2
- - - - - -
The various paragraphs that describe “example deployments” really don’t
belong
in a standards track document. If they are needed to explain the approach,
then
the description of the approach is incomplete. The reader should not have to
infer functionality by parsing example uses. If the examples remain, I
suggest
they be put in an informative appendix.
What constitutes a “remote node”?
Section 4
- - - - -
I would suggest either mentioning that these behaviors are managed via an
IANA
registry or I would add a forward pointer to the IANA Considerations
section.
Section 4.16.1.2
- - - - - - - -
The steps described to process the SRH (i.e., instruction S14.4) is
different
from the process described for SRH processing in RFC 8754, Section 4.3.1.1.
RFC
8754 seems to only create an SRH in an encapsulating header (i.e., no SRH
insertion). Why does this draft specify SRH removal?
Section 5.1
- - - - - -
What is the relationship between node N and the address T used as the source
address of the encapsulating header?
Section 6
- - - - -
This section could use some introductory text to explain what is meant by an
Operation.
Section 7
- - - - -
Is there a security issue if a SID is used as a source address?
Should any part of the prefix being used for SIDs be advertised to external
peers/networks?
_______________________________________________
Int-dir mailing list
int-...@ietf.org
https://www.ietf.org/mailman/listinfo/int-dir
_______________________________________________
spring mailing list
spring@ietf.org
https://www.ietf.org/mailman/listinfo/spring
