Skip to content

Commit

Permalink
[dash]: Don't attempt to bind empty ACL groups (sonic-net#2613)
Browse files Browse the repository at this point in the history
* [dash]: Don't attempt to bind empty ACL groups

Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
  • Loading branch information
theasianpianist committed Jul 25, 2023
1 parent 2eb306a commit 40a23b9
Show file tree
Hide file tree
Showing 2 changed files with 213 additions and 112 deletions.
7 changes: 7 additions & 0 deletions orchagent/dash/dashaclorch.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -761,6 +761,13 @@ task_process_status DashAclOrch::bindAclToEni(DashAclTable &acl_table, const str
return task_need_retry;
}

if (acl_group->m_rule_count <= 0)
{
SWSS_LOG_INFO("acl group %s contains 0 rules, waiting for rule creation", acl.m_acl_group_id->c_str());
acl.m_acl_group_id.reset();
return task_need_retry;
}

attr.id = getSaiStage(direction, *(acl_group->m_ip_version), stage);
attr.value.oid = acl_group->m_dash_acl_group_id;
}
Expand Down
Loading

0 comments on commit 40a23b9

Please sign in to comment.