Skip to content

Port consistency checker to 202311 (f2cadae6)#17672

Merged
StormLiangMS merged 1 commit intosonic-net:202311from
Ryangwaite:port-consistency-checker-to-202311
Apr 29, 2025
Merged

Port consistency checker to 202311 (f2cadae6)#17672
StormLiangMS merged 1 commit intosonic-net:202311from
Ryangwaite:port-consistency-checker-to-202311

Conversation

@Ryangwaite
Copy link
Contributor

@Ryangwaite Ryangwaite commented Mar 25, 2025

Description of PR

Summary:
This is a manual backport of the consistency-checker utility from master at commit f2cadae. This includes the following two PR's:

Manually backported due to conflicts.

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

There were conflicts in creating cherry-pick backport PR's hence this manual approach.

How did you do it?

Cherry-picked the changes from the two PR's and resolved conflicts.

How did you verify/test it?

Ran test_upgrade_path and test_advanced_reboot on this branch. The feature in general has also been tested thoroughly on master.

Any platform specific information?

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

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@Ryangwaite Ryangwaite changed the base branch from master to 202311 March 25, 2025 03:51
@Ryangwaite Ryangwaite force-pushed the port-consistency-checker-to-202311 branch from 0aa9e33 to 4ee0eec Compare March 25, 2025 03:57
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Ryangwaite Ryangwaite force-pushed the port-consistency-checker-to-202311 branch from 4ee0eec to 38f197b Compare March 25, 2025 04:02
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Ryangwaite Ryangwaite changed the title Port consistency checker to 202311 Port consistency checker to 202311 (f2cadae6) Apr 10, 2025
@azure-pipelines
Copy link

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

@Ryangwaite Ryangwaite force-pushed the port-consistency-checker-to-202311 branch from 38f197b to 9546949 Compare April 28, 2025 01:58
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit 19bc0ea into sonic-net:202311 Apr 29, 2025
12 checks passed
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