Skip to content

Reinterpret latest to mean "latest build"#607

Merged
kevinAlbs merged 6 commits intomongodb-labs:masterfrom
kevinAlbs:reinterpret-latest
Feb 20, 2025
Merged

Reinterpret latest to mean "latest build"#607
kevinAlbs merged 6 commits intomongodb-labs:masterfrom
kevinAlbs:reinterpret-latest

Conversation

@kevinAlbs
Copy link
Copy Markdown
Contributor

@kevinAlbs kevinAlbs commented Feb 18, 2025

Summary

Change latest to mean "latest build" rather than "latest release" for the downloaded server version.

Verified with this patch build in the C driver, which also includes changes from DRIVERS-3108.

Background & Motivation

Using latest as the server version currently downloads the latest release (including release candidates):

MONGODB_VERSION=latest ./.evergreen/run-orchestration.sh
mongosh --eval "db.version()"
# Prints: 8.0.5-rc2

Previously latest downloaded the latest build.

# Checkout commit before switching to mongodl.py:
git checkout 9142b7055ea5940e59ad41c4b069376f867031da
MONGODB_VERSION=latest ./.evergreen/run-orchestration.sh
mongosh --eval "db.version()"
# Prints: 8.1.0-alpha0-537-g60be604

I want drivers to test an unreleased server feature (SPM-2472). The switch from "latest build" to "latest release" seems unintended (slack). Drivers testing "latest build" seems more useful to catch test issues sooner.

Test failures

Testing this branch in the C driver resulted in two failures.

Load balancer behavior change

Latest build standalone tasks failed to run non-lb-connection-establishment.yml with error:

expected error to contain "Driver attempted to initialize in load balancing mode, but the server does not support this mode", but got: "Could not establish stream for node localhost:27017"

Filed SERVER-101078. A skip is added in DRIVERS-3108 as a short-term workaround.

macOS fail to start

A task on macOS 11 failed to start latest builds of mongod. Filed SERVER-101020.

@kevinAlbs kevinAlbs marked this pull request as ready for review February 19, 2025 20:27
@kevinAlbs kevinAlbs requested a review from blink1073 February 19, 2025 20:27
Copy link
Copy Markdown
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM!

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