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

[BFN] Reworked BFN platform thermals plugin #11723

Merged
merged 5 commits into from
Oct 11, 2022

Conversation

akokhan
Copy link
Contributor

@akokhan akokhan commented Aug 12, 2022

Signed-off-by: Andriy Kokhan [email protected]

Why I did it

  • To reduce an access to I2C bus
  • To speed-up thermals polling time
  • To fix platform.json info as per wedge100bf_65x requirements

How I did it

Added local thermal cache that is updated on read once in >5sec

How to verify it

show platform temperature
show platform firmware status

Run SONiC platform CTs

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

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

@akokhan akokhan requested a review from lguohan as a code owner August 12, 2022 17:14
@akokhan akokhan force-pushed the wedge100bf_65x_platform branch from b2315f3 to f8968d8 Compare August 13, 2022 05:59
@akokhan
Copy link
Contributor Author

akokhan commented Aug 16, 2022

@lguohan , please approve and merge.

@akokhan akokhan changed the title [BFN] Updated platform.json for wedge100bf_65x [BFN] Reworked BFN platform thermals plugin Aug 18, 2022
@akokhan akokhan force-pushed the wedge100bf_65x_platform branch 2 times, most recently from 6954090 to a844be7 Compare August 20, 2022 07:08
@akokhan
Copy link
Contributor Author

akokhan commented Aug 20, 2022

@prgeor , @jleveque , please approve and merge. Thank you.

@akokhan akokhan force-pushed the wedge100bf_65x_platform branch 6 times, most recently from 928a224 to 94c0373 Compare August 24, 2022 09:04
@akokhan
Copy link
Contributor Author

akokhan commented Aug 25, 2022

@prgeor , please review

@akokhan
Copy link
Contributor Author

akokhan commented Aug 25, 2022

@jleveque , please review

@akokhan
Copy link
Contributor Author

akokhan commented Aug 26, 2022

@lguohan , please approve and merge

@akokhan
Copy link
Contributor Author

akokhan commented Aug 30, 2022

@lguohan , could you please take a look? Thanks

@akokhan
Copy link
Contributor Author

akokhan commented Sep 1, 2022

@prgeor , could you please take a look? Thank you

@akokhan
Copy link
Contributor Author

akokhan commented Sep 6, 2022

@lguohan , @prgeor , could you please take a look? Thanks

@akokhan
Copy link
Contributor Author

akokhan commented Sep 14, 2022

@prgeor , please review

@akokhan
Copy link
Contributor Author

akokhan commented Sep 14, 2022

@lguohan , please review

@akokhan
Copy link
Contributor Author

akokhan commented Sep 14, 2022

@prsunny , could you please take a look? Thanks

@akokhan akokhan force-pushed the wedge100bf_65x_platform branch from 85ca533 to 74dd01a Compare September 22, 2022 14:10
@akokhan
Copy link
Contributor Author

akokhan commented Sep 26, 2022

@lguohan , @prgeor , this PR contains Intel platform specific changes only. Please approve and merge.

@akokhan
Copy link
Contributor Author

akokhan commented Sep 29, 2022

@Azure/sonic-platform, this PR contains Intel platform specific changes only. Please approve and merge.

@akokhan akokhan force-pushed the wedge100bf_65x_platform branch 2 times, most recently from 6751c12 to b17ee38 Compare September 30, 2022 16:31
@akokhan akokhan force-pushed the wedge100bf_65x_platform branch 2 times, most recently from cfb9450 to fe5af04 Compare October 2, 2022 08:09
@akokhan akokhan force-pushed the wedge100bf_65x_platform branch 6 times, most recently from da37778 to cf4dcb4 Compare October 7, 2022 06:35
@prgeor
Copy link
Contributor

prgeor commented Oct 7, 2022

@akokhan can you rebase your changes?

@akokhan akokhan force-pushed the wedge100bf_65x_platform branch 4 times, most recently from c85ddf0 to 8557c96 Compare October 10, 2022 11:24
@richardyu-ms richardyu-ms merged commit 304c6c8 into sonic-net:master Oct 11, 2022
renukamanavalan added a commit that referenced this pull request Oct 12, 2022
| * cb707b7 fixed nokia platform m0 asic mismatch (fixed nokia platform m0 asic mismatch #12148)
| * c75dfe8 [build] Fix dpkg front lock issue with apt-get ([build] Fix dpkg front lock issue caused by apt-get install #12332)
| * 247bd78 [action] Fix PR pre-cherry-pick action wrong author issue. ([action] Fix PR pre-cherry-pick action wrong author issue. #12339)
| * fc99265 [Nokia] Update the nokia platform submodule for Nokia-IXR7250E platform ([Nokia] Update the nokia platform submodule for Nokia-IXR7250E platform #12305)
| * 9d37b63 [Nokia] Update Nokia platform IXR7250E device data ([Nokia] Update Nokia platform IXR7250E device data #11611)
| * 304c6c8 [BFN] Reworked BFN platform thermals plugin ([BFN] Reworked BFN platform thermals plugin #11723)
| * df93a1b [Build][Bug] Fix apt-get remove version not lock issue ([Build][Bug] Fix apt-get remove version not locked issue #12193)
| * 9b2b8e3 Add gearbox taps to vs gearbox_config.json (Add gearbox taps to vs gearbox_config.json #11480)
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