Skip to content

Component Test sample#676

Draft
niranjanivivek wants to merge 2 commits into
sonic-net:masterfrom
niranjanivivek:pr_component_test_draft
Draft

Component Test sample#676
niranjanivivek wants to merge 2 commits into
sonic-net:masterfrom
niranjanivivek:pr_component_test_draft

Conversation

@niranjanivivek
Copy link
Copy Markdown
Contributor

Why I did it

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Niranjani Vivek <niranjaniv@google.com>
@niranjanivivek niranjanivivek force-pushed the pr_component_test_draft branch from e29d523 to d41dd23 Compare May 19, 2026 11:06
@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).


var clientTypes = []string{gclient.Type}

var sfePaths = []struct {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not being used, why is this added? Also please change the name of the variable

@mssonicbld
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants