Hi Jeffrey, Thanks for the review, please see my answers below.
Thanks, Yingzhen On Mon, May 1, 2023 at 11:43 AM Zhaohui Zhang via Datatracker < [email protected]> wrote: > Reviewer: Zhaohui Zhang > Review result: Has Issues > > I have the following one nit comment and one question: > > augment "/rt:routing/rt:ribs/rt:rib/" > + "rt:routes/rt:route/rt:next-hop/rt:next-hop-options/" > + "rt:next-hop-list/rt:next-hop-list/rt:next-hop" > { > description > "Augment the multiple next hops with repair path."; > uses repair-path; > } > > The description is slightly misleading. It is to agument a single next-hop > in > the next-hop-list, not "multiple next hops". > > [Yingzhen] how about: "Augment the next-hop with a repair path." Shouldn't the repair path be applicable to static routes as well? > [Yingzhen]: Theoretically you can have a repair-path for a static route, but have you seen this in deployment?
_______________________________________________ rtgwg mailing list [email protected] https://www.ietf.org/mailman/listinfo/rtgwg
