Skip to content

vpp: relax free memory check for vpp #23359

Open
yue-fred-gao wants to merge 2 commits intosonic-net:masterfrom
yue-fred-gao:relax_free_mem
Open

vpp: relax free memory check for vpp #23359
yue-fred-gao wants to merge 2 commits intosonic-net:masterfrom
yue-fred-gao:relax_free_mem

Conversation

@yue-fred-gao
Copy link
Copy Markdown
Contributor

Description of PR

Summary:
Fixes # (issue)
relax free memory check for vpp because vpp dataplane runs in host

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

Since VPP runs the data plane in the host, adding/deleting routes consumes significantly more host memory than on hardware ASICs.

How did you do it?

For vpp asic, relax free memory check to warning. The next step, we should introduce a new hwsku for vpp so it can tune the memory usage by platform, similar to other hardware platforms.

How did you verify/test it?

run sonic-mgmt

Any platform specific information?

vpp

Supported testbed topology if it's a new test case?

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@yue-fred-gao
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

⚠️ Notice: /azpw run only runs failed jobs now. If you want to trigger a whole pipline run, please rebase your branch or close and reopen the PR.
💡 Tip: You can also use /azpw retry to retry failed jobs directly.

Retrying failed(or canceled) jobs...

@mssonicbld
Copy link
Copy Markdown
Collaborator

No Azure DevOps builds found for #23359.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants