Roman Danyliw has entered the following ballot position for draft-ietf-lsr-flex-algo-24: No Objection
When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-lsr-flex-algo/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thank you to Charlie Kaufman for the SECDIR review. ** Section 4. We want the mapping between the Flex-Algorithm and its meaning to be flexible and defined by the user. Is a “Flex-Algorithm” (with hyphen) the same as a “Flex Algorithm” (no hyphen) defined in Section 3? Why the difference in notation? ** Section 4. The set consisting of (a) calculation-type, (b) metric-type, and (c) a set of constraints is referred to as a Flexible-Algorithm Definition. Flexible-Algorithm is a numeric identifier in the range 128-255 that is associated via configuration with the Flexible-Algorithm Definition. This text repeats the text in Section 3 verbatim. Is it needed twice? ** Section 5. ... and (b) are in the same Flex-Algorithm definition advertisement scope What is a “FAD advertisement scope?” Should this be an additional term defined in Section 3? ** Section 5.1. Should the explanation of “Metric-Type” say that it’s value comes from the “IGP Metric-Type Registry” per Section 18.1.2? Section 5.1. Is it possible for a peer not to support a particular Metric-Type? If so, have is that signaled? ** Section 5.1. An IS-IS L1/L2 router MAY be configured to re-generate the winning FAD from level 2, ... What is the “winning FAD?” ** Section 5.1 and 5.2. Editorial. Definition of Flex-Algorithm in the TLV. -- Section 5.1 Flex-Algorithm: Single octet value between 128 and 255 inclusive. -- Section 5.2 Flex-Algorithm:: Flex-Algorithm number. Value between 128 and 255 inclusive. Should this text be the same? ** Section 5.2. Editorial. Why is the text defining Metric-Type repeated verbatim from what is written in Section 5.1, but Calc-Type and Priority cite Section 5.1? ** Section 6.4. New flag bits may be defined in the future. Implementations MUST check all advertised flag bits in the received IS-IS FADF Sub-TLV - not just the subset currently defined. Let’s assume bits other than those define in this document are set. Is there an IANA registry to check to understand their semantics? ** Section 13.1. During the route computation, it is possible for the Flex-Algorithm specific metric to exceed the maximum value that can be stored in an unsigned 32-bit variable. In such scenarios, the value MUST be considered to be of value 0xFFFFFFFF during the computation and advertised as such. Should this guidance be referenced in Section 8 – that 0xFFFFFFFF could be 0xFFFFFFFF or an overflow value? ** Section 17. This draft adds two new ways to disrupt IGP networks: An attacker can hijack a particular Flex-Algorithm by advertising a FAD with a priority of 255 (or any priority higher than that of the legitimate nodes). An attacker could make it look like a router supports a particular Flex-Algorithm when it actually doesn't, or vice versa. What are the consequences of the described attacker behaviors? Is “disrupt[ing] the IGP networks” just a denial of service? Could it also be steering traffic in a particular way to allow inspection or modification; or to avoid network based mitigations? _______________________________________________ Lsr mailing list Lsr@ietf.org https://www.ietf.org/mailman/listinfo/lsr