ci
ci copied to clipboard
chore(tikv): set retry to 5 on new ut on lts branch
set retry to 5 on new ut on lts branch.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please ask for approval from purelind, ensuring that each of them provides their approval before proceeding. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment
I have already done a preliminary review for you, and I hope to help you do a better job.
Based on the pull request diff, it seems that the change only adds the --retries 5
argument to the EXTRA_NEXTEST_ARGS
variable in the pull_unit_test.groovy
file for three different release branches (release-6.5
, release-7.1
, and release-7.5
) of the tikv
repository.
There are no potential problems that I can see with this change. However, it would be good to make sure that the --retries
option is supported by the testing framework being used.
As for fixing suggestions, it would be good to add some more context to the pull request description to explain why the retries are being added. Additionally, it might be helpful to add some comments in the code explaining the purpose of the --retries
option and how it affects the testing process.