Update HLD for PL scenario

sonic-net / DASH

Disaggregated APIs for SONiC Hosts

Apache License 2.0

78 stars 88 forks source link

Update HLD for PL scenario #509

Closed theasianpianist closed 5 months ago

theasianpianist commented 5 months ago

Add routing_appliance_id field to VNET mapping table
Update PL encoding format
Remove 'key' attribute in routing type table in favor of 'vni'
Fix SIP encodings in PL example

r12f commented 5 months ago

The routing appliance id might need to be updated later, so do we wait for that change to be finalized or we go first and change later? @prsunny

r12f commented 5 months ago

The routing appliance id might need to be updated later, so do we wait for that change to be finalized or we go first and change later? @prsunny

actually, np. this is more like a fix on the current HLD to make things aligned on northbound, not really change any behavior, design or SAI APIs. lgtm! thanks @theasianpianist !