Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reinterpret latest to mean "latest build" #607

Merged
merged 6 commits into from
Feb 20, 2025

Conversation

kevinAlbs
Copy link
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
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!

@kevinAlbs kevinAlbs merged commit 2faa128 into mongodb-labs:master Feb 20, 2025
72 of 73 checks passed
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