facebook / fboss

Facebook Open Switching System Software for controlling network switches.
Other
866 stars 301 forks source link

platformmanager: fix publish firmware version issue #237

Closed QiuyunXie closed 1 month ago

QiuyunXie commented 1 month ago

Description

On Montblanc & Tahan & Janga device, running platform_manager will report firmware version read failed log as below image

When one CPLD has hwmon sensor path, but the cpld_ver/cpld_sub_ver file isn't at hwnon folder path. It will fail to find the cpld_ver/cpld_sub_ver file. Fix this issue, it's also suitable for "/run/devmap/cplds/FAN0_CPLD/hwmon/hwmon20/" case.

Test log

image

facebook-github-bot commented 1 month ago

@mikechoifb has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.