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

[Celestica DX010] fix fan drawer and watchdog platform testcase issues #13426

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

qnos
Copy link
Contributor

@qnos qnos commented Jan 19, 2023

Why I did it

fix DX010 fan drawer and watchdog platform test case failure issues

How I did it

  1. Add fan_drawer get_maximum_consumed_power support
  2. Adjust maximum watchdog timeout value check

How to verify it

Run test_fan_drawer and test_watchdog test cases.

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

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

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

Why I did it
fix DX010 fan drawer and watchdog platform test case issues

How I did it
1. Add fan_drawer get_maximum_consumed_power support
2. Adjust maximum watchdog timeout value check

How to verify it
Run test_fan_drawer and test_watchdog test cases.
@qnos
Copy link
Contributor Author

qnos commented Jan 19, 2023

@prgeor No reviewer assigned, please help to review it. Thanks

@prgeor
Copy link
Contributor

prgeor commented Feb 3, 2023

@lguohan please help merge.

@StormLiangMS @yxieca please cherry pick

@lguohan lguohan merged commit 1dec473 into sonic-net:master Feb 6, 2023
qiluo-msft pushed a commit that referenced this pull request Feb 8, 2023
#13426)

Why I did it
fix DX010 fan drawer and watchdog platform test case issues

How I did it
1. Add fan_drawer get_maximum_consumed_power support
2. Adjust maximum watchdog timeout value check

How to verify it
Run test_fan_drawer and test_watchdog test cases.
@prgeor
Copy link
Contributor

prgeor commented Feb 9, 2023

@StormLiangMS @yxieca please cherry pick to 202205, 202211

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Feb 9, 2023
sonic-net#13426)

Why I did it
fix DX010 fan drawer and watchdog platform test case issues

How I did it
1. Add fan_drawer get_maximum_consumed_power support
2. Adjust maximum watchdog timeout value check

How to verify it
Run test_fan_drawer and test_watchdog test cases.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #13747

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Feb 10, 2023
sonic-net#13426)

Why I did it
fix DX010 fan drawer and watchdog platform test case issues

How I did it
1. Add fan_drawer get_maximum_consumed_power support
2. Adjust maximum watchdog timeout value check

How to verify it
Run test_fan_drawer and test_watchdog test cases.
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #13770

yxieca pushed a commit that referenced this pull request Feb 10, 2023
#13426) (#13747)

Why I did it
fix DX010 fan drawer and watchdog platform test case issues

How I did it
1. Add fan_drawer get_maximum_consumed_power support
2. Adjust maximum watchdog timeout value check

How to verify it
Run test_fan_drawer and test_watchdog test cases.

Co-authored-by: Ikki Zhu <[email protected]>
mssonicbld pushed a commit that referenced this pull request Feb 10, 2023
#13426)

Why I did it
fix DX010 fan drawer and watchdog platform test case issues

How I did it
1. Add fan_drawer get_maximum_consumed_power support
2. Adjust maximum watchdog timeout value check

How to verify it
Run test_fan_drawer and test_watchdog test cases.
@prgeor
Copy link
Contributor

prgeor commented Feb 15, 2023

@assrinivasan could you help check if this PR is present in internal branch or not?

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.

8 participants