Re: [netmod] I-D Action: draft-ietf-netmod-entity-02.txt

Martin Bjorklund <mbj@tail-f.com> Tue, 24 January 2017 09:28 UTC

Return-Path: <mbj@tail-f.com>
X-Original-To: netmod@ietfa.amsl.com
Delivered-To: netmod@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A931512949A for <netmod@ietfa.amsl.com>; Tue, 24 Jan 2017 01:28:18 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.1
X-Spam-Level:
X-Spam-Status: No, score=-5.1 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-3.199, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id uia9FmUv-hWf for <netmod@ietfa.amsl.com>; Tue, 24 Jan 2017 01:28:16 -0800 (PST)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id AB37F1297C5 for <netmod@ietf.org>; Tue, 24 Jan 2017 01:28:03 -0800 (PST)
Received: from localhost (unknown [173.38.220.36]) by mail.tail-f.com (Postfix) with ESMTPSA id 35B851AE01AA; Tue, 24 Jan 2017 10:28:02 +0100 (CET)
Date: Tue, 24 Jan 2017 10:28:00 +0100
Message-Id: <20170124.102800.608993362937407790.mbj@tail-f.com>
To: bart.bogaert@nokia.com
From: Martin Bjorklund <mbj@tail-f.com>
In-Reply-To: <D62E05768DBAFF42A72B9F4954476D65010EB1F592@FR712WXCHMBA09.zeu.alcatel-lucent.com>
References: <20170123094232.GC29022@elstar.local> <20170123.115841.723508035325803360.mbj@tail-f.com> <D62E05768DBAFF42A72B9F4954476D65010EB1F592@FR712WXCHMBA09.zeu.alcatel-lucent.com>
X-Mailer: Mew version 6.7 on Emacs 24.5 / Mule 6.0 (HANACHIRUSATO)
Mime-Version: 1.0
Content-Type: Text/Plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/FKe8YpnKjJHkqP2nIAHnhJdvxyY>
Cc: netmod@ietf.org
Subject: Re: [netmod] I-D Action: draft-ietf-netmod-entity-02.txt
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: NETMOD WG list <netmod.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod>, <mailto:netmod-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod/>
List-Post: <mailto:netmod@ietf.org>
List-Help: <mailto:netmod-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod>, <mailto:netmod-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Jan 2017 09:28:18 -0000

"Bogaert, Bart (Nokia - BE)" <bart.bogaert@nokia.com> wrote:
> Hi,
> 
> If I'm not mistaken, the following BBF addition was also proposed as to be
> added to the IETF entity model.  Is there no consensus about this one?

Not many spoke up.  I think there was just one comment, and that was
not in favor of adding this.



/martin


> 
> 1. Enabling a reset of an entity by means of an action
> 
> module bbf-entity-reset-action {
>   yang-version 1.1;
> 
>   namespace "urn:broadband-forum-org:bbf-entity-reset-action";
> 
>   prefix "bbf-entity-reset-action";
> 
>   import ietf-entity {
>     prefix ent;
>   }
>   ...
> 
>   identity reset-type {
>     description
>         "Type of reset requested of entity.  Examples of resets can be:
>            hardware-reset, reset-with-selftest, reset-without-selftest,
>            software-reset, possibly others.";
>   }
> 
>   identity hardware-reset {
>     base reset-type;
>     description
>         "Hardware reset";
>   }
> 
>   augment "/ent:entity-state/ent:physical-entity" {
>     description
>         "Augment entity model with an action to request a reset of the
>           entity";
>     action reset {
>       description "Request a reset of the entity";
>       input {
>         leaf reset-type {
>           type identityref {
>             base "reset-type";
>           }
>             description
>                 "Type of reset requested of entity";
>         }
>       }
>     }
>   }
> }
> 
> Best regards - Vriendelijke groeten,
> Bart Bogaert
> 
> -----Original Message-----
> From: netmod [mailto:netmod-bounces@ietf.org] On Behalf Of Martin Bjorklund
> Sent: 23 January 2017 11:59
> To: j.schoenwaelder@jacobs-university.de
> Cc: netmod@ietf.org
> Subject: Re: [netmod] I-D Action: draft-ietf-netmod-entity-02.txt
> 
> Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de> wrote:
> > Hi,
> > 
> > I wonder when we use 'state' and when 'status' - is there a subtle 
> > distinction or should be just consistently use lets say 'state', i.e., 
> > changed to alalarm-status to alarm-state and standby-status to 
> > standby-state?
> 
> The reason in this case is that we just used the MIB names.  This said, I
> agree that "standby-state" and "alarm-state" are better.
> 
> BTW, RFC 4268, which defines the original objects, says:
> 
>    The terms "state" and "status" are used interchangeably in this memo.
> 
> 
> > I also wonder about the mapping of the MIB object names to YANG leaf
> > names:
> > 
> >    +-------------------------------------+-----------------------------+
> >    | YANG data node in /hardware-        | ENTITY-SENSOR-MIB object    |
> >    | state/component/sensor-data         |                             |
> >    +-------------------------------------+-----------------------------+
> >    | data-type                           | entPhySensorType            |
> >    | data-scale                          | entPhySensorScale           |
> >    | precision                           | entPhySensorPrecision       |
> >    | value                               | entPhySensorValue           |
> >    | oper-status                         | entPhySensorOperStatus      |
> >    | sensor-units-display                | entPhySensorUnitsDisplay    |
> >    | value-timestamp                     | entPhySensorValueTimeStamp  |
> >    | value-update-rate                   | entPhySensorValueUpdateRate |
> >    
> > +-------------------------------------+-----------------------------+
> > 
> > Is the 'data-' prefix needed? If so, why is the a prefix not used for 
> > 'precision' (scale and precision really go hand in hand).
> 
> Unclear.  I think I'm the one to blame for this inconsistency, and it goes
> back to the very first commit, but I can't remeber why.
> 
> > Why is it
> > 'sensor-units-display' and not just 'units-display'? One option could
> > be:
> > 
> >   value-type
> >   value-scale
> >   value-precision
> >   value
> >   oper-status
> >   units-display
> >   value-timestamp
> >   value-update-rate
> 
> Yes this is better.
> 
> > RFC 3433 points out that entPhySensorType and entPhySensorScale and 
> > entPhySensorPrecision SHOULD NOT change during operation. What about 
> > the YANG objects? I actually do not know what the SHOULD buys a client 
> > since you can't rely on it. To be robust, you have to fetch an n-tuple 
> > anyway and be prepared that a sensor may have changed its properties.
> > Should there be explicit text providing guidance that it is better to 
> > fetch the whole n-tuple?
> 
> This is certainly the simplest solution.   Any comments?
> 
> > Or alternatively, if supporting caching of values is a goal, we may 
> > need to provide a 'sensor-data/properties-last-changed' object that 
> > allows to make caching of value properties robust.
> 
> 
> /martin
> 
> _______________________________________________
> netmod mailing list
> netmod@ietf.org
> https://www.ietf.org/mailman/listinfo/netmod