Skip to content

Port consistency checker to 202305 (f2cadae6)#17674

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

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

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.py 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

@Ryangwaite Ryangwaite requested a review from prgeor as a code owner March 25, 2025 05:15
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Ryangwaite Ryangwaite changed the title Backported consistency-checker utility from master (f2cadae6) Port consistency checker to 202305 (f2cadae6) Apr 10, 2025
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 1ad2ae7 into sonic-net:202305 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