Re: Lars Eggert's No Objection on draft-ietf-rtgwg-yang-rib-extend-20: (with COMMENT)

[Acee Lindem <[e-mail]>]

Hi Lars,

Thanks for you comments. 

> On May 25, 2023, at 2:19 AM, Lars Eggert via Datatracker <[e-mail]> wrote:
> 
> Lars Eggert has entered the following ballot position for
> draft-ietf-rtgwg-yang-rib-extend-20: 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-rtgwg-yang-rib-extend/
> 
> 
> 
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
> 
> # GEN AD review of draft-ietf-rtgwg-yang-rib-extend-18
> 
> CC @larseggert
> 
> ## Comments
> 
> ### Boilerplate
> 
> This document does not use RFC2119 keywords, but contains the RFC8174
> boilerplate.

I’ve removed these in the -21 version. 


> 
> ### Inclusive language
> 
> Found terminology that should be reviewed for inclusivity; see
> https://www.rfc-editor.org/part2/#inclusive_language for background and more
> guidance:
> 
> * Term `master`; alternatives might be `active`, `central`, `initiator`,
>   `leader`, `main`, `orchestrator`, `parent`, `primary`, `server`

Good catch - the examples including “ipv4-master” and “ipv6-master” were migrated from RFC 8349 and enhanced with the new YANG data nodes. I’ve changed these to “ipv4-primary” and “ipv6-master” in the -21 version. 

Thanks,
Acee



> 
> ## Notes
> 
> This review is in the ["IETF Comments" Markdown format][ICMF], You can use the
> [`ietf-comments` tool][ICT] to automatically convert this review into
> individual GitHub issues. Review generated by the [`ietf-reviewtool`][IRT].
> 
> [ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md
> [ICT]: https://github.com/mnot/ietf-comments
> [IRT]: https://github.com/larseggert/ietf-reviewtool
> 
> 
>