Skip to content

Handle outbound_direction_lookup for DASH_APPLIANCE_TABLE #3571

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

Merged
merged 3 commits into from
Mar 31, 2025

Conversation

prabhataravind
Copy link
Contributor

@prabhataravind prabhataravind commented Mar 26, 2025

What I did
Handle outbound_direction_lookup for DASH_APPLIANCE_TABLE.
Why I did it
To unblock vnet-vnet tests which relies on the default value of SAI_DASH_ENI_MAC_OVERRIDE_TYPE_SRC_MAC
How I verified it
By running vstests and sonic-mgmt tests and ensuring that the default value for SAI_DIRECTION_LOOKUP_ENTRY_ATTR_DASH_ENI_MAC_OVERRIDE_TYPE is set to SAI_DASH_ENI_MAC_OVERRIDE_TYPE_SRC_MAC when no value is specified.
Details if related

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

 * If no value is specified, use a default value of SAI_DASH_ENI_MAC_OVERRIDE_TYPE_SRC_MAC, else use the specified value.

 Ref: sonic-net/SONiC#1911

Signed-off-by: Prabhat Aravind <[email protected]>
@prabhataravind prabhataravind force-pushed the paravind/mac_override branch from 9b93d1f to 1bd5333 Compare March 28, 2025 02:21
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prabhataravind prabhataravind force-pushed the paravind/mac_override branch from aa7591c to c4bf94c Compare March 29, 2025 05:08
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prabhataravind
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).

@prabhataravind prabhataravind marked this pull request as ready for review March 30, 2025 02:01
@prabhataravind prabhataravind requested a review from prsunny as a code owner March 30, 2025 02:01
@prsunny prsunny merged commit 822ab91 into sonic-net:master Mar 31, 2025
15 checks passed
prabhataravind added a commit to prabhataravind/sonic-swss that referenced this pull request Apr 5, 2025
…3571)

* Handle outbound_direction_lookup for DASH_APPLIANCE_TABLE
 * If no value is specified, use a default value of SAI_DASH_ENI_MAC_OVERRIDE_TYPE_SRC_MAC, else use the specified value.

 Ref: sonic-net/SONiC#1911
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