Skip to content

Conversation

@echuawu
Copy link
Contributor

@echuawu echuawu commented Aug 18, 2025

Description of PR

Add Mellanox BMC flows testplan
Design HLD: sonic-net/SONiC#2062

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

Approach

What is the motivation for this PR?

Add testplan for Mellanox BMC flows

How did you do it?

Add testplan for Mellanox BMC flows

How did you verify/test it?

Any platform specific information?

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

Documentation

@echuawu echuawu requested review from wangxin and yxieca as code owners August 18, 2025 11:30
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from 3bf5573 to da2f78b Compare August 19, 2025 06:48
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from da2f78b to f1aa96a Compare August 19, 2025 07:06
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from f1aa96a to cbebbce Compare August 19, 2025 07:07
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from cbebbce to 14ecced Compare August 19, 2025 07:12
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from 14ecced to 5ad9389 Compare August 19, 2025 08:58
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from 5ad9389 to 73cbc0b Compare August 19, 2025 11:03
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from 73cbc0b to c9077c2 Compare August 19, 2025 11:05
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from c9077c2 to c8e9327 Compare August 20, 2025 04:44
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from c8e9327 to 350ab70 Compare August 20, 2025 05:05
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from 350ab70 to 782b4ac Compare August 20, 2025 07:09
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu force-pushed the bmc_flow_test_plan branch from 782b4ac to cf9c83b Compare August 20, 2025 07:25
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

Add Mellanox BMC flows testplan

Change-Id: I7e2bc1c40e3ee947a61877307d58f7f591121e45
@echuawu echuawu force-pushed the bmc_flow_test_plan branch from cf9c83b to 2e56f00 Compare August 20, 2025 08:58
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

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

@echuawu echuawu marked this pull request as draft August 25, 2025 02:11
### Test Case # 13 - Test BMC dump in techsupport
1. Run command 'show techsupport' to generate a switch dump
2. Wait until the dump generated
3. Extract the dump file and validate the BMC dump files existence
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we also planning to test the scenario where the BMC dump is not ready before collection begins in the generate_dump_script logic?

Design document was talking on how we wait for 60s before we timeout and ignore this dump.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Are we also planning to test the scenario where the BMC dump is not ready before collection begins in the generate_dump_script logic?

Design document was talking on how we wait for 60s before we timeout and ignore this dump.

Do you have some special case need to cover?

Copy link
Contributor

@ymd-arista ymd-arista Oct 8, 2025

Choose a reason for hiding this comment

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

Just checking to see if we can close that test gap. No special case requirement from my end.

@echuawu echuawu marked this pull request as ready for review September 28, 2025 03:12
@echuawu echuawu marked this pull request as draft September 29, 2025 01:57
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