Skip to content

Port table should be ProducerTable#556

Merged
qiluo-msft merged 2 commits intosonic-net:masterfrom
qiluo-msft:qiluo/porttable
Aug 2, 2018
Merged

Port table should be ProducerTable#556
qiluo-msft merged 2 commits intosonic-net:masterfrom
qiluo-msft:qiluo/porttable

Conversation

@qiluo-msft
Copy link
Copy Markdown
Contributor

The entries in APP_PORT_TABLE_NAME are sequence sensitive because there are event entries (PortConfigDone, PortInitDone). Change it back to ProducerTable.

Copy link
Copy Markdown
Contributor

@jipanyang jipanyang left a comment

Choose a reason for hiding this comment

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

Could you update the VS test cases which use ProducerStateTable for app port table? #Resolved

Signed-off-by: Qi Luo <[email protected]>
Copy link
Copy Markdown
Contributor

@jipanyang jipanyang left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks.

@qiluo-msft qiluo-msft merged commit 944a80d into sonic-net:master Aug 2, 2018
@qiluo-msft qiluo-msft deleted the qiluo/porttable branch August 2, 2018 16:05
qiluo-msft added a commit that referenced this pull request Aug 4, 2018
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
* Port table should be ProducerTable
* Fix vs test
jianyuewu pushed a commit to jianyuewu/sonic-swss that referenced this pull request Dec 24, 2025
…Orch (sonic-net#556)

* Add AppDB ACL schema values shared between the P4RT app and P4Orch
* Add gmock
* Add libgmock to pipelines test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants