Skip to content

Conversation

@AkeelAli
Copy link
Contributor

Changes in VPP's SAI ACL code to successfully run sonic-mgmt ACL testing on Sonic-VPP.

Related PRs:
Sonic-mgmt changes: sonic-net/sonic-mgmt#18313
Sonic-VPP changes: sonic-net/sonic-platform-vpp#178

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kcudnik
Copy link
Collaborator

kcudnik commented Jun 12, 2025

there are no unittests for this code

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yue-fred-gao
Copy link
Contributor

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yue-fred-gao
Copy link
Contributor

/azpw run Azure.sonic-sairedis (BuildAsan amd64)

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-sairedis (BuildAsan amd64)

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yue-fred-gao yue-fred-gao merged commit b70dfa7 into sonic-net:master Dec 9, 2025
16 checks passed
vikram-nexthop pushed a commit to nexthop-ai/sonic-sairedis that referenced this pull request Dec 12, 2025
* Enabling sonic-mgmt ACL testing for Sonic-VPP

* Use retval in debugs

Signed-off-by: Vikram Menon <vikram@nexthop.ai>
croos12 pushed a commit to croos12/sonic-sairedis that referenced this pull request Dec 14, 2025
* Enabling sonic-mgmt ACL testing for Sonic-VPP

* Use retval in debugs
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.

4 participants