Re: [Supa] I-D Action: draft-ietf-supa-generic-policy-data-model-00.txt

"Liushucheng (Will)" <liushucheng@huawei.com> Thu, 28 July 2016 08:15 UTC

Return-Path: <liushucheng@huawei.com>
X-Original-To: supa@ietfa.amsl.com
Delivered-To: supa@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 37F9B126579 for <supa@ietfa.amsl.com>; Thu, 28 Jul 2016 01:15:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.508
X-Spam-Level:
X-Spam-Status: No, score=-5.508 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.287, 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 BtxVaxPcKgNT for <supa@ietfa.amsl.com>; Thu, 28 Jul 2016 01:15:47 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 8A68212B013 for <supa@ietf.org>; Thu, 28 Jul 2016 01:15:46 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml704-cah.china.huawei.com) ([172.18.7.190]) by lhrrg01-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id CTI56235; Thu, 28 Jul 2016 08:15:43 +0000 (GMT)
Received: from SZXEMA417-HUB.china.huawei.com (10.82.72.34) by lhreml704-cah.china.huawei.com (10.201.5.130) with Microsoft SMTP Server (TLS) id 14.3.235.1; Thu, 28 Jul 2016 09:15:42 +0100
Received: from SZXEMA509-MBS.china.huawei.com ([169.254.2.215]) by SZXEMA417-HUB.china.huawei.com ([10.82.72.34]) with mapi id 14.03.0235.001; Thu, 28 Jul 2016 16:15:37 +0800
From: "Liushucheng (Will)" <liushucheng@huawei.com>
To: 'John Strassner' <strazpdj@gmail.com>, Joel Halpern Direct <jmh.direct@joelhalpern.com>
Thread-Topic: [Supa] I-D Action: draft-ietf-supa-generic-policy-data-model-00.txt
Thread-Index: AQHR4nMt9/v07SaKDUOCuUc3ALBFtaAqchPg
Date: Thu, 28 Jul 2016 08:15:36 +0000
Message-ID: <C9B5F12337F6F841B35C404CF0554ACB897C931E@SZXEMA509-MBS.china.huawei.com>
References: <20160720104004.22615.62487.idtracker@ietfa.amsl.com>
In-Reply-To: <20160720104004.22615.62487.idtracker@ietfa.amsl.com>
Accept-Language: en-US, zh-CN
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.66.78.84]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-CFilter-Loop: Reflected
X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A090206.5799BF30.0058, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=169.254.2.215, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32
X-Mirapoint-Loop-Id: c4ba38a28594f389600798ca4b0337eb
Archived-At: <https://mailarchive.ietf.org/arch/msg/supa/k6qdV0KCZcUEln-RsZrRRIYT8ss>
Cc: "supa@ietf.org" <supa@ietf.org>
Subject: Re: [Supa] I-D Action: draft-ietf-supa-generic-policy-data-model-00.txt
X-BeenThere: supa@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: "This list is to discuss SUPA \(Simplified Use of Policy Abstractions\) related issues." <supa.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/supa>, <mailto:supa-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/supa/>
List-Post: <mailto:supa@ietf.org>
List-Help: <mailto:supa-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/supa>, <mailto:supa-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 28 Jul 2016 08:15:50 -0000

Hi authors,

Thanks for your efforts. I'm starting to review your new draft. Quick comments are:
1. Most of the parts are easy to understand. However, some examples of how to use certain parts might be added, e.g., decorated-container,  concrete-metadata-container, etc. 
2. You may consider to put the Pyang output tree into your draft, so that people without pyang tool could easily get the relationship among containers.

Pyang Output
module: ietf-supa-policydatamodel
   +--rw supa-encoding-clause-container
   |  +--rw supa-encoding-clause-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                              string
   |     +--rw entity-class?                               identityref
   |     +--rw supa-policy-object-ID-encoding              policy-data-type-id-encoding-list
   |     +--rw supa-policy-object-description?             string
   |     +--rw supa-policy-name?                           string
   |     +--rw supa-has-policy-metadata-agg*               instance-identifier
   |     +--rw supa-has-policy-component-decorator-part    instance-identifier
   |     +--rw supa-policy-clause-exec-status?             enumeration
   |     +--rw supa-has-policy-clause-part*                instance-identifier
   |     +--rw supa-encoded-clause-content                 string
   |     +--rw supa-encoded-clause-encoding                policy-data-type-id-encoding-list
   |     +--rw supa-encoded-clause-language                enumeration
   |     +--rw supa-encoded-clause-response?               boolean
   +--rw supa-policy-variable-container
   |  +--rw supa-policy-variable-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                              string
   |     +--rw entity-class?                               identityref
   |     +--rw supa-policy-object-ID-encoding              policy-data-type-id-encoding-list
   |     +--rw supa-policy-object-description?             string
   |     +--rw supa-policy-name?                           string
   |     +--rw supa-has-policy-metadata-agg*               instance-identifier
   |     +--rw supa-has-policy-component-decorator-part    instance-identifier
   |     +--rw supa-has-policy-component-decorator-agg*    instance-identifier
   |     +--rw supa-decorator-constraints*                 string
   |     +--rw supa-has-decorator-constraint-encoding?     policy-constraint-language-list
   |     +--rw supa-policy-term-is-negated?                boolean
   |     +--rw supa-policy-variable-name?                  string
   +--rw supa-policy-operator-container
   |  +--rw supa-policy-operator-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                              string
   |     +--rw entity-class?                               identityref
   |     +--rw supa-policy-object-ID-encoding              policy-data-type-id-encoding-list
   |     +--rw supa-policy-object-description?             string
   |     +--rw supa-policy-name?                           string
   |     +--rw supa-has-policy-metadata-agg*               instance-identifier
   |     +--rw supa-has-policy-component-decorator-part    instance-identifier
   |     +--rw supa-has-policy-component-decorator-agg*    instance-identifier
   |     +--rw supa-decorator-constraints*                 string
   |     +--rw supa-has-decorator-constraint-encoding?     policy-constraint-language-list
   |     +--rw supa-policy-term-is-negated?                boolean
   |     +--rw supa-policy-value-op-type                   enumeration
   +--rw supa-policy-value-container
   |  +--rw supa-policy-value-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                              string
   |     +--rw entity-class?                               identityref
   |     +--rw supa-policy-object-ID-encoding              policy-data-type-id-encoding-list
   |     +--rw supa-policy-object-description?             string
   |     +--rw supa-policy-name?                           string
   |     +--rw supa-has-policy-metadata-agg*               instance-identifier
   |     +--rw supa-has-policy-component-decorator-part    instance-identifier
   |     +--rw supa-has-policy-component-decorator-agg*    instance-identifier
   |     +--rw supa-decorator-constraints*                 string
   |     +--rw supa-has-decorator-constraint-encoding?     policy-constraint-language-list
   |     +--rw supa-policy-term-is-negated?                boolean
   |     +--rw supa-policy-value-content*                  string
   |     +--rw supa-policy-value-encoding?                 policy-data-type-encoding-list
   +--rw supa-policy-generic-decorated-container
   |  +--rw supa-encoding-clause-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                              string
   |     +--rw entity-class?                               identityref
   |     +--rw supa-policy-object-ID-encoding              policy-data-type-id-encoding-list
   |     +--rw supa-policy-object-description?             string
   |     +--rw supa-policy-name?                           string
   |     +--rw supa-has-policy-metadata-agg*               instance-identifier
   |     +--rw supa-has-policy-component-decorator-part    instance-identifier
   |     +--rw supa-has-policy-component-decorator-agg*    instance-identifier
   |     +--rw supa-decorator-constraints*                 string
   |     +--rw supa-has-decorator-constraint-encoding?     policy-constraint-language-list
   |     +--rw supa-policy-generic-decorated-content*      string
   |     +--rw supa-policy-generic-decorated-encoding?     policy-data-type-encoding-list
   +--rw supa-policy-collection-container
   |  +--rw supa-policy-collection-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                              string
   |     +--rw entity-class?                               identityref
   |     +--rw supa-policy-object-ID-encoding              policy-data-type-id-encoding-list
   |     +--rw supa-policy-object-description?             string
   |     +--rw supa-policy-name?                           string
   |     +--rw supa-has-policy-metadata-agg*               instance-identifier
   |     +--rw supa-has-policy-component-decorator-part    instance-identifier
   |     +--rw supa-has-policy-component-decorator-agg*    instance-identifier
   |     +--rw supa-decorator-constraints*                 string
   |     +--rw supa-has-decorator-constraint-encoding?     policy-constraint-language-list
   |     +--rw supa-policy-collection-content*             string
   |     +--rw supa-policy-collection-encoding             enumeration
   |     +--rw supa-policy-collection-function?            enumeration
   |     +--rw supa-policy-collection-is-ordered?          boolean
   |     +--rw supa-policy-collection-type                 enumeration
   +--rw supa-policy-concrete-metadata-container
   |  +--rw supa-policy-concrete-metadata-list* [supa-policy-metadata-id]
   |     +--rw supa-policy-metadata-id                    string
   |     +--rw entity-class?                              identityref
   |     +--rw supa-policy-metadata-id-encoding           policy-data-type-id-encoding-list
   |     +--rw supa-policy-metadata-description?          string
   |     +--rw supa-policy-metadata-name?                 string
   |     +--rw supa-has-policy-metadata-part*             instance-identifier
   |     +--rw supa-policy-metadata-decorator-part        instance-identifier
   |     +--rw supa-policy-metadata-valid-period-end?     yang:date-and-time
   |     +--rw supa-policy-metadata-valid-period-start?   yang:date-and-time
   +--rw supa-policy-metadata-decorator-access-container
   |  +--rw supa-policy-metadata-decorator-access-list* [supa-policy-metadata-id]
   |     +--rw supa-policy-metadata-id                string
   |     +--rw entity-class?                          identityref
   |     +--rw supa-policy-metadata-id-encoding       policy-data-type-id-encoding-list
   |     +--rw supa-policy-metadata-description?      string
   |     +--rw supa-policy-metadata-name?             string
   |     +--rw supa-has-policy-metadata-part*         instance-identifier
   |     +--rw supa-policy-metadata-decorator-part    instance-identifier
   |     +--rw supa-policy-metadata-decorator-agg*    instance-identifier
   +--rw supa-policy-metadata-decorator-version-container
   |  +--rw supa-policy-metadata-decorator-version-list* [supa-policy-metadata-id]
   |     +--rw supa-policy-metadata-id                string
   |     +--rw entity-class?                          identityref
   |     +--rw supa-policy-metadata-id-encoding       policy-data-type-id-encoding-list
   |     +--rw supa-policy-metadata-description?      string
   |     +--rw supa-policy-metadata-name?             string
   |     +--rw supa-has-policy-metadata-part*         instance-identifier
   |     +--rw supa-policy-metadata-decorator-part    instance-identifier
   |     +--rw supa-policy-metadata-decorator-agg*    instance-identifier
   +--rw supa-policy-metadata-detail-container
   |  +--rw supa-policy-metadata-detail-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                               string
   |     +--rw entity-class?                                identityref
   |     +--rw supa-has-policy-metadata-object-ptr?         instance-identifier
   |     +--rw supa-has-policy-metadata-ptr?                instance-identifier
   |     +--rw supa-policy-metadata-detail-is-applicable?   boolean
   |     +--rw supa-policy-metadata-detail-constraint*      string
   |     +--rw supa-policy-metadata-detail-encoding?        string
   +--rw supa-policy-component-decorator-detail-container
   |  +--rw supa-policy-component-decorator-detail-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                            string
   |     +--rw entity-class?                             identityref
   |     +--rw supa-policy-component-decorator-ptr?      instance-identifier
   |     +--rw supa-policy-component-ptr?                instance-identifier
   |     +--rw supa-has-decorator-constraint*            string
   |     +--rw supa-has-decorator-constraint-encoding?   string
   +--rw supa-policy-source-detail-container
   |  +--rw supa-policy-source-detail-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                         string
   |     +--rw entity-class?                          identityref
   |     +--rw supa-policy-source-structure-ptr?      instance-identifier
   |     +--rw supa-policy-source-ptr?                instance-identifier
   |     +--rw supa-policy-source-is-authenticated?   boolean
   |     +--rw supa-policy-source-is-trusted?         boolean
   +--rw supa-policy-target-detail-container
   |  +--rw supa-policy-target-detail-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                         string
   |     +--rw entity-class?                          identityref
   |     +--rw supa-policy-target-structure-ptr?      instance-identifier
   |     +--rw supa-policy-target-ptr?                instance-identifier
   |     +--rw supa-policy-source-is-authenticated?   boolean
   |     +--rw supa-policy-source-is-enabled?         boolean
   +--rw supa-policy-clause-detail-container
   |  +--rw supa-policy-clause-detail-list* [supa-policy-ID]
   |     +--rw supa-policy-ID                      string
   |     +--rw entity-class?                       identityref
   |     +--rw supa-policy-clause-structure-ptr?   instance-identifier
   |     +--rw supa-policy-clause-ptr?             instance-identifier
   +--rw supa-policy-exec-fail-take-action-detail-container
      +--rw supa-policy-exec-fail-take-action-detail-list* [supa-policy-ID]
         +--rw supa-policy-ID                                string
         +--rw entity-class?                                 identityref
         +--rw supa-policy-structure-action-src-ptr?         instance-identifier
         +--rw supa-policy-structure-action-dst-ptr?         instance-identifier
         +--rw supa-policy-exec-fail-take-action-encoding?   policy-data-type-id-encoding-list
         +--rw supa-policy-exec-fail-take-action-name*       string

Regards,
Will (Shucheng LIU)


> -----Original Message-----
> From: Supa [mailto:supa-bounces@ietf.org] On Behalf Of
> internet-drafts@ietf.org
> Sent: Wednesday, July 20, 2016 6:40 PM
> To: i-d-announce@ietf.org
> Cc: supa@ietf.org
> Subject: [Supa] I-D Action: draft-ietf-supa-generic-policy-data-model-00.txt
> 
> 
> A New Internet-Draft is available from the on-line Internet-Drafts directories.
> This draft is a work item of the Simplified Use of Policy Abstractions of the IETF.
> 
>         Title           : Generic Policy Data Model for Simplified Use of
> Policy Abstractions (SUPA)
>         Authors         : Joel Halpern
>                           John Strassner
> 	Filename        : draft-ietf-supa-generic-policy-data-model-00.txt
> 	Pages           : 48
> 	Date            : 2016-07-20
> 
> Abstract:
>    This document defines two YANG policy data models. The first is a
>    generic policy model that is meant to be extended on an application-
>    specific basis. The second is an exemplary extension of the first
>    generic policy model, and defines rules as event-condition-action
>    policies. Both models are independent of the level of abstraction of
>    the content and meaning of a policy.
> 
> 
> 
> The IETF datatracker status page for this draft is:
> https://datatracker.ietf.org/doc/draft-ietf-supa-generic-policy-data-model/
> 
> There's also a htmlized version available at:
> https://tools.ietf.org/html/draft-ietf-supa-generic-policy-data-model-00
> 
> 
> Please note that it may take a couple of minutes from the time of submission
> until the htmlized version and diff are available at tools.ietf.org.
> 
> Internet-Drafts are also available by anonymous FTP at:
> ftp://ftp.ietf.org/internet-drafts/
> 
> _______________________________________________
> Supa mailing list
> Supa@ietf.org
> https://www.ietf.org/mailman/listinfo/supa