Skip to content

[tests/vxlan]: Fixed VxLAN default parameters and scale limits - #1376

Open
Aakash Tiwari (aakashtiwari2630-max) wants to merge 1 commit into
Azure:202511_azdfrom
aakashtiwari2630-max:aaktiwar/vxlan_mgmt
Open

[tests/vxlan]: Fixed VxLAN default parameters and scale limits#1376
Aakash Tiwari (aakashtiwari2630-max) wants to merge 1 commit into
Azure:202511_azdfrom
aakashtiwari2630-max:aaktiwar/vxlan_mgmt

Conversation

@aakashtiwari2630-max

Copy link
Copy Markdown

Porting master PR: sonic-net/sonic-mgmt#25063

Summary of fix:

  1. Currently the defaults set in conftest were pretty high and readjusting the defaults in the test doesn't work since default is already set. So, the correct approach is to fix the defaults in conftest.
  2. The ECMP endpoint tests currently doesn't check for maximum scale limits. This has problem since configuring higher limit will fail the test and subsequent tests. Added logic to check for platform limits.
  3. Currently there is no cleanup of vnet route after each test which could have issues in case we run the whole suite and a particular test fails. Added clean_vnet_route which has scope function so that it runs after each test and every test runs with a clean slate.
  4. Currently, there is no logic to reconfigure vxlan router mac after config reload. Due to this, tests will fail. Added APIs to reprogram router mac after reload.
  5. In VxLAN scale tests, currently the sleep is fixed as 20s. This should be dependent upon the route scale. Added scale aware logic in sleep.

Signed-off-by: Aakash Tiwari <aaktiwar@cisco.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant