Skip to content

Conversation

matthewdale
Copy link
Collaborator

@matthewdale matthewdale commented Jul 29, 2025

GODRIVER-3622

Summary

  • Add the retry_on_failure: true config to all test tasks that use the setup-test task.
  • Remove the type: test config from the setup-test command because those are setup steps, not test steps.

Background & Motivation

We consistently have a few flaky tests. We generally start troubleshooting failed tasks by re-running them before looking at any failures. Instead of that manual process, it would be more effective to automate the process, making consistently failing tasks much more apparent.

@mongodb-drivers-pr-bot mongodb-drivers-pr-bot bot added the review-priority-low Low Priority PR for Review: within 3 business days label Jul 29, 2025
Copy link
Contributor

API Change Report

No changes found!

@matthewdale matthewdale force-pushed the godriver3622-retry-task branch from 624802a to 6dfc1fb Compare July 30, 2025 00:41
@matthewdale matthewdale marked this pull request as ready for review July 30, 2025 00:48
@matthewdale matthewdale requested a review from a team as a code owner July 30, 2025 00:48
@matthewdale matthewdale requested a review from qingyang-hu July 30, 2025 00:48
Copy link
Collaborator

@qingyang-hu qingyang-hu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matthewdale matthewdale merged commit 0c85ece into mongodb:master Jul 30, 2025
34 checks passed
prestonvasquez pushed a commit to prestonvasquez/mongo-go-driver that referenced this pull request Jul 31, 2025
alcaeus added a commit that referenced this pull request Aug 5, 2025
* master: (65 commits)
  Replace all uses of 'interface{}' with 'any' in the bson package. (#2138)
  GODRIVER-3473 Short-cicruit cursor.next() on invalid timeouts (#2135)
  GODRIVER-3622 Automatically retry some test tasks. (#2147)
  Replace all uses of 'interface{}' with 'any' in the repo docs. (#2142)
  GODRIVER-3102: Perf comparison (#2134)
  GODRIVER-3587 Use raw bytes in valueReader (#2120)
  Replace all uses of 'interface{}' with 'any' in the internal/ packages. (#2140)
  Replace all uses of 'interface{}' with 'any' in the x/ packages. (#2137)
  GODRIVER-2016 Unskip all Transactions unified spec tests. (#2132)
  GODRIVER-2540 Run govulncheck in CI builds. (#2136)
  GODRIVER-3549 Update Client BulkWrite prose tests. (#2131)
  Add guidelines for contributing features to the Go Driver (#2125)
  Bump alcaeus/automatic-merge-up-action from 1.0.0 to 1.0.1 in the actions group (#2126)
  Add wrappedMsgOnly to mongo.MarshalError and mongo.MongocryptError.
  Bump testdata/specifications from `db69351` to `6689929`
  fix wiremessage oob in case of intmin (#2076)
  GODRIVER-3399: PoolClearedError should have TransientTransactionError label appended to it (#2114)
  PR feedback.
  Prevent integration tests from running when testing with -short
  Skip AWS Test if no URI (#2102)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-priority-low Low Priority PR for Review: within 3 business days

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants