[CCAMP] draft-ietf-ccamp-alarm-module-02 CHANGES

stefan vallin <stefan@wallan.se> Sat, 11 August 2018 17:19 UTC

Return-Path: <stefan@wallan.se>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 57FA3130FF7 for <ccamp@ietfa.amsl.com>; Sat, 11 Aug 2018 10:19:27 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, T_DKIMWL_WL_MED=-0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=wallan-se.20150623.gappssmtp.com
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 HjEdgqwsAOes for <ccamp@ietfa.amsl.com>; Sat, 11 Aug 2018 10:19:25 -0700 (PDT)
Received: from mail-lj1-x230.google.com (mail-lj1-x230.google.com [IPv6:2a00:1450:4864:20::230]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 17FD0130FD0 for <ccamp@ietf.org>; Sat, 11 Aug 2018 10:19:24 -0700 (PDT)
Received: by mail-lj1-x230.google.com with SMTP id w16-v6so9456570ljh.12 for <ccamp@ietf.org>; Sat, 11 Aug 2018 10:19:24 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wallan-se.20150623.gappssmtp.com; s=20150623; h=from:content-transfer-encoding:mime-version:subject:message-id:date :to; bh=fvEAgGnDwf7Yq+/Hpupl+rqUr8n7QOCwe3ywU/SPRq4=; b=X2eXgNL9Nc5xAhVABK3OLo0pu6eoKsiQqyRsACDPzdNgTps7hoZaUcRhPH/pieWoso rjTAgK29Pgx7TUwm+6H2i4a10iD1jGCnNXcHhq9LZ50K+Hsli4XMffE4bjVvxxXOcLHX 0frGN/Uyiyl/RQ5aVxSOhD8TWtz97a0hj/OYkuC31URlQHPAfBisG6ubEPcA7SYnxFZ+ 88A9K3e4Tr39+H2MBqBRxvaDghYF4gcre5nSHnIYs6BHSdUNhdqZ/mNLX6FpRB4hgxBp 3FUjH4ctANT1fNaDtwG4CXurFE2Cu7NpsOA/lpfzVnhicLqY3im9cN1TJPBP3RhoxiU3 PQ7g==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:content-transfer-encoding:mime-version :subject:message-id:date:to; bh=fvEAgGnDwf7Yq+/Hpupl+rqUr8n7QOCwe3ywU/SPRq4=; b=fZgnssxZVrIP4mCBRcReBUexiCTvV5YPeeaiud+LDWTt6b3ZyD01NpG1BuV6KRdAnY rHpJvwxAQZFb6gohpMYsYHIbqbDdnU9yzR0M5/aDniHrs2Fy0y7d/VLEv58wJ18lfROJ I3+q2ME3xmqRY9QUIb9B/bZrIjnNUByka8bM83OjWk7hCy8swldwFzn0CBICbzG+TJcY NrDnIlZtXQTUpwzdxVU+Eu2WClnC4GxbFDZrp9Sv60zHCk/SA81lfJUD9gGvXqoA4vzM uktL7uMQ79+a6uKkTxJCkQYLdAUkX3CwJDL1gPAviWCkuEBgm+IXGJCj//GygWQRG6QB PoMQ==
X-Gm-Message-State: AOUpUlH/fZM/DQN5utm9sON+AkX8+E4ifOFtA6IiHcxhgqFJsKUjdO9F ea3zxfLRGJyWxBCaTRrcnSQSWiCeWEc=
X-Google-Smtp-Source: AA+uWPye7oDPufgfydfmbdWn2GDRHd+SHiZ6CRvS0bK+Y/pKVxKiFfs87aDj+kCOhdTlRu60TyNdJg==
X-Received: by 2002:a2e:8457:: with SMTP id u23-v6mr7471566ljh.95.1534007963115; Sat, 11 Aug 2018 10:19:23 -0700 (PDT)
Received: from [192.168.72.11] (h95-155-237-105.cust.se.alltele.net. [95.155.237.105]) by smtp.gmail.com with ESMTPSA id d24-v6sm2514250lfl.53.2018.08.11.10.19.22 for <ccamp@ietf.org> (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 11 Aug 2018 10:19:22 -0700 (PDT)
From: stefan vallin <stefan@wallan.se>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (Mac OS X Mail 11.2 \(3445.5.20\))
Message-Id: <EFABFB61-48A1-44C3-B813-658751F947F5@wallan.se>
Date: Sat, 11 Aug 2018 19:19:21 +0200
To: "CCAMP (ccamp@ietf.org)" <ccamp@ietf.org>
X-Mailer: Apple Mail (2.3445.5.20)
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/94Soa8m1pPPeWQxCxkon_i79xoM>
Subject: [CCAMP] draft-ietf-ccamp-alarm-module-02 CHANGES
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 11 Aug 2018 17:19:27 -0000


Hi All!
Please find a new version 02 of the alarm module, changes below.
Going forward we would like to focus on clarifications and possible errors and limit new features.

Br Stefan and Martin

Alarm Model Changes
==================

Comments from [ITU-T Study Group 15]
================================
#1 Clearly state that the module does not define the instrumentation
Regarding the scope of the YANG Alarm Module, the draft should
explicitly state that the modeling of how transport resource alarms
are raised and cleared by the underlying transport instrumentation is
out of the scope of this draft. [...] should belong to the SDO/group
that owns the transport technology.
---
     Added clarification to the module description:

     This alarm module does not define how the underlying
     instrumentation detects and clears the specific alarms.
     That belongs to the SDO or enterprise that owns that
     specific technology.

#2 The alarm YANG draft adopts the X.733 alarm severities, but doesn’t
support the function of alarm severity assignment (i.e., configuring
the severity of an alarm type of a resource), i.e., the feature of the
Alarm Severity Assignment Profile (ASAP) object of M.3160/M.3100 is
not supported by the draft. Is this intended to be out of the scope of
the alarm YANG draft?
---
    Added alarm profile to the model:
     +--rw alarm-profile* [alarm-type-id alarm-type-qualifier-match resource] {alarm-profile}?
        +--rw alarm-type-id                        al:alarm-type-id
        +--rw alarm-type-qualifier-match           string
        +--rw resource                             al:resource-match
        +--rw description                          string
        +--rw alarm-severity-assignment-profile {severity-assignment}?
           +--rw severity-levels*   al:severity

#3 For alarm reporting control, the alarm YANG draft supports only the
simple alarm shelf function. It doesn’t support the rich features of
the Alarm Reporting Control (ARC) function of M.3160/M.3100, which
is required by G.7710.
---
Added severity filter to the notification control. Alarm shelving and
the notification filter maps to ARC.


Comments from Qin Wu Huawei
==========================
#1 Document related alarms, impacted resources, root cause
---
    Updated 3.6.  Root Cause, Impacted Resources and Related Alarms

#2 Review use of remove, delete and purge
---
    Cleaned up throughout the document

#3 Add uuid RFC6991 to resource
---
    Added uuid to the union:
    typedef resource {
        type union {
            type instance-identifier {
            require-instance false;
           }
        type yang:object-identifier;
        type string;
        type yang:uuid;
    }


Comments from Nick Hancock, Adtran
==============================

#1 Admin actions on shelved alarms
---
    Added rpc compress-shelved-alarms
    purge rpc description updated to include shelved alarms

#2 Add support for alarm severity assignment
---
    Same as ITU, see above

#3 Alarm summary should be feature
---
      feature alarm-summary {
      ...
      container summary {
          if-feature alarm-summary;

#4 Move notifications inside list
---
     notification operator-action moved into alarm list

#4 Notification filter on severity
---
    Added severity level to notification filter
          choice notify-status-changes {
        description
          "This leaf controls the notifications sent for alarm status
           updates. There are three options:
           1. notifications are sent for all updates, severity level
              changes and alarm text changes
           2. notifications are only sent for alarm raise and clear
           3. notifications are sent for status changes equal to or
              above the specified severity level. Clear notifications
              shall always be sent
              Notifications shall also be sent for state changes that
              makes an alarm less severe than the specified level.
           In option 3, assuming the severity level is set to major,
           and that the alarm has the following state changes
           [(Time, severity, clear)]:
           [(T1, major, -), (T2, minor, -), (T3, warning, -),
            (T4, minor, -), (T5, major, -), (T6, critical, -),
            (T7, major. -), (T8, major, clear)]
           In that case, notifications will be sent at
           T1, T2, T5, T6, T7 and T8.";
        leaf notify-all-state-changes {
          type empty;
          description
            "Send notifications for all status changes.";
        }
        leaf notify-raise-and-clear {
          type empty;
          description
            "Send notifications only for raise, clear, and re-raise.
             Notifications for severity level changes or alarm text
             changes are not sent.";
        }
        leaf notify-severity-level {
          type severity;
          description
            "Only send notifications for alarm state changes
             crossing the specified level. Always send clear
             notifications.";
        }
      }


Comments from Marta Seda, Calix.
============================
The ietf-alarms-x733.yang module does not contain all X733/X736
parameters.
---
   Added all X.733 parameters:
     augment /al:alarms/al:alarm-list/al:alarm:
    +--ro event-type?                event-type
    +--ro probable-cause?            uint32
    +--ro probable-cause-string?     string
    +--ro threshold-information
    |  +--ro triggered-threshold?   string
    |  +--ro observed-value?        value-type
    |  +--ro (threshold-level)?
    |  |  +--:(up)
    |  |  |  +--ro up-high?         value-type
    |  |  |  +--ro up-low?          value-type
    |  |  +--:(down)
    |  |     +--ro down-low?        value-type
    |  |     +--ro down-high?       value-type
    |  +--ro arm-time?              yang:date-and-time
    +--ro monitored-attributes* []
    |  +--ro id?      al:resource
    |  +--ro value?   string
    +--ro proposed-repair-actions*   string
    +--ro trend-indication?          trend
    +--ro backedup-status?           boolean
    +--ro backup-object?             al:resource
    +--ro additional-information* []
    |  +--ro identifier?    string
    |  +--ro significant?   boolean
    |  +--ro information?   string
    +--ro security-alarm-detector?   al:resource
    +--ro service-user?              al:resource
    +--ro service-provider?          al:resource


Authors
======

#1 Clarify that alarm-inventory is mandatory

#2 Added require-instance false for alarm-type-id and
 alarm-type-qualifier leafs in rpc compress-alarms

# Editorial changes