Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix LGTM alerts
Browse files Browse the repository at this point in the history
Jostar Yang committed Jun 30, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent 87aa7cd commit 0865106
Showing 2 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -10,7 +10,6 @@
import sys
import time
from sonic_platform_pddf_base.pddf_chassis import PddfChassis
from sonic_platform.component import Component
except ImportError as e:
raise ImportError(str(e) + "- required module not found")

Original file line number Diff line number Diff line change
@@ -211,8 +211,8 @@ def manage_fans(self):
logging.critical("Alarm for all fan faulty/absent is detected, reset DUT")
cmd_str = "i2cset -y -f 3 0x60 0x4 0xE4"
time.sleep(2)
status, output = subprocess.getstatusoutput('sync; sync; sync')
status, output = subprocess.getstatusoutput(cmd_str)
subprocess.getstatusoutput('sync; sync; sync')
subprocess.getstatusoutput(cmd_str)
elif sum(fan_fail_list) != 0:
# Set the 100% speed only for first fan failure detection
logging.warning('Fan_{} failed, set remaining fan speed to 100%'.format(

0 comments on commit 0865106

Please sign in to comment.