Skip to content

Commit c863d48

Browse files
oleksandrivantsivtheasianpianist
authored andcommitted
[dash] Do not use an action drop with the inbound routing table. (#2710)
* [dash] Do not use an action drop with the inbound routing table. (#2710)
1 parent 89ce4e0 commit c863d48

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

orchagent/dash/dashrouteorch.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,6 @@ bool DashRouteOrch::addInboundRouting(const string& key, InboundRoutingBulkConte
344344
}
345345

346346
sai_inbound_routing_entry_t inbound_routing_entry;
347-
bool deny = !ctxt.action_type.compare("drop");
348347

349348
inbound_routing_entry.switch_id = gSwitchId;
350349
inbound_routing_entry.eni_id = dash_orch_->getEni(ctxt.eni)->eni_id;
@@ -358,8 +357,7 @@ bool DashRouteOrch::addInboundRouting(const string& key, InboundRoutingBulkConte
358357
vector<sai_attribute_t> inbound_routing_attrs;
359358

360359
inbound_routing_attr.id = SAI_INBOUND_ROUTING_ENTRY_ATTR_ACTION;
361-
inbound_routing_attr.value.u32 = deny ? SAI_INBOUND_ROUTING_ENTRY_ACTION_DENY : (ctxt.pa_validation ?
362-
SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP_PA_VALIDATE : SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP);
360+
inbound_routing_attr.value.u32 = ctxt.pa_validation ? SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP_PA_VALIDATE : SAI_INBOUND_ROUTING_ENTRY_ACTION_VXLAN_DECAP;
363361
inbound_routing_attrs.push_back(inbound_routing_attr);
364362

365363
if (!ctxt.vnet.empty())

0 commit comments

Comments
 (0)