fix bad parameter for gCrmOrch->incCrmAclUsedCounter in qosorch (sonic-net#830)

* fix bad parameter for gCrmOrch->incCrmAclUsedCounter in qosorch

* fix VS test failure for test_crm.py::test_CrmAcl FAILED

Author: JaredLius

orchagent/qosorch.cpp

@@ -728,7 +728,7 @@ sai_object_id_t QosOrch::initSystemAclTable()
     }
     SWSS_LOG_NOTICE("Create a system ACL table for ECN coloring");
 
-    gCrmOrch->incCrmAclUsedCounter(CrmResourceType::CRM_ACL_TABLE, (sai_acl_stage_t) attr.value.s32, SAI_ACL_BIND_POINT_TYPE_PORT);
+    gCrmOrch->incCrmAclUsedCounter(CrmResourceType::CRM_ACL_TABLE, SAI_ACL_STAGE_INGRESS, SAI_ACL_BIND_POINT_TYPE_PORT);
 
     for (auto& pair: gPortsOrch->getAllPorts())
     {

tests/test_crm.py

@@ -554,6 +554,8 @@ def test_CrmAcl(dvs, testlog):
 
     bind_ports = ["Ethernet0", "Ethernet4"]
 
+    old_table_used_counter = getCrmCounterValue(dvs, 'ACL_STATS:INGRESS:PORT', 'crm_stats_acl_table_used')
+
     # create ACL table
     ttbl = swsscommon.Table(db, "ACL_TABLE")
     fvs = swsscommon.FieldValuePairs([("policy_desc", "test"), ("type", "L3"), ("ports", ",".join(bind_ports))])
@@ -565,8 +567,9 @@ def test_CrmAcl(dvs, testlog):
     rtbl.set("test|acl_test_rule", fvs)
 
     time.sleep(2)
-
-    table_used_counter = getCrmCounterValue(dvs, 'ACL_STATS:INGRESS:PORT', 'crm_stats_acl_table_used')
+    
+    new_table_used_counter = getCrmCounterValue(dvs, 'ACL_STATS:INGRESS:PORT', 'crm_stats_acl_table_used')
+    table_used_counter = new_table_used_counter - old_table_used_counter
     assert table_used_counter == 1
 
     # get ACL table key
@@ -596,6 +599,7 @@ def test_CrmAcl(dvs, testlog):
 
     time.sleep(2)
 
-    table_used_counter = getCrmCounterValue(dvs, 'ACL_STATS:INGRESS:PORT', 'crm_stats_acl_table_used')
+    new_table_used_counter = getCrmCounterValue(dvs, 'ACL_STATS:INGRESS:PORT', 'crm_stats_acl_table_used')
+    table_used_counter = new_table_used_counter - old_table_used_counter
     assert table_used_counter == 0