Skip to content

[upgrade path] Fix ptftest failure: add preboot_files to the param list#4776

Merged
vaibhavhd merged 1 commit intosonic-net:masterfrom
vaibhavhd:upgrade-path-fix
Nov 30, 2021
Merged

[upgrade path] Fix ptftest failure: add preboot_files to the param list#4776
vaibhavhd merged 1 commit intosonic-net:masterfrom
vaibhavhd:upgrade-path-fix

Conversation

@vaibhavhd
Copy link
Copy Markdown
Contributor

Description of PR

Summary: After fixing advance-reboot sad cases in #4758, upgrade path cases have regressed. Update the param list for upgrade path cases same as in test_advanced_reboot.

Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911

Approach

What is the motivation for this PR?

Fix test_upgrade_path regression where ptftest fails as it is expecting list of preboot files as modified in #4758

How did you do it?

How did you verify/test it?

Tested in physical testbed, and the test works well with the fix.

Any platform specific information?

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

Documentation

@vaibhavhd vaibhavhd requested a review from a team as a code owner November 29, 2021 07:54
@vaibhavhd vaibhavhd requested a review from shi-su November 29, 2021 08:00
@vaibhavhd
Copy link
Copy Markdown
Contributor Author

/Azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@vaibhavhd vaibhavhd merged commit ce83bca into sonic-net:master Nov 30, 2021
@vaibhavhd vaibhavhd deleted the upgrade-path-fix branch November 30, 2021 11:44
AntonHryshchuk pushed a commit to AntonHryshchuk/sonic-mgmt that referenced this pull request Jan 4, 2022
…st (sonic-net#4776)

Fix upgrade_path case:
After fixing advance-reboot sad cases in sonic-net#4758, upgrade path cases have regressed. Update the param list for upgrade path cases same as in test_advanced_reboot.
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.

2 participants