Skip to content

Conversation

@wlanmac
Copy link

@wlanmac wlanmac commented Nov 26, 2025

cloned from #1378

Add GLOME YANG paths for Telemetry. The descriptions and leaves were already defined, just not linked to YANG paths.

The paths are:
/system/aaa/authentication/glome/state/enabled
/system/aaa/authentication/glome/state/active-glome-key-version
/system/aaa/authentication/glome/state/active-glome-key-created-on

Change Scope

  • Adding basic GLOME telemetry support.
  • Only adding leafs of already defined model.

Platform Implementations

  • Implementation A/B: Multi-vendor support

Tree View

 module: openconfig-system
   +--rw system
      +--rw config
      |  +--rw hostname?       oc-inet:domain-name
      |  +--rw domain-name?    oc-inet:domain-name
      |  +--rw login-banner?   string
      |  +--rw motd-banner?    string
      +--ro state
      |  +--ro hostname?                       oc-inet:domain-name
      |  +--ro domain-name?                    oc-inet:domain-name
      |  +--ro login-banner?                   string
      |  +--ro motd-banner?                    string
      |  +--ro current-datetime?               oc-yang:date-and-time
      |  +--ro up-time?                        oc-types:timeticks64
      |  +--ro boot-time?                      oc-types:timeticks64
      |  +--ro software-version?               string
      |  +--ro last-configuration-timestamp?   oc-types:timeticks64
      +--ro mount-points
      |  +--ro mount-point* [name]
      |     +--ro name     -> ../state/name
      |     +--ro state
      |        +--ro name?                string
      |        +--ro storage-component?   -> /oc-platform:components/component/name
      |        +--ro size?                uint64
      |        +--ro available?           uint64
      |        +--ro utilized?            uint64
      |        +--ro type?                union
      +--rw aaa
      |  +--rw config
      |  +--ro state
      |  +--rw authentication
+     |  |  +--rw glome
+     |  |  |  +--ro state
+     |  |  |     +--ro oc-gnsi-credz:active-glome-key-version?      version
+     |  |  |     +--ro oc-gnsi-credz:active-glome-key-created-on?   oc-types:timeticks64
+     |  |  |     +--ro oc-gnsi-credz:enabled?                       boolean

@wlanmac wlanmac requested a review from a team as a code owner November 26, 2025 16:37
@dplore dplore moved this to Ready to discuss in OC Operator Review Nov 27, 2025
@dplore
Copy link
Member

dplore commented Nov 27, 2025

/gcbrun

@OpenConfigBot
Copy link

No major YANG version changes in commit ab00b04

@dplore
Copy link
Member

dplore commented Nov 27, 2025

THanks for continuing this @wlanmac . Please also add an augment statement for the access-points mirror of this tree, so we don't have an empty container there.

Something like this should do it:

augment "/oc-ap:access-points/oc-ap:access-point/oc-sys:system/oc-sys:aaa/oc-sys:authentication/oc-sys:glome/" 
        "oc-sys:state" {
    when "../../../../oc-sys-grpc:grpc-servers/oc-sys-grpc:grpc-server" +
        "/oc-sys-grpc:config[contains(oc-sys-grpc:services, 'oc-gnsi:GNSI')]" +
        "/oc-sys-grpc:enable = 'true'";
    description
        "glome freshness information.";
    uses glome-key-version;
  }

@dplore dplore self-assigned this Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Ready to discuss

Development

Successfully merging this pull request may close these issues.

3 participants