Skip to content

Added changes for updating TestbedProcessing module#4221

Merged
wangxin merged 1 commit intosonic-net:masterfrom
OleksandrKozodoi:updating_testbedprocessing_module
Sep 11, 2021
Merged

Added changes for updating TestbedProcessing module#4221
wangxin merged 1 commit intosonic-net:masterfrom
OleksandrKozodoi:updating_testbedprocessing_module

Conversation

@OleksandrKozodoi
Copy link
Contributor

Signed-off-by: Oleksandr Kozodoi [email protected]

Description of PR

Summary:

  • Fixed incorrect name of num_asics value.
  • Added extra approach for getting os value from testbed-new.yaml.
  • Added extra approach for getting dev value from testbed dictionary.

Type of change

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

Back port request

  • 201911

Approach

What is the motivation for this PR?

Sometimes the device configuration is located in testbed dictionary of testbed-new.yaml module.
The fanouthosts fixture gets 'os' type from host_vars.

How did you do it?

Added the possibility for getting dev value from testbed dictionary.
Added the possibility for defining os type by the extra parameter os in testbed-new.yaml.
Example:
testbed-new.yaml:

devices:
    str-msn2700-01:
        os: sonic               
        device_type: DevSonic                       # source: sonic-mgmt/ansible/files/sonic_lab_devices-github.csv      

How did you verify/test it?

Any platform specific information?

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

Documentation

@OleksandrKozodoi OleksandrKozodoi requested a review from a team as a code owner September 9, 2021 16:37
@yozhao101
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wangxin wangxin merged commit b91870a into sonic-net:master Sep 11, 2021
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
Summary:

* Fixed incorrect name of num_asics value.
* Added extra approach for getting os value from testbed-new.yaml.
* Added extra approach for getting dev value from testbed dictionary.

What is the motivation for this PR?
Sometimes the device configuration is located in testbed dictionary of testbed-new.yaml module.
The fanouthosts fixture gets 'os' type from host_vars.

How did you do it?
Added the possibility for getting dev value from testbed dictionary.
Added the possibility for defining os type by the extra parameter os in testbed-new.yaml.
Example:
testbed-new.yaml:

devices:
    str-msn2700-01:
        os: sonic               
        device_type: DevSonic                       # source: sonic-mgmt/ansible/files/sonic_lab_devices-github.csv 

Signed-off-by: Oleksandr Kozodoi <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (#25405)

#### Why I did it
src/sonic-swss
```
* 13227d02 - (HEAD -> 202511, origin/202511) [countersyncd]: Add communication statistics recording and utilities (sonic-net#4222) (2 days ago) [mssonicbld]
* 3c4d3b2b - [countersyncd]: Add retry between client and otel collector (sonic-net#4220) (3 days ago) [mssonicbld]
* 77acf5a0 - [countersyncd] fix otel actor log level (sonic-net#4221) (3 days ago) [mssonicbld]
* 03ec77c7 - [countersyncd]: Add benchmark suite for countersyncd and optimize otel actor (sonic-net#4216) (5 days ago) [mssonicbld]
* 08050f2e - [hft]: Fix TAM type capability enable list (sonic-net#4215) (6 days ago) [mssonicbld]
* d0793b45 - [Fixbug]: Fix delete default HFT configuration issue (sonic-net#4138) (7 days ago) [mssonicbld]
* 246d9575 - [hft]: Enable output queue for HFT (sonic-net#4187) (7 days ago) [mssonicbld]
* ae6a9887 - [countersyncd]: Fix netlink fd leakage and deadlock issue (sonic-net#4191) (7 days ago) [mssonicbld]
* c468e1fc - [countersyncd]: Fix compiling warning of otel (sonic-net#4192) (7 days ago) [mssonicbld]
* d675062c - Enabling the FEC histogram for gbsyncd counters (sonic-net#4195) (9 days ago) [mssonicbld]
```
#### 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.

3 participants