Skip to content

Conversation

@cel-gl
Copy link
Contributor

@cel-gl cel-gl commented Dec 3, 2025

Pre-submission checklist
[heshiw@AZUHPS26 fboss]$ git commit -m "Add th6 getMaxAclEntries value in AclScaleTestUtils.h"
clang-format.............................................................Passed
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check json...........................................(no files to check)Skipped
check for merge conflicts................................................Passed

Summary

AgentNSFScaleTest.Test test fail on icetea and other th6 platforms, root cause getMaxAclEntries doesn't support th6 , after modify as below show, and add acl group related configs in agent, the case passed

Test Plan

config change : "enable_acl_table_group": "true",

add
"aclTableGroup": {
"name": "acl-table-group-ingress",
"aclTables": [
{
"name": "AclTable1",
"priority": 0,
"aclEntries": [

  ],
  "actionTypes": [
    
  ],
  "qualifiers": [
    
  ],
  "udfGroups": [
    
  ]
}

],
"stage": 0
},
"aclTableGroups": [
{
"name": "acl-table-group-ingress",
"aclTables": [
{
"name": "AclTable1",
"priority": 0,
"aclEntries": [

    ],
    "actionTypes": [
      
    ],
    "qualifiers": [
      
    ],
    "udfGroups": [
      
    ]
  }
],
"stage": 0

}
],

tested on icetea passed

@meta-cla meta-cla bot added the CLA Signed label Dec 3, 2025
@cel-gl
Copy link
Contributor Author

cel-gl commented Dec 9, 2025

@huruinan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant