Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix device type and add cluster in DEVICE_NEIGHBOR_METADATA yang model #17049

Merged

Conversation

Gfrom2016
Copy link
Contributor

@Gfrom2016 Gfrom2016 commented Oct 31, 2023

Why I did it

The current DEVICE_NEIGHBOR_METADATA yang model has two issues that would block GCU operation when it checks if the current config aligns with the YANG model:

  1. Missing cluster field in YANG
  2. Incomplete set of device type. The device type in YANG model doesn't include all the device type.
Work item tracking
  • Microsoft ADO (number only): 25577813

How I did it

  1. Add cluster field in DEVICE_NEIGHBOR_METADATA YANG model.
  2. Change device type to string.
  3. Fix the UT test accordingly.

How to verify it

Build the image and verify the unit tests passed.

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)

Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
@Gfrom2016 Gfrom2016 force-pushed the fix_device_neighbor_metadata branch from 99be6ce to 5f796cb Compare October 31, 2023 07:16
Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
Signed-off-by: zitingguo <zitingguo@microsoft.com>
@Gfrom2016
Copy link
Contributor Author

/azpw ms_conflict

@Gfrom2016 Gfrom2016 changed the title [TEST]fix device type and add cluster Fix device type and add cluster in DEVICE_NEIGHBOR_METADATA yang model Nov 7, 2023
@Gfrom2016 Gfrom2016 marked this pull request as ready for review November 7, 2023 03:21
@Gfrom2016 Gfrom2016 requested a review from qiluo-msft as a code owner November 7, 2023 03:21
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS requested a review from wen587 November 7, 2023 04:37
@Gfrom2016
Copy link
Contributor Author

/azpw ms_conflict

@mssonicbld
Copy link
Collaborator

@Gfrom2016 PR conflicts with 202305 branch

Gfrom2016 added a commit to Gfrom2016/sonic-buildimage that referenced this pull request Nov 7, 2023
sonic-net#17049)

Why I did it
The current DEVICE_NEIGHBOR_METADATA yang model has two issues that would block GCU operation when it checks if the current config aligns with the YANG model:

Missing cluster field in YANG
Incomplete set of device type. The device type in YANG model doesn't include all the device type.
Work item tracking
Microsoft ADO (number only): 25577813
How I did it
Add cluster field in DEVICE_NEIGHBOR_METADATA YANG model.
Change device type to string.
Fix the UT test accordingly.
How to verify it
Build the image and verify the unit tests passed.

Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
@Gfrom2016 Gfrom2016 deleted the fix_device_neighbor_metadata branch November 7, 2023 07:20
StormLiangMS pushed a commit that referenced this pull request Nov 8, 2023
#17049) (#17109)

Why I did it
The current DEVICE_NEIGHBOR_METADATA yang model has two issues that would block GCU operation when it checks if the current config aligns with the YANG model:

Missing cluster field in YANG
Incomplete set of device type. The device type in YANG model doesn't include all the device type.
Work item tracking
Microsoft ADO (number only): 25577813
How I did it
Add cluster field in DEVICE_NEIGHBOR_METADATA YANG model.
Change device type to string.
Fix the UT test accordingly.
How to verify it
Build the image and verify the unit tests passed.

Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
Gfrom2016 added a commit to Gfrom2016/sonic-buildimage that referenced this pull request Nov 21, 2023
sonic-net#17049)

Why I did it
The current DEVICE_NEIGHBOR_METADATA yang model has two issues that would block GCU operation when it checks if the current config aligns with the YANG model:

Missing cluster field in YANG
Incomplete set of device type. The device type in YANG model doesn't include all the device type.
Work item tracking
Microsoft ADO (number only): 25577813
How I did it
Add cluster field in DEVICE_NEIGHBOR_METADATA YANG model.
Change device type to string.
Fix the UT test accordingly.
How to verify it
Build the image and verify the unit tests passed.

Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
@mssonicbld
Copy link
Collaborator

@Gfrom2016 PR conflicts with 202205 branch

yxieca pushed a commit that referenced this pull request Nov 28, 2023
#17049) (#17251)

Why I did it
The current DEVICE_NEIGHBOR_METADATA yang model has two issues that would block GCU operation when it checks if the current config aligns with the YANG model:

Missing cluster field in YANG
Incomplete set of device type. The device type in YANG model doesn't include all the device type.
Work item tracking
Microsoft ADO (number only): 25577813
How I did it
Add cluster field in DEVICE_NEIGHBOR_METADATA YANG model.
Change device type to string.
Fix the UT test accordingly.
How to verify it
Build the image and verify the unit tests passed.

Signed-off-by: zitingguo-ms <zitingguo@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants