Skip to content

nexthopbmc: add CIT kernel module presence test - #291

Open
nate-nexthop wants to merge 1 commit into
facebook:heliumfrom
nexthop-ai:nate.kernel-module
Open

nate-nexthop wants to merge 1 commit into
facebook:heliumfrom
nexthop-ai:nate.kernel-module

Conversation

@nate-nexthop

Copy link
Copy Markdown
Contributor

Description

Check that the AMD APML kernel modules (apml_sbtsi, apml_sbrmi) used to read the host CPU temperature and RMI mailbox are loaded on nexthopbmc.

Motivation

Ensure we have the correct kernel modules loaded.

Test Plan

Run on a nexthopbmc with tests2 copied to /run/tests2:

$ python3 cit_runner.py --run-test tests.nexthopbmc.test_kernel_module_presence
test_installed_kmods (tests.nexthopbmc.test_kernel_module_presence.KernelModulePresenceTest.test_installed_kmods) ... ok

----------------------------------------------------------------------
Ran 1 test in 0.056s

OK

Summary:
Check that the AMD APML kernel modules (apml_sbtsi, apml_sbrmi) used to
read the host CPU temperature and RMI mailbox are loaded on nexthopbmc.

# Test Plan

Run on a nexthopbmc with tests2 copied to /run/tests2:

    $ python3 cit_runner.py --run-test tests.nexthopbmc.test_kernel_module_presence
    test_installed_kmods (tests.nexthopbmc.test_kernel_module_presence.KernelModulePresenceTest.test_installed_kmods) ... ok

    ----------------------------------------------------------------------
    Ran 1 test in 0.056s

    OK
@meta-cla meta-cla Bot added the CLA Signed label Sep 22, 2026
@meta-codesync

meta-codesync Bot commented Sep 22, 2026

Copy link
Copy Markdown

This pull request has been imported. If you are a Meta employee, you can view this in D121231751. (Because this pull request was imported automatically, there will not be any future comments.)

@nate-nexthop

Copy link
Copy Markdown
Contributor Author

@ezeob002 FYI, more nexthopbmc tests.

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