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

[orchagent] Add SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes for DASH #2514

Merged
merged 1 commit into from
Nov 11, 2022

Conversation

prabhataravind
Copy link
Contributor

@prabhataravind prabhataravind commented Nov 11, 2022

What I did
Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow
Why I did it

How I verified it
Loaded DASH configuration referenced in sonic-net/sonic-mgmt#6735 and verified that SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes are added

Details if related
2022-11-11.18:59:49.362557|c|SAI_OBJECT_TYPE_ENI:oid:0x6d000000000018|SAI_ENI_ATTR_VNET_ID=oid:0x71000000000016|SAI_ENI_ATTR_ADMIN_STATE=false|SAI_ENI_ATTR_VM_UNDERLAY_DIP=ac5c:99c2:30c9:98eb:c617:f88f:7c5c:ea90|SAI_ENI_ATTR_VM_VNI=4321
2022-11-11.18:59:49.364037|c|SAI_OBJECT_TYPE_ENI:oid:0x6d000000000019|SAI_ENI_ATTR_VNET_ID=oid:0x71000000000017|SAI_ENI_ATTR_ADMIN_STATE=false|SAI_ENI_ATTR_VM_UNDERLAY_DIP=50.2.2.2|SAI_ENI_ATTR_VM_VNI=4321
2022-11-11.18:59:49.365416|c|SAI_OBJECT_TYPE_ENI:oid:0x6d00000000001a|SAI_ENI_ATTR_VNET_ID=oid:0x71000000000017|SAI_ENI_ATTR_ADMIN_STATE=false|SAI_ENI_ATTR_VM_UNDERLAY_DIP=d95d:46e5:d5b2:4ce0:2a3:d9d:44a4:f20b|SAI_ENI_ATTR_VM_VNI=4321
2022-11-11.18:59:49.366890|c|SAI_OBJECT_TYPE_ENI:oid:0x6d00000000001b|SAI_ENI_ATTR_VNET_ID=oid:0x71000000000016|SAI_ENI_ATTR_ADMIN_STATE=false|SAI_ENI_ATTR_VM_UNDERLAY_DIP=25.1.1.1|SAI_ENI_ATTR_VM_VNI=4321

Signed-off-by: Prabhat Aravind [email protected]

…o ENI entry

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
@prabhataravind prabhataravind changed the title Add SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes t… [orchagent] Add SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes for DASH Nov 11, 2022
@prabhataravind prabhataravind merged commit 7c3f76c into sonic-net:dash Nov 11, 2022
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Dec 21, 2022
…o ENI entry (sonic-net#2514)

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
Pterosaur pushed a commit to Pterosaur/sonic-swss that referenced this pull request Dec 23, 2022
…o ENI entry (sonic-net#2514)

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
Pterosaur pushed a commit to Pterosaur/sonic-swss that referenced this pull request Dec 23, 2022
…o ENI entry (sonic-net#2514)

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
theasianpianist pushed a commit to theasianpianist/sonic-swss that referenced this pull request Jul 21, 2023
…o ENI entry (sonic-net#2514)

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
theasianpianist pushed a commit to theasianpianist/sonic-swss that referenced this pull request Jul 25, 2023
…o ENI entry (sonic-net#2514)

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
theasianpianist pushed a commit to theasianpianist/sonic-swss that referenced this pull request Jul 27, 2023
…o ENI entry (sonic-net#2514)

* Added SAI_ENI_ATTR_VM_UNDERLAY_DIP and SAI_ENI_ATTR_VM_VNI attributes to support inbound routing flow

Signed-off-by: Prabhat Aravind <[email protected]>
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