Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[testbed_doc] Design doc for deploying a single testbed with multiple servers #15395

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

w1nda
Copy link
Member

@w1nda w1nda commented Nov 6, 2024

Description of PR

Summary:
Fixes # (issue)
To leverage the servers instead of a single server for deploying a single testbed, we proposal this design for deploying testbeds with multiple servers.

Related PRs:

PR title State Context
[testbed-cli] Code change on add-topo and deploy-minigraph for deploying testbed with peers on multiple servers state context
[Pending #15643][testbed] ptf data plane connection for multi-servers testbed state context
[sonic-mgmt-docker-image] Support ptf dataplane packet poll with multiple ptf nn agents connection state context

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405

Approach

What is the motivation for this PR?

When deploying a testbed with a vast number of virtual ceos neighbors, we will create ceos containers on same server, however, the server doesn't have infinite resources such as memory to deploy that.
To leverage the servers instead of a single server, we proposal this design for deploying testbeds with multiple servers.

How did you do it?

Design for deploying testbed with multiple servers.

How did you verify/test it?

Any platform specific information?

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

Documentation

@w1nda w1nda requested review from wangxin and yxieca as code owners November 6, 2024 15:13
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@w1nda w1nda requested review from r12f, Blueve and sdszhang and removed request for wangxin and yxieca November 6, 2024 15:14
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
docs/testbed/README.testbed.BgpPeersNumberScaling.md Outdated Show resolved Hide resolved
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

wangxin pushed a commit that referenced this pull request Jan 20, 2025
…ing testbed with peers on multiple servers (#15643)

In PR #15547, we define server index in topology file which make topology file and testbed yaml file is coupled.
To decouple them, in new design #15395, we remove server index in topology file and add dut_interfaces in testbed yaml, so there is no more change on topology file schema.

What is the motivation for this PR?
Decouple topology file and testbed file in design.

How did you do it?
Record all information in testbed schema

How did you verify/test it?
Deploy testbed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 In Plan Features
Development

Successfully merging this pull request may close these issues.

5 participants