Skip to content

[BFN] Extended Fans platform plugin implementation#5

Closed
KostiantynYarovyiBf wants to merge 2 commits intoakokhan:bfn_platform_api_updatefrom
KostiantynYarovyiBf:bfn_fans_api_platform
Closed

[BFN] Extended Fans platform plugin implementation#5
KostiantynYarovyiBf wants to merge 2 commits intoakokhan:bfn_platform_api_updatefrom
KostiantynYarovyiBf:bfn_fans_api_platform

Conversation

@KostiantynYarovyiBf
Copy link
Copy Markdown

@KostiantynYarovyiBf KostiantynYarovyiBf commented Dec 3, 2021

Why I did it

For sonic-mgmt Platform API tests to have data to compare with

How I did it

Run sonic-mgmt Platform API TestFanDrawerFans tests:

  1. test_get_model
  2. test_get_serial
  3. test_get_direction
  4. test_get_fans_target_speed

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

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

@KostiantynYarovyiBf KostiantynYarovyiBf marked this pull request as ready for review December 3, 2021 18:00
@akokhan akokhan removed the request for review from lguohan December 4, 2021 08:37
akokhan pushed a commit that referenced this pull request Dec 7, 2021
Also add out of tree pca9548 mux driver to use platform data to mapping i2c bus with front panel port.

Signed-off-by: Jakkapan Jangmuang <jjangmua@celestica.com>

Co-authored-by: Saikrishna Arcot <sarcot@microsoft.com>
akokhan pushed a commit that referenced this pull request Dec 17, 2021
#### What I did 

[sonic-linkmgrd][master] submodule update

6c6151b Fix unstable unit tests (state change handler wasn't invoked) (#8)
2f7dc0a support code diff coverage (#5)
83f0002 Force mux state switch to standby if triggered from Cli (#6)

signed-off-by: Jing Zhang zhangjing@microsoft.com
@akokhan akokhan closed this Dec 22, 2021
akokhan pushed a commit that referenced this pull request Oct 25, 2022
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