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

Mocking path early so it will applied to sonic_py_common, mock more swsscommon classes #187

Merged
merged 1 commit into from
May 21, 2021

Conversation

qiluo-msft
Copy link
Contributor

Description

Existing test will mix true swsscommon package with the mocked one in sonic_py_common

Motivation and Context

This is blocking sonic-net/sonic-buildimage#7655

How Has This Been Tested?

Unit test

Additional Information (Optional)

@qiluo-msft qiluo-msft marked this pull request as ready for review May 21, 2021 21:29
@qiluo-msft qiluo-msft requested a review from jleveque May 21, 2021 21:29
@qiluo-msft qiluo-msft merged commit 9ba52a2 into sonic-net:master May 21, 2021
@qiluo-msft qiluo-msft deleted the qiluo/mockmore branch May 21, 2021 22:24
qiluo-msft added a commit that referenced this pull request May 24, 2021
…chassis (#188)

#### Description
Following #187 to fix the same issue with other packages

#### Motivation and Context
This is blocking sonic-net/sonic-buildimage#7655

#### How Has This Been Tested?
Unit test
andywongarista pushed a commit to andywongarista/sonic-platform-daemons that referenced this pull request Jun 30, 2021
…common classes (sonic-net#187)

#### Description
Existing test will mix true swsscommon package with the mocked one in sonic_py_common

#### Motivation and Context
This is blocking sonic-net/sonic-buildimage#7655

#### How Has This Been Tested?
Unit test
andywongarista pushed a commit to andywongarista/sonic-platform-daemons that referenced this pull request Jun 30, 2021
…chassis (sonic-net#188)

#### Description
Following sonic-net#187 to fix the same issue with other packages

#### Motivation and Context
This is blocking sonic-net/sonic-buildimage#7655

#### How Has This Been Tested?
Unit test
vdahiya12 pushed a commit to vdahiya12/sonic-platform-daemons that referenced this pull request Apr 4, 2022
…net#187)

Add a stop function to thermal manager. Allow thermalctld to call the stop function to stop run_policy function fast
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants