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.

Sign up for GitHub

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

Jump to bottom

[submodule][master] Update submodule sonic-mgmt-common and sonic-mgmt-framework to the latest HEAD automatically #17054

Merged
merged 1 commit into from
Nov 13, 2023
Merged

[submodule][master] Update submodule sonic-mgmt-common and sonic-mgmt-framework to the latest HEAD automatically #17054

merged 1 commit into from
Nov 13, 2023

Conversation

ranjinidn
Copy link
Contributor

Why I did it

sonic-mgmt-framework

  • dfac87c - (HEAD -> master, origin/master, origin/HEAD) Query parameters enhancements in rest-server. (#119) (3 weeks ago) [ranjinidn]

sonic-mgmt-common

  • a16b59b - (HEAD -> master, origin/master, origin/HEAD) Query Parameters Pruning API and Tests (#105) (3 weeks ago) [a-barboza]
  • 2dadfdc - DB Access Layer Merges: Address the review comments from GetTablePattern() (#106) (3 weeks ago) [a-barboza]
  • cdd83d7 - Query Parameter support in transformer infra for REST GET operation (#102) (5 weeks ago) [amrutasali]

How I did it

How to verify it

Description for the changelog

@ranjinidn ranjinidn requested a review from lguohan as a code owner October 31, 2023 20:46
@sachinholla
Copy link
Contributor

@qiluo-msft, please review this.. Automatic submodule update is failing due to interdependency between PRs. Hence updating together manually.

@ranjinidn ranjinidn mentioned this pull request Nov 2, 2023
Merged
liushilongbuaa
liushilongbuaa approved these changes Nov 2, 2023
View reviewed changes
Copy link
Contributor

@liushilongbuaa liushilongbuaa left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM.

@Praveen-Brcm Praveen-Brcm merged commit 5567a79 into sonic-net:master Nov 13, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Praveen-Brcm Praveen-Brcm Praveen-Brcm approved these changes

@anand-kumar-subramanian anand-kumar-subramanian anand-kumar-subramanian approved these changes

@sachinholla sachinholla sachinholla approved these changes

@liushilongbuaa liushilongbuaa liushilongbuaa approved these changes

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ranjinidn @sachinholla @liushilongbuaa @anand-kumar-subramanian @Praveen-Brcm