Optics insertion removal testplan by az-pz · Pull Request #20524 · sonic-net/sonic-mgmt

az-pz · 2025-09-04T22:07:43Z

Description of PR

Summary:
This PR adds a testplan for optics insertion and removal testing in sonic-mgmt.
Fixes # (issue)

Type of change

Back port request

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

ADO: 34410994

mssonicbld · 2025-09-04T22:07:50Z

/azp run

azure-pipelines · 2025-09-04T22:08:06Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2025-09-04T22:10:28Z

/azp run

azure-pipelines · 2025-09-04T22:10:44Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2025-09-09T18:10:59Z

/azp run

azure-pipelines · 2025-09-09T18:11:07Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md

mssonicbld · 2025-09-11T20:11:54Z

/azp run

azure-pipelines · 2025-09-11T20:12:01Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2025-09-11T20:14:46Z

/azp run

azure-pipelines · 2025-09-11T20:14:54Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md

mssonicbld · 2025-09-11T20:51:59Z

/azp run

azure-pipelines · 2025-09-11T20:52:06Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md

docs/testplan/transceiver/optics_insertion_removal_testplan.md

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md

docs/testplan/transceiver/optics_insertion_removal_testplan.md

Copilot

Pull Request Overview

This PR adds a comprehensive test plan for validating the insertion and removal of optical transceivers in SONiC. The document establishes testing procedures for both physical OIR (Online Insertion and Removal) operations and remote reseat scenarios.

Defines test cases for physical and remote optics insertion/removal validation
Specifies configuration attributes and API structure for OIR testing
Covers various optics types including AOC, AEC, DR8, DAC, SR/LR, FR, ZR, and breakout cables

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md

mssonicbld · 2025-09-12T15:30:30Z

/azp run

azure-pipelines · 2025-09-12T15:30:38Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2025-09-12T15:31:11Z

/azp run

azure-pipelines · 2025-09-12T15:31:19Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2025-09-12T15:32:09Z

/azp run

azure-pipelines · 2025-09-12T15:32:18Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

azure-pipelines · 2026-02-18T20:29:14Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2026-02-18T20:30:56Z

/azp run

azure-pipelines · 2026-02-18T20:31:05Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2026-02-20T14:32:11Z

/azp run

azure-pipelines · 2026-02-20T14:32:20Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2026-02-20T14:41:01Z

/azp run

azure-pipelines · 2026-02-20T14:41:10Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

docs/testplan/transceiver/optics_insertion_removal_testplan.md

mssonicbld · 2026-03-05T18:33:52Z

/azp run

azure-pipelines · 2026-03-05T18:34:00Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2026-03-10T16:02:04Z

/azp run

azure-pipelines · 2026-03-10T16:02:13Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2026-03-10T19:02:00Z

/azp run

azure-pipelines · 2026-03-10T19:02:09Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

mssonicbld · 2026-03-10T20:01:09Z

/azp run

azure-pipelines · 2026-03-10T20:01:18Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

Add OIR API. Address PR comments. Update docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Co-authored-by: mihirpat1 <112018033+mihirpat1@users.noreply.github.com> Add tables to monitor Update OIR API. Update testplan to check for presence data. Add checks for peer port as well. Added the sample PhysicalOIR class. Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Updated the expected results for optics removal validation to include link flap count for the local port. Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

…R test plan Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

…parameter Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

mssonicbld · 2026-03-16T14:56:09Z

/azp run

azure-pipelines · 2026-03-16T14:56:18Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

yxieca

AI agent on behalf of Ying. Reviewed; no issues found.

az-pz marked this pull request as ready for review September 9, 2025 22:21

az-pz requested review from wangxin and yxieca as code owners September 9, 2025 22:21

mihirpat1 reviewed Sep 10, 2025

View reviewed changes

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Outdated Show resolved Hide resolved

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Outdated Show resolved Hide resolved

qinchuanares reviewed Sep 11, 2025

View reviewed changes

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Outdated Show resolved Hide resolved

qinchuanares reviewed Sep 11, 2025

View reviewed changes

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Outdated Show resolved Hide resolved

qinchuanares reviewed Sep 11, 2025

View reviewed changes

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Outdated Show resolved Hide resolved

qinchuanares reviewed Sep 11, 2025

View reviewed changes

docs/testplan/transceiver_onboarding/optics_insertion_removal_testplan.md Outdated Show resolved Hide resolved

mihirpat1 mentioned this pull request Sep 11, 2025

[sonic-mgmt] Update transceiver onboarding HLD to make parameters attribute based #20230

Merged

11 tasks

mihirpat1 requested a review from Copilot September 12, 2025 04:16

mihirpat1 reviewed Sep 12, 2025

View reviewed changes

Copilot AI reviewed Sep 12, 2025

View reviewed changes

az-pz force-pushed the ariz/create-optics-insertion-removal-testplan branch from 8d0c871 to 0c4d3a9 Compare February 20, 2026 14:40

az-pz commented Feb 20, 2026

View reviewed changes

docs/testplan/transceiver/optics_insertion_removal_testplan.md Show resolved Hide resolved

az-pz mentioned this pull request Mar 12, 2026

Add physical OIR tests. #22913

Merged

12 tasks

az-pz and others added 8 commits March 16, 2026 14:55

Move the testplan to transceiver directory.

3e78f43

Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Fix link.

f43b1c5

Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Add port_startup_wait_sec attribute details.

f32ae52

Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Refine optics removal validation test case

37e130e

Updated the expected results for optics removal validation to include link flap count for the local port. Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Update paths for physical OIR and remote reseat JSON files in test plan

0f624d9

Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Fix attribute name from 'port_under_test' to 'ports_under_test' in OI…

c6e6b64

…R test plan Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

Update PhysicalOir class constructor to include port_attributes_dict …

60f5838

…parameter Signed-off-by: Ariz Zubair(Microsoft) <arizzubair@microsoft.com>

az-pz force-pushed the ariz/create-optics-insertion-removal-testplan branch from aa32b4d to 60f5838 Compare March 16, 2026 14:56

yxieca approved these changes Mar 18, 2026

View reviewed changes

Conversation

az-pz commented Sep 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of PR

Type of change

Back port request

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Uh oh!

mssonicbld commented Sep 4, 2025

Uh oh!

azure-pipelines bot commented Sep 4, 2025

Uh oh!

mssonicbld commented Sep 4, 2025

Uh oh!

azure-pipelines bot commented Sep 4, 2025

Uh oh!

mssonicbld commented Sep 9, 2025

Uh oh!

azure-pipelines bot commented Sep 9, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mssonicbld commented Sep 11, 2025

Uh oh!

azure-pipelines bot commented Sep 11, 2025

Uh oh!

mssonicbld commented Sep 11, 2025

Uh oh!

azure-pipelines bot commented Sep 11, 2025

Uh oh!

Uh oh!

mssonicbld commented Sep 11, 2025

Uh oh!

azure-pipelines bot commented Sep 11, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

mssonicbld commented Sep 12, 2025

Uh oh!

azure-pipelines bot commented Sep 12, 2025

Uh oh!

mssonicbld commented Sep 12, 2025

Uh oh!

azure-pipelines bot commented Sep 12, 2025

Uh oh!

mssonicbld commented Sep 12, 2025

Uh oh!

azure-pipelines bot commented Sep 12, 2025

Uh oh!

azure-pipelines bot commented Feb 18, 2026

Uh oh!

mssonicbld commented Feb 18, 2026

Uh oh!

azure-pipelines bot commented Feb 18, 2026

Uh oh!

mssonicbld commented Feb 20, 2026

az-pz commented Sep 4, 2025 •

edited

Loading