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

[fpmsyncd][WR][2305] Relax the static schema constraint for ROUTE_TABLE #2983

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

vivekrnv
Copy link
Contributor

@vivekrnv vivekrnv commented Dec 7, 2023

What I did

Cherry-pick #2981

Why I did it

How I verified it

Details if related

@vivekrnv vivekrnv force-pushed the fpmsyncd_wr_fix_2305 branch 2 times, most recently from 4fc6619 to 3456f5c Compare December 7, 2023 19:46
@dgsudharsan dgsudharsan marked this pull request as ready for review December 8, 2023 00:39
@dgsudharsan dgsudharsan requested a review from prsunny as a code owner December 8, 2023 00:39
@dgsudharsan
Copy link
Collaborator

@StormLiangMS Tested on top of 202305 commit sonic-net/sonic-buildimage@688245a

…nic-net#2981)

Remove the assert when the schema changes and in that case, imply a diff and pass it down to orchagent
@vivekrnv vivekrnv force-pushed the fpmsyncd_wr_fix_2305 branch from 504ae4a to 7ea7f1f Compare December 13, 2023 20:56
@liat-grozovik
Copy link
Collaborator

@prsunny @StormLiangMS could you please help to approve and merge this one?

@vivekrnv
Copy link
Contributor Author

/azpw run Azure.sonic-swss

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-swss

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@StormLiangMS StormLiangMS merged commit 1377bb4 into sonic-net:202305 Dec 15, 2023
13 of 14 checks passed
@vivekrnv vivekrnv deleted the fpmsyncd_wr_fix_2305 branch January 3, 2024 17:19
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.

5 participants