Re: [Supa] review for draft-ietf-supa-policy-based-management-framework

"Liushucheng (Will Liu)" <liushucheng@huawei.com> Mon, 03 July 2017 14:02 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 4DC4512EB9B for <supa@ietfa.amsl.com>; Mon, 3 Jul 2017 07:02:06 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.221
X-Spam-Level:
X-Spam-Status: No, score=-4.221 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=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=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 GzNQPQp64p7G for <supa@ietfa.amsl.com>; Mon, 3 Jul 2017 07:02:04 -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 72E9C129462 for <supa@ietf.org>; Mon, 3 Jul 2017 07:02:03 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml705-cah.china.huawei.com) ([172.18.7.190]) by lhrrg01-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id DQI02000; Mon, 03 Jul 2017 14:02:02 +0000 (GMT)
Received: from DGGEML403-HUB.china.huawei.com (10.3.17.33) by lhreml705-cah.china.huawei.com (10.201.108.46) with Microsoft SMTP Server (TLS) id 14.3.301.0; Mon, 3 Jul 2017 15:02:01 +0100
Received: from DGGEML510-MBX.china.huawei.com ([169.254.2.202]) by DGGEML403-HUB.china.huawei.com ([fe80::74d9:c659:fbec:21fa%31]) with mapi id 14.03.0301.000; Mon, 3 Jul 2017 22:01:53 +0800
From: "Liushucheng (Will Liu)" <liushucheng@huawei.com>
To: Johannes Merkle <johannes.merkle@secunet.com>
CC: "supa@ietf.org" <supa@ietf.org>
Thread-Topic: review for draft-ietf-supa-policy-based-management-framework
Thread-Index: AQHS6agud63mWLnjlk2CICe6J02KPqJCFFrg
Date: Mon, 03 Jul 2017 14:01:53 +0000
Message-ID: <C9B5F12337F6F841B35C404CF0554ACB8A32B22B@dggeml510-mbx.china.huawei.com>
References: <C9B5F12337F6F841B35C404CF0554ACB8A305BE9@dggeml510-mbx.china.huawei.com> <e21a77b3-72a0-ff14-6875-c3cb5f13edc5@secunet.com> <C9B5F12337F6F841B35C404CF0554ACB8A3071A1@dggeml510-mbx.china.huawei.com> <e68fdde6-d94c-83b0-399b-20df8964e88a@secunet.com>
In-Reply-To: <e68fdde6-d94c-83b0-399b-20df8964e88a@secunet.com>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.70.79.69]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020205.595A4E5A.008B, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=169.254.2.202, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32
X-Mirapoint-Loop-Id: ac17bab59862326d03f95bd734b6aace
Archived-At: <https://mailarchive.ietf.org/arch/msg/supa/jNQ5A8Ey3r1zdt7aTYX8jAPwjNM>
Subject: Re: [Supa] review for draft-ietf-supa-policy-based-management-framework
X-BeenThere: supa@ietf.org
X-Mailman-Version: 2.1.22
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: Mon, 03 Jul 2017 14:02:06 -0000

Dear Johannes,

Many thanks for your detailed review. We've updated the draft according to all your comments. Please check some responses below inline with [Will].


-----Original Message-----
From: Johannes Merkle [mailto:johannes.merkle@secunet.com] 
Sent: Tuesday, June 20, 2017 5:33 PM
To: supa@ietf.org
Cc: Liushucheng (Will Liu) <liushucheng@huawei.com>
Subject: review for draft-ietf-supa-policy-based-management-framework

I have conducted a review of draft-ietf-supa-policy-based-management-framework. These are my comments:

- Generally, the specification is very abstract and not easy to understand. For instance, it is not explained, what exactly the translation of the information models to data models means. Maybe I lack the necessary experience in the field of network management. I was also surprised that the ID does not define any specific syntax or semantic. For an ID / RFC, the specification is quite vague and unspecific, and it reads like an introduction.

[Will] The relationship of IM and DM can be founded in RFC 3444. We've added this reference into the draft. 
The translation of IM to DM (or in some paragraphs we used "map"),can be found at the book Policy-based Network Management: Solutions for the Next Generation by John,  "A model mapping is a translation from one type of model to another type of model. Model mapping changes the representation and/or  level of abstraction used in one model to another representation and/or level of abstraction in another model. The most ommon form of model mapping is from an information model to a data model..." 
The detailed example can be founded at the draft draft-ietf-supa-generic-policy-data-model, which is mapped from draft-ietf-supa-generic-policy-info-model.

- Abstract:
YANG data models to encode policy, which will point to should be YANG data models to encode policies, which point to or YANG data models to encode a policy, which points to

Section 1,

3rd paragraph:
   The GPIM defines concepts and terminology needed by policy management
   independent of the form and content of the policy rule.  The ECA What does "form" exactly mean here. Syntax? Structure? Encoding?
 
[Will] form here refers to the way of expressing content, such as data repository, data definition language, query language, implementation language, and protocol (typically one or more of these).

4th paragraph:
                                                  The GPIM and the EPRIM
   will both be translated into corresponding YANG [RFC6020][RFC6020bis]
   modules that define policy concepts,
I suggest to replace "will" by "can".

                                additional YANG modules may also be
   defined from the GPIM and/or EPRIM to manage specific functions.
I suggest to change "defined" to "derived"

Section 2

General: The following Terms are missing: SUPA, ECA, YANG, EMS, EMS/NMS/Controller, OSS/BSS/Orchestrator

3rd Paragrapgh (GPDM):
  the service(s) to be managed using policy.
should be
  the service(s) to be managed using policies.
or
  the service(s) to be managed using a policy.

Section 3.1:

1st paragraph:
  information (e.g., part of a sentence that was cut out).) The meaning of "sentence" in this context is not clear to me.
[Will] sentence here is used to explain the concept of snippet as authors received many comments that the term "snippet" is not clear. A sentence refers to a complete expression while snippet is part of it.

2nd paragraph:
   operator interacts with the interface, which is then translated to
   configuration snippets.
I don't understand. The interface is translated to configuration data?
[Will] Fixed. "Which" here refers to the policies input by operators. We actually did a demo in Buenos Aires to show the policy input website for operators. 

3rd paragraph:
   Note that YANG models may not exist.
In which context?
[Will]Fixed.  "that YANG models" refer to Resource and Service YANG data models. 

7th paragraph (1st paragraph on page 6):
   types of policies: ECA policy rules and declarative policy
   statements.
The distinction between ECA policies and declarative policies is not explained. Furthermore, you could mention already here, that declarative policies are out-of-scope for this ID.
[Will] Fixed. There is the explanation in section Operation: "Declarative policies that specify the goals to be achieved but not how to achieve those goals (also called "intent-based" policies) are out of scope for the initial phase of SUPA." We now move this sentence to the terminology section.

9th paragraph:
   During the run time, components communicate with the data instances
   for management and monitoring.
I don't understand what you mean by "data instance" and how it could communicate?
[Will] Fixed by removing the section as this paragraph is missing leading and not necessary explanation for the figure below.

Figure 2:
Some spaces are missing: ECAPolicyRule and PolicyRule

Figure 2: OSS/BSS Orchestrator is shown but not listed in the explanation above.

Explanation of Figure 2:

  ECA Policy Rule Information Data Model (EPRIM):
should be
  ECA Policy Rule Information Model (EPRIM):

   Network Service and Resource Data Models: models of the service as These are not shown in Figure 2.
[Will] Fixed. This is for explaining Figure 1. We now change it to " Resource and Service YANG Data Models" and move this paragraph after figure 1.

Below Figure 3:

   Policies are used to control the management of resources and
   services, while data from resources and services are used to select
   and/or modify policies during runtime.
The latter type of interaction should be mentioned in the explanation of the communication (1). Thus
      (1) policy manages and can adjust service behavior as necessary
      (1:1..n)
should be
      (1) policy manages and can adjust service behavior as necessary
      (1:1..n). In addition, data from resources and services are used to select
   and/or modify policies during runtime.


Section 3.3, 1st paragraph:
  ERPIM --> EPRIM
[Will] Fixed above ones.

Regards, / 致礼! 
Will LIU  / 刘树成