fixing protocol versions for restarting tests
#8090 but for main too. It appears this isn't actively causing test failures on main, probably because apple's 7.1 didn't have the exact same protocol version as it was cut at a different time, but i think it's incorrect on main currently the same as it was on 71.2.
I ran a restarting ensemble with this change and there were some failures, but they weren't the same failures on snowflake/release-71.2, so i ran another ensemble on the head of main without these changes and it hit the same failures on main at a similar rate, so i assume restarting tests are just broken on main right now.
Code-Reviewer Section
The general pull request guidelines can be found here.
Please check each of the following things and check all boxes before accepting a PR.
- [ ] The PR has a description, explaining both the problem and the solution.
- [ ] The description mentions which forms of testing were done and the testing seems reasonable.
- [ ] Every function/class/actor that was touched is reasonably well documented.
For Release-Branches
If this PR is made against a release-branch, please also check the following:
- [ ] This change/bugfix is a cherry-pick from the next younger branch (younger
release-branchormainif this is the youngest branch) - [ ] There is a good reason why this PR needs to go into a release branch and this reason is documented (either in the description above or in a linked GitHub issue)
Doxense CI Report for Windows 10
- Commit ID: c333b518c0e14d4fa452795f3b2a9f77b4b33318
- Result: :heavy_check_mark: SUCCEEDED
- Build Logs (available for 30 days)
Result of foundationdb-pr on Linux CentOS 7
- Commit ID: c333b518c0e14d4fa452795f3b2a9f77b4b33318
- Duration 1:09:31
- Result: :white_check_mark: SUCCEEDED
- Error:
N/A - Build Logs (available for 30 days)
- Build Artifact (available for 30 days)
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
- Commit ID: c333b518c0e14d4fa452795f3b2a9f77b4b33318
- Duration 4:17:08
- Result: :x: FAILED
- Error:
Build has timed out. - Build Logs (available for 30 days)
- Build Artifact (available for 30 days)
Result of foundationdb-pr on Linux CentOS 7
- Commit ID: 7a6a618c84b0b38e869d063bb334bfc8e268fa26
- Duration 1:05:07
- Result: :white_check_mark: SUCCEEDED
- Error:
N/A - Build Logs (available for 30 days)
- Build Artifact (available for 30 days)
Doxense CI Report for Windows 10
- Commit ID: 7a6a618c84b0b38e869d063bb334bfc8e268fa26
- Result: :heavy_check_mark: SUCCEEDED
- Build Logs (available for 30 days)
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
- Commit ID: 7a6a618c84b0b38e869d063bb334bfc8e268fa26
- Duration 2:45:02
- Result: :x: FAILED
- Error:
Error while executing command: if $(grep -q -- "--- FAIL:" ${CODEBUILD_SRC_DIR}/fdb-kubernetes-tests/logs/*.log); then echo "TESTS FAILED SEE THESE LOGS:"; echo ; grep -l -- "--- FAIL:" ${CODEBUILD_SRC_DIR}/fdb-kubernetes-tests/logs/*.log; exit 1; fi. Reason: exit status 1 - Build Logs (available for 30 days)
- Build Artifact (available for 30 days)