-
Notifications
You must be signed in to change notification settings - Fork 3.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Smoketest]Add try 3 times with 30s sleeping time #17848
Conversation
Have you tested it? |
Yes, passed the test locally whether it's PowerShell or Windows Powershell. |
How about the negative case: you deliberately make a test case fail and check:
|
Yes, change the value of the environment variable "$ErrorActionPreference" to "Continue" when the command runs incorrectly and retry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Add try 3 times with 30s sleeping time between each try.
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added