Skip to content

[Yang-model] Support dot in the additional information of the PFC watchdog event in the yang model#18235

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
stephenxs:yang-model-support-dot-in-addinfo-in-pfc-wd
Mar 15, 2024
Merged

[Yang-model] Support dot in the additional information of the PFC watchdog event in the yang model#18235
qiluo-msft merged 1 commit intosonic-net:masterfrom
stephenxs:yang-model-support-dot-in-addinfo-in-pfc-wd

Conversation

@stephenxs
Copy link
Copy Markdown
Collaborator

Why I did it

Support dot (.) in the additional_info field in the PFC watchdog event in the yang model.

The additional_info field was introduced to represent diagnosis information when a PFC storm is detected, which can include fragments.

Signed-off-by: Stephen Sun stephens@nvidia.com

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Unit test.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

…n the yang model

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs stephenxs marked this pull request as ready for review March 4, 2024 02:17
@stephenxs stephenxs requested a review from qiluo-msft as a code owner March 4, 2024 02:17
@zbud-msft
Copy link
Copy Markdown
Contributor

@stephenxs Can you please test this change with 202305?

@stephenxs
Copy link
Copy Markdown
Collaborator Author

@stephenxs Can you please test this change with 202305?

Yes. It's the same as 2311. No issues.

@qiluo-msft qiluo-msft added the YANG YANG model related changes label Mar 11, 2024
@qiluo-msft qiluo-msft merged commit a333309 into sonic-net:master Mar 15, 2024
@stephenxs stephenxs deleted the yang-model-support-dot-in-addinfo-in-pfc-wd branch March 15, 2024 02:10
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Mar 21, 2024
…n the yang model (sonic-net#18235)

### Why I did it

Support dot (`.`) in the `additional_info` field in the PFC watchdog event in the yang model.

The `additional_info` field was introduced to represent diagnosis information when a PFC storm is detected, which can include fragments.

Signed-off-by: Stephen Sun <stephens@nvidia.com>

### How I did it

#### How to verify it

Unit test.
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202311: #18424

mssonicbld pushed a commit that referenced this pull request Mar 21, 2024
…n the yang model (#18235)

### Why I did it

Support dot (`.`) in the `additional_info` field in the PFC watchdog event in the yang model.

The `additional_info` field was introduced to represent diagnosis information when a PFC storm is detected, which can include fragments.

Signed-off-by: Stephen Sun <stephens@nvidia.com>

### How I did it

#### How to verify it

Unit test.
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Apr 26, 2024
…n the yang model (sonic-net#18235)

### Why I did it

Support dot (`.`) in the `additional_info` field in the PFC watchdog event in the yang model.

The `additional_info` field was introduced to represent diagnosis information when a PFC storm is detected, which can include fragments.

Signed-off-by: Stephen Sun <stephens@nvidia.com>

### How I did it

#### How to verify it

Unit test.
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202305: #18809

mssonicbld pushed a commit that referenced this pull request Apr 26, 2024
…n the yang model (#18235)

### Why I did it

Support dot (`.`) in the `additional_info` field in the PFC watchdog event in the yang model.

The `additional_info` field was introduced to represent diagnosis information when a PFC storm is detected, which can include fragments.

Signed-off-by: Stephen Sun <stephens@nvidia.com>

### How I did it

#### How to verify it

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

8 participants