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.csv] Add new fields #2822

Merged
merged 3 commits into from
Feb 18, 2021
Merged

[testbed.csv] Add new fields #2822

merged 3 commits into from
Feb 18, 2021

Conversation

lolyu
Copy link
Contributor

@lolyu lolyu commented Jan 15, 2021

Description of PR

Add two new fields to testbed.csv:

  • inv_name: specifies the lab inventory that defines the DUTs.
  • auto_recover: decide whether to recover the testbed in running
    recover_server.py

Signed-off-by: Longxiang Lyu [email protected]

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

For server recovery script, we need the new fields inv_name to decide which lab inventory to use in recover tasks(deploy-mg), and we need the new fields auto_recover to decide whether we should recover this testbed.

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

@lolyu lolyu requested a review from a team January 15, 2021 08:15
@lolyu lolyu added the Testbed label Jan 15, 2021
@bingwang-ms
Copy link
Collaborator

Retest this, please

@lguohan
Copy link
Contributor

lguohan commented Jan 31, 2021

/Azurepipelines run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

lolyu added 3 commits February 9, 2021 18:15
Add two new fields to `testbed.csv`:
- `inv_name`: specifies the lab inventory that defines the DUTs.
- `auto_recover`: decide whether to recover the testbed in running
		  `recover_server.py`

Signed-off-by: Longxiang Lyu <[email protected]>
Signed-off-by: Longxiang Lyu <[email protected]>
Signed-off-by: Longxiang Lyu <[email protected]>
@lolyu lolyu requested a review from wangxin as a code owner February 10, 2021 02:17
@lolyu lolyu merged commit a0836c8 into sonic-net:master Feb 18, 2021
@lolyu lolyu deleted the add_new_fields branch February 18, 2021 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants