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

Sonic pit #12224

Closed
wants to merge 2 commits into from
Closed

Conversation

clarklee-guizhao
Copy link
Contributor

Why I did it

sonic-pit is originated from S3IP(Simplify Switch System Integration Program) PIT(Platform Integration Test). It's designed to verify switch devices in the process of development, manufacturing and delivering stages. It focus on 2 roles: ODM and user. ODM may use PIT in developing and manufaturing stages, user may use PIT in delivery stage for acceptance test. It aims to provide an verify system for siwtch devices' hardware, drivers and firmware. More information, see HLD: https://github.com/clarklee-guizhao/SONiC/blob/pit/doc/pit/Platform_Integration_Test_high_level_design.md

How I did it

Porting sonic-pit from S3IP-PIT, make the original code clean enough, add PIT source as sonic-pit, add PIT running environment as docker-pit.

How to verify it

Build docker docker-pit, deploy it onto devices which support sonic-pit, run pit.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Add PIT, whihc include sonic-pit(the PIT system) and docker-pit(the running environment for sonic-pit).

Ensure to add label/tag for the feature raised.

PR#1014Add PIT HLD document.

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 29, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

@lgtm-com
Copy link

lgtm-com bot commented Sep 29, 2022

This pull request introduces 95 alerts when merging 90d15a5 into 1f9c89a - view on LGTM.com

new alerts:

  • 27 for Variable defined multiple times
  • 25 for Unused import
  • 20 for 'import *' may pollute namespace
  • 9 for Except block handles 'BaseException'
  • 6 for Module is imported with 'import' and 'import from'
  • 5 for Unused local variable
  • 3 for Testing equality to None

@clarklee-guizhao
Copy link
Contributor Author

Close this PR, replace it with a several break-down PRs. Replacement PRs initial PR: #12441. The reason is break-down PRs may be easier for review.

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.

1 participant