Skip to content

Conversation

@CliffyWong
Copy link
Contributor

@CliffyWong CliffyWong commented Nov 4, 2025

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run
image

Summary

This PR is about updating the platform name from tahansb to tahansb800bc

Motivation

As required for the platform name update to tahansb800bc, the following modifications should be included:

  1. Updated the platform and PM unit name to TAHANSB800BC in platform configurations.
  2. Updated the platform name in the BSP platform mapping.
  3. Updated the platform name in the general platform mapping.

Test Plan

1.Used clang and jq command to pretty the format
2.Compilation and config validation have passed
3.Bsp mapping json file could be generated succesfully
4.Platform services(platform_mananger/sensor_service/data_corral_service/fan_service/weutil/fwutil) and qsfp_service could be launched successfully.

image

pls refer the full logs:
PM Services&QSFP Service.zip

5.Hardware tests of platform_manager/sensor_service/data_corral_service/fan_service/weutil/fwutil passed, pls refer the logs:
PM Services HwTests.zip

logs of push f8609ae:
20251110 PM&QSFP.zip

@meta-cla meta-cla bot added the CLA Signed label Nov 4, 2025
@CliffyWong CliffyWong force-pushed the rename_to_tahansb800bc branch 2 times, most recently from 031057c to 63efafe Compare November 7, 2025 05:08
@CliffyWong CliffyWong force-pushed the rename_to_tahansb800bc branch from 63efafe to f8609ae Compare November 10, 2025 07:19
@meta-codesync
Copy link

meta-codesync bot commented Nov 11, 2025

@ezeob002 has imported this pull request. If you are a Meta employee, you can view this in D86722417.

@meta-codesync
Copy link

meta-codesync bot commented Nov 11, 2025

@ezeob002 merged this pull request in 73c3fdd.

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.

2 participants