Skip to content

Add Console-Monitor-Test-Plan.md#22127

Merged
lizhijianrd merged 2 commits into
sonic-net:masterfrom
wiperi:cliffchen/submit/console_monitor_testplan
May 9, 2026
Merged

Add Console-Monitor-Test-Plan.md#22127
lizhijianrd merged 2 commits into
sonic-net:masterfrom
wiperi:cliffchen/submit/console_monitor_testplan

Conversation

@wiperi
Copy link
Copy Markdown
Contributor

@wiperi wiperi commented Jan 26, 2026

Description of PR

Add test plan document for Console Monitor service, covering functional tests, filter tests, enable/disable toggle tests, and performance tests.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • [] New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Approach

What is the motivation for this PR?

Console Monitor is a new feature that provides link operational state detection between Console Server (DCE) and SONiC Switch (DTE) via serial ports. This PR adds the test plan to verify the feature's correct operation.

How did you do it?

Added test plan document covering:

  • Functional tests (initial state, link up/unknown detection)
  • Filter timeout and correctness tests
  • Enable/disable toggle tests
  • Performance tests (48-port scalability)

How did you verify/test it?

Documentation only - test plan document.

Any platform specific information?

Requires Console Switch hardware with serial ports.

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

C0

Documentation

Related HLD: sonic-net/SONiC#2178

Signed-off-by: cliffchen <t-cliffchen+github@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

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

lizhijianrd
lizhijianrd previously approved these changes Jan 28, 2026
…prove clarity

Signed-off-by: cliffchen <t-cliffchen+github@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

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

kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…atically (sonic-net#22127)

#### Why I did it
src/sonic-utilities
```
* 3cfd9a68 - (HEAD -> 202411, origin/202411) Fixed the issues with sonic-clear queuecounter for egress queue and voq (sonic-net#3816) (10 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Copy link
Copy Markdown
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Apr 12, 2026

AI agent on behalf of Ying.\n\nReview note (non-blocking):\n- Only docs/ changes detected; per review rules I am not approving docs-only PRs.\n\n

@lizhijianrd lizhijianrd merged commit b49b5d8 into sonic-net:master May 9, 2026
6 checks passed
johanna-nexthop pushed a commit to nexthop-ai/sonic-mgmt that referenced this pull request May 14, 2026
What is the motivation for this PR?
Console Monitor is a new feature that provides link operational state detection between Console Server (DCE) and SONiC Switch (DTE) via serial ports. This PR adds the test plan to verify the feature's correct operation.

How did you do it?
Added test plan document covering:

Functional tests (initial state, link up/unknown detection)
Filter timeout and correctness tests
Enable/disable toggle tests
Performance tests (48-port scalability)
How did you verify/test it?
Documentation only - test plan document.

Any platform specific information?
Requires Console Switch hardware with serial ports.

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

---------

Signed-off-by: cliffchen <t-cliffchen+github@microsoft.com>
Signed-off-by: Johanna Jegan <johanna@nexthop.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants