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

[submodule][202211] Update submodule to the latest HEAD automatically #14502

Closed
wants to merge 1 commit into from

Conversation

EdenGri
Copy link
Contributor

@EdenGri EdenGri commented Apr 3, 2023

Why I did it

src/sonic-swss

*b6d8f13 [flowcounterrouter] Fix the Route remove flow for non-bound prefixes (#2691)
*9f08cf1 [bfdorch] add default TOS value for BFD session (#2689)
*868db24 [EVPN]Handling error scenarios during route programming and IMR add (#2670)
*9d38fbc [ACL] Write ACL table/rule creation status into STATE_DB (#2662)
*317f43a - Fixed set admin_status for deleted subintf due to late notification (#2659)

How I did it

How to verify it

@EdenGri EdenGri requested a review from lguohan as a code owner April 3, 2023 15:06
@EdenGri
Copy link
Contributor Author

EdenGri commented Apr 4, 2023

Hey @bingwang-ms, the build failed with an error related to ACL aclorch.cpp:3422:39: error: 'STATE_ACL_TABLE_TABLE_NAME' was not declared in this scope : https://dev.azure.com/mssonic/build/_build/results?buildId=246142&view=logs&j=d37bc48d-29a0-534f-a1dc-3d699deb17a6&t=d31d2d46-3ae6-5432-200f-6da25f8a20cc&l=8120

It might be related to your changes? Can you please take a look?
Related commit:
*9d38fbc [ACL] Write ACL table/rule creation status into STATE_DB (#2662)

@bingwang-ms
Copy link
Contributor

Hey @bingwang-ms, the build failed with an error related to ACL aclorch.cpp:3422:39: error: 'STATE_ACL_TABLE_TABLE_NAME' was not declared in this scope : https://dev.azure.com/mssonic/build/_build/results?buildId=246142&view=logs&j=d37bc48d-29a0-534f-a1dc-3d699deb17a6&t=d31d2d46-3ae6-5432-200f-6da25f8a20cc&l=8120

It might be related to your changes? Can you please take a look? Related commit: *9d38fbc [ACL] Write ACL table/rule creation status into STATE_DB (#2662)

Thanks @EdenGri . Yes, it's because a cherry-pick of sonic-swss-common is missing in 202211 branch. I have raised a PR to fix the issue. sonic-net/sonic-swss-common#770

@EdenGri EdenGri closed this Apr 10, 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.

2 participants