Warren Kumari's Discuss on draft-ietf-bfd-yang-16: (with DISCUSS and COMMENT)
Warren Kumari <warren@kumari.net> Wed, 04 July 2018 17:17 UTC
Return-Path: <warren@kumari.net>
X-Original-To: rtg-bfd@ietf.org
Delivered-To: rtg-bfd@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 4403C130DE6; Wed, 4 Jul 2018 10:17:24 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Warren Kumari <warren@kumari.net>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-bfd-yang@ietf.org, Jeffrey Haas <jhaas@pfrc.org>, bfd-chairs@ietf.org, jhaas@pfrc.org, rtg-bfd@ietf.org
Subject: Warren Kumari's Discuss on draft-ietf-bfd-yang-16: (with DISCUSS and COMMENT)
X-Test-IDTracker: no
X-IETF-IDTracker: 6.81.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <153072464426.27514.14043810545632452277.idtracker@ietfa.amsl.com>
Date: Wed, 04 Jul 2018 10:17:24 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-bfd/KeRhO1ZqrzNz4M8xp6NMOY9bUSc>
X-BeenThere: rtg-bfd@ietf.org
X-Mailman-Version: 2.1.26
List-Id: "RTG Area: Bidirectional Forwarding Detection DT" <rtg-bfd.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-bfd/>
List-Post: <mailto:rtg-bfd@ietf.org>
List-Help: <mailto:rtg-bfd-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 04 Jul 2018 17:17:25 -0000
Warren Kumari has entered the following ballot position for draft-ietf-bfd-yang-16: Discuss 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/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-bfd-yang/ ---------------------------------------------------------------------- DISCUSS: ---------------------------------------------------------------------- Don't panic, this should be an easy DISCUSS to clear, but I think it important for interoperability. In multiple places, you have: +--ro number-of-sessions? +--ro number-of-sessions-up? +--ro number-of-sessions-down? +--ro number-of-sessions-admin-down? I'm a little confused by the meaning of the counters, and didn't see them clearly defined anywhere. Apologies if I missed it... Are "number-of-sessions-admin-down" included in "number-of-sessions-down"? Is 'number-of-sessions' always equal to 'number-of-sessions-up' + 'number-of-sessions-down', or is it always equal to 'number-of-sessions-up' + 'number-of-sessions-down' + 'number-of-sessions-admin-down', or are there other cases? E.g: I have created 10 sessions (because I have 10 interfaces). 5 of them are down because there is no peer, 3 of them I've configured to be down (admin down), and so 2 of them are up. What should be in each of: number-of-sessions? number-of-sessions-up? number-of-sessions-down? number-of-sessions-admin-down? ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- Thank you. I also had a few minor nits: Nits: Section 1: "The YANG modules in this document conform to the Network Management Datastore Architecture (NMDA) Network Management Datastore Architecture [RFC8342]. " The Department of Redundancy Department called and wants some of their words back please :-) Section 2: "Since BFD is used for liveliness detection of various forwarding paths, there is no uniform key to identify a BFD session. So the BFD data model is split in multiple YANG modules where each module corresponds to one type of forwarding path." I think this would be more readable as: "... to identify a BFD session, and so the BFD..." (hey, I said it was a nit)
- Warren Kumari's Discuss on draft-ietf-bfd-yang-16… Warren Kumari
- Re: Warren Kumari's Discuss on draft-ietf-bfd-yan… Warren Kumari
- Re: Warren Kumari's Discuss on draft-ietf-bfd-yan… Reshad Rahman (rrahman)
- Re: Warren Kumari's Discuss on draft-ietf-bfd-yan… Warren Kumari
- Re: Warren Kumari's Discuss on draft-ietf-bfd-yan… Reshad Rahman (rrahman)