JAVA-6284 add server 9.0 test variant to evergreen - #2037
Conversation
strogiyotec
changed the title
|
Assigned |
There was a problem hiding this comment.
🔵 Needs a closer look
It changes human-review-only Evergreen and release-versioning configuration, with the version bump outside the stated scope.
Pull request overview
Adds MongoDB 9.0 coverage to the Evergreen test matrix while selecting compatible Ubuntu hosts.
Changes:
- Adds MongoDB 9.0 to applicable test variants.
- Splits Ubuntu 20.04 and 22.04 matrix variants.
- Bumps the driver snapshot version outside the stated scope.
File summaries
| File | Description |
|---|---|
.evergreen/.evg.yml |
Adds 9.0 coverage and Ubuntu-specific matrices. |
gradle.properties |
Changes the snapshot version to 5.11.0. |
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| # | ||
|
|
||
| version=5.10.0-SNAPSHOT | ||
| version=5.11.0-SNAPSHOT |
nhachicha
left a comment
nhachicha
left a comment
There was a problem hiding this comment.
LGTM, only left over in gradle.properties to be reverted
| # | ||
|
|
||
| version=5.10.0-SNAPSHOT | ||
| version=5.11.0-SNAPSHOT |
| matrix_spec: { ssl: "nossl", jdk: [ "jdk8", "jdk17", "jdk21" ], version: [ "4.4", "5.0", "6.0", "7.0", "8.0", "latest" ], os: "ubuntu", | ||
| matrix_spec: { ssl: "nossl", jdk: [ "jdk8", "jdk17", "jdk21" ], version: [ "6.0", "7.0", "8.0", "9.0", "latest" ], os: "ubuntu2204", | ||
| aws-credential-provider: "*" } | ||
| display_name: "MONGODB-AWS Basic Auth test ${version} ${jdk} ${aws-credential-provider}" |
There was a problem hiding this comment.
nit: include ${os} name
| display_name: "MONGODB-AWS Basic Auth test ${version} ${jdk} ${aws-credential-provider}" | |
| display_name: "MONGODB-AWS Basic Auth test ${version} ${jdk} ${aws-credential-provider} ${os}" |
| - matrix_name: "aws-ec2-auth-test" | ||
| matrix_spec: { ssl: "nossl", jdk: [ "jdk21" ], version: [ "7.0" ], os: "ubuntu", aws-credential-provider: "*" } | ||
| matrix_spec: { ssl: "nossl", jdk: [ "jdk21" ], version: [ "7.0" ], os: "ubuntu2204", aws-credential-provider: "*" } | ||
| display_name: "MONGODB-AWS Advanced Auth test ${version} ${jdk} ${aws-credential-provider}" |
There was a problem hiding this comment.
nit: include ${os} name
| display_name: "MONGODB-AWS Advanced Auth test ${version} ${jdk} ${aws-credential-provider}" | |
| display_name: "MONGODB-AWS Advanced Auth test ${version} ${jdk} ${aws-credential-provider} ${os}" |
| matrix_spec: { ssl: "nossl", jdk: [ "jdk21" ], version: [ "7.0" ], os: "ubuntu", aws-credential-provider: "*" } | ||
| matrix_spec: { ssl: "nossl", jdk: [ "jdk21" ], version: [ "7.0" ], os: "ubuntu2204", aws-credential-provider: "*" } | ||
| display_name: "MONGODB-AWS Advanced Auth test ${version} ${jdk} ${aws-credential-provider}" | ||
| run_on: "ubuntu2204-small" |
There was a problem hiding this comment.
nit: could be removed similar to https://github.com/mongodb/mongo-java-driver/pull/2037/changes#diff-bcd2c099e589f1ef43957739e25b7ac58b6a847e92cc1e5cc9d9c91acb2fe9f9L2506 (L2506) since os is already supplied in the matrix
| run_on: "ubuntu2204-small" |
JAVA-6284 add server 9.0 test variant to evergreen
Also because 9.0 is not publishing ubuntu2004 version the ubuntu matrix was split into two
The server versions 4.4 and 5.0 work with 20.04