Skip to content

[action] [PR:561] Update Amphenol package for mock test#80

Merged
r12f merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/561
Apr 23, 2025
Merged

[action] [PR:561] Update Amphenol package for mock test#80
r12f merged 1 commit intoAzure:202412from
mssonicbld:cherry/msft-202412/561

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description

Add missing Amphenol package to support changes merged via sonic-net/sonic-platform-common#555

Motivation and Context

The current changeset is to fix the below traceback seen while building SONiC image

<frozen importlib._bootstrap>:690: in _load_unlocked
    ???
/usr/lib/python3/dist-packages/_pytest/assertion/rewrite.py:168: in exec_module
    exec(co, module.__dict__)
tests/test_PsuChassisInfo.py:14: in <module>
    from .mock_platform import MockChassis, MockPsu, MockFanDrawer, MockModule
tests/mock_platform.py:1: in <module>
    from sonic_platform_base import chassis_base
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/__init__.py:1: in <module>
    from . import chassis_base
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/chassis_base.py:10: in <module>
    from . import sfp_base
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/sfp_base.py:11: in <module>
    from .sonic_xcvr.xcvr_api_factory import XcvrApiFactory
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/sonic_xcvr/xcvr_api_factory.py:22: in <module>
    from .api.amphenol.backplane import AmphBackplaneImpl
E   ModuleNotFoundError: No module named 'sonic_platform_base.sonic_xcvr.api.amphenol'

How Has This Been Tested?

Confirmed image build is successful and loaded image on DUT to confirm the fix to ensure that xcvrd is running without crashing.

Additional Information (Optional)

MSFT ADO - 32488018

<!-- Provide a general summary of your changes in the Title above -->

#### Description
<!--
     Describe your changes in detail
-->
Add missing Amphenol package to support changes merged via sonic-net/sonic-platform-common#555

#### Motivation and Context
<!--
     Why is this change required? What problem does it solve?
     If this pull request closes/resolves an open Issue, make sure you
     include the text "fixes #xxxx", "closes #xxxx" or "resolves #xxxx" here
-->
The current changeset is to fix the below traceback seen while building SONiC image

```
<frozen importlib._bootstrap>:690: in _load_unlocked
    ???
/usr/lib/python3/dist-packages/_pytest/assertion/rewrite.py:168: in exec_module
    exec(co, module.__dict__)
tests/test_PsuChassisInfo.py:14: in <module>
    from .mock_platform import MockChassis, MockPsu, MockFanDrawer, MockModule
tests/mock_platform.py:1: in <module>
    from sonic_platform_base import chassis_base
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/__init__.py:1: in <module>
    from . import chassis_base
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/chassis_base.py:10: in <module>
    from . import sfp_base
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/sfp_base.py:11: in <module>
    from .sonic_xcvr.xcvr_api_factory import XcvrApiFactory
/usr/local/lib/python3.11/dist-packages/sonic_platform_base/sonic_xcvr/xcvr_api_factory.py:22: in <module>
    from .api.amphenol.backplane import AmphBackplaneImpl
E   ModuleNotFoundError: No module named 'sonic_platform_base.sonic_xcvr.api.amphenol'

```

#### How Has This Been Tested?
<!--
     Please describe in detail how you tested your changes.
     Include details of your testing environment, and the tests you ran to
     see how your change affects other areas of the code, etc.
-->
Confirmed image build is successful and loaded image on DUT to confirm the fix to ensure that xcvrd is running without crashing.

#### Additional Information (Optional)
MSFT ADO - 32488018
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-platform-common#561

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@r12f r12f merged commit add9e9d into Azure:202412 Apr 23, 2025
3 checks passed
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