Skip to content

[doc] Use correct inventory file in CLI for deploying minigraph for VS seup#3195

Merged
wangxin merged 1 commit intosonic-net:masterfrom
wangxin:fix-doc
Mar 25, 2021
Merged

[doc] Use correct inventory file in CLI for deploying minigraph for VS seup#3195
wangxin merged 1 commit intosonic-net:masterfrom
wangxin:fix-doc

Conversation

@wangxin
Copy link
Copy Markdown
Collaborator

@wangxin wangxin commented Mar 24, 2021

Description of PR

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?

In current document for VS setup:
https://github.com/Azure/sonic-mgmt/blob/master/docs/testbed/README.testbed.VsSetup.md

Inventory file name lab is used in the example command line for deploying minigraph to VS setup:
$ ./testbed-cli.sh -t vtestbed.csv -m veos_vtb deploy-mg vms-kvm-t0 lab password.txt

The correct inventory file name for VS setup should be veos_vtb:
$ ./testbed-cli.sh -t vtestbed.csv -m veos_vtb deploy-mg vms-kvm-t0 veos_vtb password.txt

How did you do it?

Update inventory file name in example command line.

How did you verify/test it?

Any platform specific information?

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

Documentation

In current document for VS setup:
https://github.com/Azure/sonic-mgmt/blob/master/docs/testbed/README.testbed.VsSetup.md

Inventory file name `lab` is used in the example command line for deploying minigraph to VS setup:
    $ ./testbed-cli.sh -t vtestbed.csv -m veos_vtb deploy-mg vms-kvm-t0 lab password.txt

The correct inventory file name for VS setup should be `veos_vtb`:
    $ ./testbed-cli.sh -t vtestbed.csv -m veos_vtb deploy-mg vms-kvm-t0 veos_vtb password.txt

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@wangxin wangxin requested a review from a team March 24, 2021 02:38
@wangxin wangxin requested a review from yxieca as a code owner March 24, 2021 02:38
@wangxin wangxin merged commit 28ca9ef into sonic-net:master Mar 25, 2021
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
…c-net#3195)

In current document for VS setup:
https://github.com/Azure/sonic-mgmt/blob/master/docs/testbed/README.testbed.VsSetup.md

Inventory file name `lab` is used in the example command line for deploying minigraph to VS setup:
    $ ./testbed-cli.sh -t vtestbed.csv -m veos_vtb deploy-mg vms-kvm-t0 lab password.txt

The correct inventory file name for VS setup should be `veos_vtb`:
    $ ./testbed-cli.sh -t vtestbed.csv -m veos_vtb deploy-mg vms-kvm-t0 veos_vtb password.txt

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…atically (sonic-net#18331)

#### Why I did it
src/sonic-utilities
```
* c0ba32ad - (HEAD -> 202305, origin/202305) CLI to skip polling for periodic information for a port in DomInfoUpdateTask thread (sonic-net#3187) (16 hours ago) [mihirpat1]
* 261cfdf7 - CLI enhancements to revtrieve data from TRANSCEIVER_FIRMWARE_INFO table (sonic-net#3177) (sonic-net#3189) (19 hours ago) [mssonicbld]
* 6160ee79 - [202305][config] Add YANG alerting for override (sonic-net#3195) (20 hours ago) [jingwenxie]
* a55624d8 - [fast/warm-reboot] Put ERR message in syslog when a failure is seen (sonic-net#3186) (34 hours ago) [Vaibhav Hemant Dixit]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

2 participants