Skip to content

Conversation

@eramongodb
Copy link
Contributor

@eramongodb eramongodb commented Jul 30, 2025

Cherry-picks several recent commits (with minor modifications) to address ongoing task failures on the r1.30 branch. These changes are roughly summarized as:

  • Sync change streams spec tests for compatibility with latest server builds.
  • Migrate Poetry -> Astral UV for Python scripts and tools.
  • Migrate Artifactory -> Amazon ECR for OCI images and Earthly.
  • Miscellaneous C standard and compiler compatibility fixes.

The loadbalanced-* still fail with these changes. I could not determine which commit(s) are appropriate to fix the "client writeConcern ignored for client bulkWrite in transaction" spec test failure:

error: expected result, but got error: Transaction 92bae3e2-aaf3-4a23-8867-21f8862a4b14 - 47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU= -  - :1 was aborted on statement 0 due to: a non-retryable snapshot error :: caused by :: Encountered error from localhost:27217 during a transaction :: caused by :: Database transaction-tests has undergone a catalog change operation at time Timestamp(1753907336, 96) and no longer satisfies the requirements for the current transaction which requires Timestamp(1753907336, 8

eramongodb and others added 9 commits July 30, 2025 16:11
* Ensure CC and CXX consistency, use CMAKE_GENERATOR for Visual Studio
* Use ls_distro to reduce boilerplate
* Sort std-matrix tasks by compiler and version
* Support /std:clatest with MSVC
* Exclude -Wc++98-compat warnings from public-header-warnings
* Exclude -Wpre-c2x-compat warnings from public-header-warnings
* Exclude -Wunsafe-buffer-usage from public-header-warnings
* Address C2X compatibility issues due to _Bool macro expansions
…ongodb#1971)

* CDRIVER-4469 sync change streams unified spec tests with d9a0e7b2
* CDRIVER-4471 sync change streams unified spec tests with 9a8abe81
* CDRIVER-4469 sync change streams unified spec tests with 1a3b359a
* CDRIVER-4639 sync change streams unified spec tests with 474ddfcc
* CDRIVER-5863 sync change streams unified spec tests with 449d0397
* CDRIVER-5959 sync change streams unified spec tests with 0aee4aad
…ongodb#2044)

* CDRIVER-5873 Drop Debian 9
* CDRIVER-5874 Drop Debian 10
* Tidy up distros.py
* Extend Ubuntu/Debian task coverage to latest releases
* Migrate primary server coverage tasks to latest RHEL distros
* Simplify matrices and update CSFLE vs. QE comments
* Address Earthly 0.8 compatibility warnings
* Workaround repository metalink retrieval errors on CentOS
* Update release instructions to use AWS ECR instead of Artifactory
* Use `$default_search_registry` to avoid forcing use of Amazon ECR
* Also migrate sbom task from Artifactory to Amazon ECR
To fix `no such host` error logs observed after the Earthly cloud shutdown.
@eramongodb eramongodb requested a review from kevinAlbs July 30, 2025 21:48
@eramongodb eramongodb self-assigned this Jul 30, 2025
@eramongodb eramongodb requested a review from a team as a code owner July 30, 2025 21:48
Copy link
Collaborator

@kevinAlbs kevinAlbs left a comment

Choose a reason for hiding this comment

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

Thank you! LGTM with a possible additional cherry-pick:

could not determine which commit(s)

I expect it is 27c0581. Quoting PR:

This change appears to fix ongoing failures for the client bulkWrite in a transaction test

* capture cluster time after initial data operations
* advance cluster time in newly created session entities
* handle tests that do not specify initial data
* do not try to read cluster time from standalone
@eramongodb
Copy link
Contributor Author

I expect it is 27c0581.

Looks like that was it! Thank you. 👍

@eramongodb eramongodb merged commit 85d89f5 into mongodb:r1.30 Jul 31, 2025
1 check passed
@eramongodb eramongodb deleted the cdriver-evg-r1.30 branch July 31, 2025 17:25
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