Meru800biac: add PM, fan_service, sensor_service and wedge_qsfp_util #502
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Jul 30, 2025
huyc-arista
changed the title
harshitgulati18
approved these changes
Aug 5, 2025
facebook-github-bot
pushed a commit
that referenced
this pull request
Aug 9, 2025
Summary: # Description > NOTE: This is part of a series of PRs to add support for Meru800biac. The dependency chain of the PRs is as follows: > > Meru800biac: define new platform, add platform and port mapping #500 > Meru800biac: add bsp mapping, platform mapping, qsfp_service, led_service #501 - Depends on #500 > Meru800biac: add PM, fan_service, sensor_service and wedge_qsfp_util support #502 - Depends on #501 - Add PlatformType `PLATFORM_MERU800BIAC = 40` to fboss_common.thrift - Define the string definition for the platform type - Updated PlatformProductInfo.cpp to identify meru800biac - Reference meru800bia platform_mapping and platform_port_mapping Pull Request resolved: #500 Reviewed By: xanabani Differential Revision: D79726590 Pulled By: harshitgulati18 fbshipit-source-id: 97f68d757161d979bd24510faa1a534763e2218b
facebook-github-bot
pushed a commit
that referenced
this pull request
Aug 11, 2025
Summary: # Description > NOTE: This is part of a series of PRs to add support for Meru800biac. The dependency chain of the PRs is as follows: > > Meru800biac: define new platform, add platform and port mapping #500 > Meru800biac: add bsp mapping, qsfp_service, led_service #501 - Depends on #500 > Meru800biac: add PM, fan_service, sensor_service and wedge_qsfp_util support #502 - Depends on #501 Meru800biac will use the same bsp mapping mapping as Meru800bia - Add support for qsfp_service and led_service support # Testing ## qsfp_service qsfp_service use the same config as meru800bia ``` Jul 29 18:00:24 vpr405.sjc.aristanetworks.com systemd[1]: Started Start qsfp_service. │ Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: firmware_upgrade_on_coldboot=true │~ Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: firmware_upgrade_on_link_down=false │] Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: firmware_upgrade_on_tcvr_insert=true │# Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: firmware_upgrade_supported=true │ Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: mode=meru800bia │[ Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: remediation_enabled=false │r Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: Overriding default flag from config: set_max_fec_sampling=true │o Jul 29 18:00:24 vpr405.sjc.aristanetworks.com run_qsfp_service.sh[6190]: I0729 18:00:24.145640 6190 PlatformProductInfo.cpp:356] Success parsing product info fields ... ``` [fboss2_show_interface_phy.txt](https://github.com/user-attachments/files/21495202/fboss2_show_interface_phy.txt) [fboss2_show_port.txt](https://github.com/user-attachments/files/21495203/fboss2_show_port.txt) [fboss2_show_transceiver.txt](https://github.com/user-attachments/files/21495204/fboss2_show_transceiver.txt) ## led_service [data_corral_service_hw_test.txt](https://github.com/user-attachments/files/21553598/data_corral_service_hw_test.txt) - succedded Pull Request resolved: #501 Reviewed By: shiva-menta Differential Revision: D79941243 Pulled By: harshitgulati18 fbshipit-source-id: 1249466fa6d51bc267fe0caec4dbc10bafff9291
Contributor
|
@harshitgulati18 has imported this pull request. If you are a Meta employee, you can view this in D80010013. |
Contributor
|
@harshitgulati18 merged this pull request in 72a697e. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Meru800biac will use the same configs as Meru800bia:
Testing
platform_manager
platform_manager use the same config as meru800bia and succeed
sensor_service
sensor_service is running and uses the same config as meru800bia
sensor_service_sw_test.txt PASSED
fan_service
fan_service is running and uses the same config as meru800bia
fan_service_hw_test.txt
wedge_qsfp_util