Skip to content

DRIVERS-3556 skip affected CSOT change stream tests on 9.0+ sharded#1962

Merged
kevinAlbs merged 5 commits into
mongodb:masterfrom
kevinAlbs:DRIVERS-3556
Jul 10, 2026
Merged

DRIVERS-3556 skip affected CSOT change stream tests on 9.0+ sharded#1962
kevinAlbs merged 5 commits into
mongodb:masterfrom
kevinAlbs:DRIVERS-3556

Conversation

@kevinAlbs

@kevinAlbs kevinAlbs commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Skip time-sensitive CSOT change stream tests on 9.0+ sharded clusters.

Details

Motivated by DRIVERS-3556. See comment for motivation to skip rather than try to workaround.

override-operation-timeoutMS.yml.template is not modified since it is not used to generate the test file pending DRIVERS-3266.

Please complete the following before merging:

  • Is the relevant DRIVERS ticket in the PR title?
  • [ ] Update changelog. (Test changes only)
  • Test changes in at least one language driver. (Tested locally with Go)
  • [ ] Test these changes against all server versions and topologies (including standalone, replica set, and sharded
    clusters).
    (Skipping. This only adds a test skip)

@kevinAlbs
kevinAlbs requested a review from prestonvasquez July 9, 2026 18:51
@kevinAlbs
kevinAlbs marked this pull request as ready for review July 9, 2026 18:51
@kevinAlbs
kevinAlbs requested a review from a team as a code owner July 9, 2026 18:51
- minServerVersion: "4.4"
topologies: ["replicaset"]
- minServerVersion: "4.4"
maxServerVersion: "8.99" # Skip server 9.0+ due to reasons noted in DRIVERS-3556.

@prestonvasquez prestonvasquez Jul 9, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

[blocking] We should add a migration note in case these tests are re-purposed to use maxTimeMS

If this test is updated to use maxTimeMS, see DRIVERS-3556. The unified test format may need to be extended to include a runner option for advancing the config server's cluster time immediately before opening a change stream. Otherwise, existing tests using maxTimeMS < periodicNoopIntervalSecs on 9.0+ sharded clusters may unexpectedly fail with server timeouts.

https://jira.mongodb.org/browse/DRIVERS-3556?focusedCommentId=8635908&focusedId=8635908&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-8635908

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated comment as suggested with minor wording tweaks.

@kevinAlbs
kevinAlbs requested a review from prestonvasquez July 9, 2026 19:41

@prestonvasquez prestonvasquez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@kevinAlbs The following tests must also be skipped:

- description: "operation is retried multiple times for non-zero timeoutMS - createChangeStream on client"

- description: "operation is retried multiple times for non-zero timeoutMS - createChangeStream on database"

- description: "operation is retried multiple times for non-zero timeoutMS - createChangeStream on collection"

@kevinAlbs
kevinAlbs requested a review from prestonvasquez July 10, 2026 20:15

@prestonvasquez prestonvasquez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good 👍

@kevinAlbs
kevinAlbs merged commit d75d82b into mongodb:master Jul 10, 2026
6 checks passed
alexbevi pushed a commit to alexbevi/specifications that referenced this pull request Jul 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants