Skip to content

[Accton][m4052actm] Support new platform with configlib alias and sensor_service 3.2.x versionedSensor - #1602

Open
ec-michael-shih wants to merge 1 commit into
facebook:mainfrom
ec-michael-shih:20260904_add_versionedSensor_3_2_x_to_support_m4052actm
Open

[Accton][m4052actm] Support new platform with configlib alias and sensor_service 3.2.x versionedSensor#1602
ec-michael-shih wants to merge 1 commit into
facebook:mainfrom
ec-michael-shih:20260904_add_versionedSensor_3_2_x_to_support_m4052actm

Conversation

@ec-michael-shih

Copy link
Copy Markdown
Contributor

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
clang-format.............................................................Passed
shellcheck...........................................(no files to check)Skipped
shfmt................................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check json...............................................................Passed
check for merge conflicts................................................Passed
ruff check...........................................(no files to check)Skipped
ruff format..........................................(no files to check)Skipped
Prevent sai_impl in fboss manifest.......................................Passed

Summary

This PR adds support for new production hardware revisions, including SMB, 3V3_L, and 3V3_R.

Key Changes

  1. Added versionedSensors configuration in sensor_service.json.
  2. Supported the following versions for SMB and 3V3:
    • 3.2.10 (second source)
    • 3.2.20

Reason for Version Update

  • The new platform m4052actm aliases to minipack3bam via the configlib framework, inheriting its configuration settings.
  • As m4052actm is currently in the PVT phase, its hardware utilizes the minipack3bam MP Production Sub-State=2 version. Therefore, support for version 3.2.x is required.

Test Plan

Verification & Validation

  • Background: Hardware revisions for SMB, 3V3_L, and 3V3_R (versions 4.1.x and 4.2.x) on minipack3bam were previously validated and passed in prior PRs.
  • Testing Approach: Since the currently available m4052actm hardware is version 3.1.10, the test was conducted by temporarily applying 3.1.10 configurations to sensor_service.json.
  1. Build & Config: Compilation and configuration validation completed successfully.
  2. Hardware Verification: Verified on minipack3bam devices equipped with each SMB/3V3 v3.1.10:
  • Validation Results: The test passed without issues.
  • Production Configuration: Given that the production hardware will operate under Production Sub-State=2, the target 3.2.x configurations follow the exact same logic validated during the 3.1.x test.

…sor_service 3.2.x versionedSensor

Signed-off-by: michael_shih <michael_shih@accton.com>
@ec-michael-shih
ec-michael-shih requested review from a team as code owners September 4, 2026 08:09
@meta-cla meta-cla Bot added the CLA Signed label Sep 4, 2026
@github-actions github-actions Bot added the accton label Sep 4, 2026
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.

1 participant