[SDFAB-857] Add drop action in routing table to support blackhole routes

stratum / fabric-tna

The SD-Fabric data plane

https://docs.sd-fabric.org/

30 stars 15 forks source link

[SDFAB-857] Add drop action in routing table to support blackhole routes #439

Closed Wailoks closed 2 years ago

Wailoks commented 2 years ago

[X] Add drop action to forwarding p4
[X] Modify treatment interpreter
[X] Check stage dependency (p4i)
[x] Unit tests
[x] Re-test with black hole scenario (ptf)

codecov[bot] commented 2 years ago

The author of this PR, Wailoks, is not an activated member of this organization on Codecov. Please activate this user on Codecov to display this PR comment. Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations. Please don't hesitate to email us at success@codecov.io with any questions.

Wailoks commented 2 years ago

https://docs.google.com/document/d/17Yw-882cGY-Vz9yAFYzuOg0w_VTk5oKKHqSIk7YQTpo/edit#

pierventre commented 2 years ago

retest this please

pierventre commented 2 years ago

@charlesmcchan @ccascone does it sound reasonable to have this FIXME in the ACL actions ?

ccascone commented 2 years ago

@charlesmcchan @ccascone does it sound reasonable to have this FIXME in the ACL actions ?

Yes makes sense, no need to solve now