Skip to content

Add support for SEND_TO_INGRESS port table. #2816

Merged
prsunny merged 20 commits intosonic-net:masterfrom
baxia-lan:master
Oct 12, 2023
Merged

Add support for SEND_TO_INGRESS port table. #2816
prsunny merged 20 commits intosonic-net:masterfrom
baxia-lan:master

Conversation

@baxia-lan
Copy link
Copy Markdown
Contributor

What I did
Implement support for SEND_TO_INGRESS port table in CONFIG DB.

HLD: https://github.com/sonic-net/SONiC/blob/master/doc/pins/send_to_ingress_hld.md
Required swss-common PR: sonic-net/sonic-swss-common#792
Why I did it

How I verified it
Add below entry in config_db.json to use it.

    "SEND_TO_INGRESS_PORT": {
        "SEND_TO_INGRESS":{}
    },

The APPL DB(0) and APPL STATE DB(14)(previously 63) will have the entry added
image

Details if related

@baxia-lan baxia-lan requested a review from prsunny as a code owner June 8, 2023 01:18
@baxia-lan
Copy link
Copy Markdown
Contributor Author

@StephenWangGoogle
PTAL

@baxia-lan
Copy link
Copy Markdown
Contributor Author

@mint570

PTAL

@baxia-lan
Copy link
Copy Markdown
Contributor Author

swss-common PR sonic-net/sonic-swss-common#792 was submitted

@baxia-lan
Copy link
Copy Markdown
Contributor Author

@prsunny
Hi Prince, could you please take a look? Thanks!

@baxia-lan
Copy link
Copy Markdown
Contributor Author

@prsunny
Hi Prince, could you please take a look? Thanks!

@prsunny prsunny requested a review from marian-pritsak July 18, 2023 20:28
@baxia-lan
Copy link
Copy Markdown
Contributor Author

/AzurePipelines run Azure.sonic-swss

@azure-pipelines
Copy link
Copy Markdown

No commit pushedDate could be found for PR 2816 in repo sonic-net/sonic-swss

@baxia-lan
Copy link
Copy Markdown
Contributor Author

The pytest failures(nexthop and acl) do not seem to related to the CL. cc. @prsunny and @marian-pritsak

@baxia-lan
Copy link
Copy Markdown
Contributor Author

Kindly ping for review @prsunny @marian-pritsak

@baxia-lan
Copy link
Copy Markdown
Contributor Author

Kindly ping for review/suggestions @prsunny @marian-pritsak

Copy link
Copy Markdown
Collaborator

@prsunny prsunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. @dgsudharsan , can you take a look?

@prsunny prsunny merged commit cb98893 into sonic-net:master Oct 12, 2023
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
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