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

[202205][yang]Add missing fields in PortChannel yang model #14232

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

dgsudharsan
Copy link
Collaborator

Porting fix #14045 to 202205. 202205 doesn't have fast_rate and hence only fallback is updated in yang model

Why I did it

Added Missing fields in sonic-portchannel yang model.
"fallback" is present in configuration schema but not in yang model. This leads to traceback when yang is validated

How I did it

Updated yang model

How to verify it

Added tests to verify

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

Part of the PR

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

@dgsudharsan dgsudharsan requested a review from qiluo-msft as a code owner March 14, 2023 01:04
@dgsudharsan dgsudharsan changed the title [yang]Add missing fields in PortChannel yang model [202205][yang]Add missing fields in PortChannel yang model Mar 14, 2023
@dgsudharsan dgsudharsan requested a review from ganglyu March 14, 2023 02:43
@dgsudharsan
Copy link
Collaborator Author

@ganglyu Can you please review this cherry-pick?

@qiluo-msft qiluo-msft merged commit 492b2b6 into sonic-net:202205 Mar 14, 2023
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