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

[saisubmodule] Update SAI submodule to include route group changes #1410

Closed

Conversation

theasianpianist
Copy link
Contributor

@theasianpianist theasianpianist commented Aug 7, 2024

  • Advancing submodule pointer to include most recent DASH changes (PL, route group)
  • Also update all outbound routing entries to use outbound_routing_group_id instead of eni_id (and related tests)
  • Update unit tests to use new route group API

Includes the following commits:

18ba20f [DASH] Add Routing Group API (#2026)
abc8f02 [DASH] Add missing prefixes to PL mapping (#2035)

Signed-off-by: Lawrence Lee <[email protected]>
kcudnik
kcudnik previously approved these changes Aug 7, 2024
- Outbound routing entries now use "outbound_routing_group_id" instead of "eni_id"

Signed-off-by: Lawrence Lee <[email protected]>
@theasianpianist theasianpianist changed the title [saisubmodule] Update SAI submodule to latest master [saisubmodule] Update SAI submodule to include route group changes Aug 7, 2024
Signed-off-by: Lawrence Lee <[email protected]>
@theasianpianist
Copy link
Contributor Author

closing in favor of #1409

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