public icon indicating copy to clipboard operation
public copied to clipboard

Need a top level grouping for Telemetry module

Open bijoymc opened this issue 9 years ago • 2 comments

Hi,

All the other yang modules defines a top level grouping. Please add a top level grouping for Telemetry module as well, which can help in making it consistent with the other yang models. Also this helps in adding features on top of this model.

Looking at something like, @@ -29,6 +29,12 @@ reference "0.2.0"; }

  • grouping telemetry-system-top {

  • description
    
  •  "Top-level grouping for interface configuration and
    
  •  operational state data";
    

    container telemetry-system { description "Top level configuration and state for the @@ -438,6 +444,7 @@ } } }

  • }

    // extension statements

@@ -722,6 +729,8 @@ } }

  • uses telemetry-system-top ;

// data definition statements

// augment statements

Thanks! Bijoy.

bijoymc avatar Jul 14 '16 05:07 bijoymc

Hi Bijoy,

Thanks for the issue - I agree that there are some elements of this module that need to be brought in-line with the OpenConfig style guide. We'll start to do this, and push a new revision with a telemetry-top grouping following review in the private repo.

Thanks! r.

robshakir avatar Jul 14 '16 13:07 robshakir

This issue is stale because it has been open 180 days with no activity. If you wish to keep this issue active, please remove the stale label or add a comment, otherwise will be closed in 14 days.

github-actions[bot] avatar Jul 06 '24 02:07 github-actions[bot]