foundationdb icon indicating copy to clipboard operation
foundationdb copied to clipboard

fixing protocol versions for restarting tests

Open sfc-gh-jslocum opened this issue 3 years ago • 6 comments

#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-branch or main if 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)

sfc-gh-jslocum avatar Sep 02 '22 23:09 sfc-gh-jslocum

Doxense CI Report for Windows 10

  • Commit ID: c333b518c0e14d4fa452795f3b2a9f77b4b33318
  • Result: :heavy_check_mark: SUCCEEDED
  • Build Logs (available for 30 days)

fdb-windows-ci avatar Sep 02 '22 23:09 fdb-windows-ci

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)

foundationdb-ci avatar Sep 03 '22 00:09 foundationdb-ci

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)

foundationdb-ci avatar Sep 03 '22 03:09 foundationdb-ci

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)

foundationdb-ci avatar Sep 09 '22 17:09 foundationdb-ci

Doxense CI Report for Windows 10

  • Commit ID: 7a6a618c84b0b38e869d063bb334bfc8e268fa26
  • Result: :heavy_check_mark: SUCCEEDED
  • Build Logs (available for 30 days)

fdb-windows-ci avatar Sep 09 '22 18:09 fdb-windows-ci

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)

foundationdb-ci avatar Sep 09 '22 19:09 foundationdb-ci