Skip to content

[Mellanox] Skip warm-reboot test for two cases when issu is disabled#22104

Open
JibinBao wants to merge 3 commits into
sonic-net:masterfrom
JibinBao:skip_warm_no_issu
Open

[Mellanox] Skip warm-reboot test for two cases when issu is disabled#22104
JibinBao wants to merge 3 commits into
sonic-net:masterfrom
JibinBao:skip_warm_no_issu

Conversation

@JibinBao
Copy link
Copy Markdown
Contributor

Description of PR

skip warm-reboot for test_loopback_action_reload and test_static_dns_basic when issu is disabled on mellanox device

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
  • 202511

Approach

What is the motivation for this PR?

skip warm-reboot for test_loopback_action_reload and test_static_dns_basic when issu is disabled on mellanox device

How did you do it?

skip warm-reboot when issu is disabled

How did you verify/test it?

run the two cases on mellanox device

Any platform specific information?

Any

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

Any

Documentation

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao JibinBao changed the title [Mellanox] Skip warm-reboot test for two case when issu is disabled [Mellanox] Skip warm-reboot test for two cases when issu is disabled Jan 23, 2026
@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

nhe-NV
nhe-NV previously approved these changes Jan 27, 2026
@nhe-NV nhe-NV added the Request for 202511 branch Request to backport a change to 202511 branch label Jan 27, 2026
@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

JibinBao commented Feb 2, 2026

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-mgmt

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…2104)

* Add new T1 Mellanox-SN4280-O8C40 SKU

* updated link pointer files to point to Mellanox-SN4280-O28 SKU since ACS-SN4280 was removed
bpar9
bpar9 previously approved these changes Mar 4, 2026
@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Mar 6, 2026

@JibinBao can you elaborate your tests? I have concerns:

  1. do you have the enabled pattern on Mellanox switches that supports ISSU? How about old branches?
  2. Depending on the answer above, should the default value be enabled unless explicitly disabled?

@vaibhavhd for awareness.

@JibinBao JibinBao dismissed stale reviews from yxieca, bpar9, and nhe-NV via d840281 April 15, 2026 07:10
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao JibinBao force-pushed the skip_warm_no_issu branch from d840281 to 21bb135 Compare April 15, 2026 07:14
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao JibinBao force-pushed the skip_warm_no_issu branch from 21bb135 to 796265d Compare April 15, 2026 07:17
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

/azpw run Azure.sonic-mgmt

@mssonicbld
Copy link
Copy Markdown
Collaborator

⚠️ Notice: /azpw run only runs failed jobs now. If you want to trigger a whole pipline run, please rebase your branch or close and reopen the PR.
💡 Tip: You can also use /azpw retry to retry failed jobs directly.

Retrying failed(or canceled) jobs...

@mssonicbld
Copy link
Copy Markdown
Collaborator

Retrying failed(or canceled) stages in build 1089226:

✅Stage Test:

  • Job impacted-area-kvmtest-multi-asic-t1 by Elastictest: retried.
  • Job impacted-area-kvmtest-t0-sonic by Elastictest: retried.

@yxieca
Copy link
Copy Markdown
Collaborator

yxieca commented Apr 23, 2026

Review note (non-blocking):

  • Unresolved human change requests present.

AI agent on behalf of Ying.

@JibinBao JibinBao force-pushed the skip_warm_no_issu branch from 796265d to 10a21b4 Compare May 7, 2026 09:37
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JibinBao
Copy link
Copy Markdown
Contributor Author

JibinBao commented May 8, 2026

/azpw retry

@mssonicbld
Copy link
Copy Markdown
Collaborator

Retrying failed(or canceled) jobs...

@mssonicbld
Copy link
Copy Markdown
Collaborator

Retrying failed(or canceled) stages in build 1107694:

✅Stage Test:

  • Job kvmtest-t1-lag-vpp by Elastictest: retried.

@JibinBao
Copy link
Copy Markdown
Contributor Author

JibinBao commented May 9, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 22104 in repo sonic-net/sonic-mgmt

JibinBao added 3 commits May 15, 2026 09:38
…ellanox device

Signed-off-by: jbao <jbao@nvidia.com>
…basic when issu is disabled on mellanox device

Signed-off-by: jbao <jbao@nvidia.com>
update get_reload_type_list for all vendors

Signed-off-by: jbao <jbao@nvidia.com>
@JibinBao JibinBao force-pushed the skip_warm_no_issu branch from 10a21b4 to c090c3f Compare May 15, 2026 06:39
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved for 202511 branch Request for 202511 branch Request to backport a change to 202511 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants