Add storage counters related to errors

release/models/platform/.spec.yml

@@ -11,6 +11,7 @@
     - yang/platform/openconfig-platform-cpu.yang
     - yang/platform/openconfig-platform-ext.yang
     - yang/platform/openconfig-platform-software.yang
+    - yang/platform/openconfig-platform-storage.yang
     - yang/platform/openconfig-platform-fabric.yang
     - yang/platform/openconfig-platform-pipeline-counters.yang
     - yang/platform/openconfig-platform-integrated-circuit.yang
@@ -31,6 +32,7 @@
     - yang/platform/openconfig-platform-ext.yang
     - yang/platform/openconfig-platform-cpu.yang
     - yang/platform/openconfig-platform-software.yang
+    - yang/platform/openconfig-platform-storage.yang
     - yang/platform/openconfig-platform-fabric.yang
     - yang/platform/openconfig-platform-pipeline-counters.yang
     - yang/platform/openconfig-platform-integrated-circuit.yang

release/models/platform/openconfig-platform-common.yang

@@ -20,7 +20,13 @@ submodule openconfig-platform-common {
     "This modules contains common groupings that are used in multiple
     components within the platform module.";
 
-  oc-ext:openconfig-version "0.27.0";
+  oc-ext:openconfig-version "0.28.0";
+
+  revision "2024-08-26" {
+    description
+      "Add storage state io-errors.";
+    reference "0.28.0";
+  }
 
   revision "2024-05-29" {
     description

release/models/platform/openconfig-platform-storage.yang

@@ -0,0 +1,122 @@
+module openconfig-platform-storage {
+
+  yang-version "1";
+
+  // namespace
+  namespace "http://openconfig.net/yang/platform/storage";
+
+  prefix "oc-storage";
+
+  import openconfig-platform {
+    prefix oc-platform;
+  }
+  import openconfig-extensions {
+    prefix oc-ext;
+  }
+  import openconfig-yang-types {
+    prefix oc-yang;
+  }
+
+  // meta
+  organization
+    "OpenConfig working group";
+
+  contact
+    "OpenConfig working group
+     www.openconfig.net";
+
+  description
+    "This module defines data related to STORAGE components in the
+     OpenConfig platform model.";
+
+  oc-ext:openconfig-version "0.1.0";
+
+  revision "2024-08-26" {
+    description
+      "Initial revision.";
+    reference
+      "0.1.0";
+  }
+
+  // OpenConfig specific extensions for module metadata.
+  oc-ext:regexp-posix;
+  oc-ext:catalog-organization "openconfig";
+  oc-ext:origin "openconfig";
+
+  // identity statements
+  // typedef statements
+  // grouping statements
+  grouping storage-counters-state {
+    description
+      "Operational state for storage component statistics.  These leaves
+      are derived from a list of commonly supported S.M.A.R.T. counters.
+      Note that while common, these attributes may not be supported by
+      all storage device vendors and media types.";
+
+    leaf soft-read-error-rate {
+      type oc-yang:counter64;
+      description
+        "Uncorrected read errors reported to the operating system. SMART ID
+        201.";
+    }
+
+    leaf reallocated-sectors {
+        type oc-yang:counter64;
+        description
+          "Count of reallocated sectors. The raw value represents a count of
+          the bad sectors that have been found and remapped. SMART ID 5.";
+        reference
+          "S.M.A.R.T. Attribute: Reallocated Sectors Count | Knowledge Base -
+          kb.acronis.com";
+    }
+
+    leaf end-to-end-error {
+      type oc-yang:counter64;
+      description
+        "Count of parity errors which occur in the data path to the media.
+        SMART ID 184.";
+      reference
+        "Acronis Drive Monitor: Disk Health Calculation Knowledge Base -
+        kb.acronis.com";
+    }
+
+    leaf offline-uncorrectable-sectors-count {
+      type oc-yang:counter64;
+      description
+        "The total count of uncorrectable errors when reading/writing a
+        sector. SMART ID 198.";
+      reference
+        "Acronis Drive Monitor: Disk Health Calculation Knowledge Base -
+        kb.acronis.com";
+    }
+
+  }
+
+  grouping storage-state {
+    description
+      "Storage component state.  These counters are derived from the
+       linux kernel block layer statistics in /sys/block/<dev>/stat.
+       Implementations which do not use the linux kernel to access
+       storage should provide equivalent counters.";
+    reference
+      "https://www.kernel.org/doc/Documentation/block/stat.txt";
+
+    container counters {
+      description
+        "A collection of storage specific statistics entitites.";
+
+      uses storage-counters-state;
+    }
+  }
+
+  // data definition statements
+  // augment statements
+  augment "/oc-platform:components/oc-platform:component/" +
+    "oc-platform:storage/oc-platform:state" {
+    description
+      "Adding storage data to component model";
+
+    uses storage-state;
+  }
+}
+

release/models/platform/openconfig-platform.yang

@@ -65,7 +65,13 @@ module openconfig-platform {
     (presence or absence of a component) and state (physical
     attributes or status).";
 
-  oc-ext:openconfig-version "0.27.0";
+  oc-ext:openconfig-version "0.28.0";
+
+  revision "2024-08-26" {
+    description
+      "Add storage state io-errors.";
+    reference "0.28.0";
+  }
 
   revision "2024-05-29" {
     description
@@ -82,13 +88,11 @@ module openconfig-platform {
 
   revision "2024-01-30" {
     description
-      "Add rules for controller-card power-off
-       Also introduces last-poweroff-reason container
-       and last-poweroff-time leaf";
+      "Updated description for component-power-type";
     reference "0.25.0";
   }
 
-  revision "2023-11-28" {
+    revision "2023-11-28" {
     description
       "Add model-name";
     reference "0.24.0";