Skip to content

Conversation

matthewdale
Copy link
Collaborator

GODRIVER-3636

Summary

  • Set the MONGODB_VERSION based on the task matrix variable for the FaaS and search index tasks.
  • Fixes GODRIVER-3636.

Background & Motivation

@Copilot Copilot AI review requested due to automatic review settings August 11, 2025 18:38
@matthewdale matthewdale requested a review from a team as a code owner August 11, 2025 18:38
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the MongoDB version configuration for FaaS (Functions as a Service) and search index tasks in the Evergreen CI configuration. The changes ensure that these tasks use the correct MongoDB version from the task matrix variable instead of hardcoded or incorrect values.

  • Replace hardcoded MongoDB version "7.0" with dynamic ${VERSION} variable
  • Update environment variable inclusion method for AWS credentials
  • Set MONGODB_VERSION environment variable using the matrix variable for consistency

@matthewdale matthewdale added bug review-priority-normal Medium Priority PR for Review: within 1 business day labels Aug 11, 2025
Copy link
Contributor

API Change Report

No changes found!

Copy link
Contributor

🧪 Performance Results

Commit SHA: f0b05a6

The following benchmark tests for version 689a38c3f4691b0007ab3f39 had statistically significant changes (i.e., |z-score| > 1.96):

Benchmark Measurement % Change Patch Value Stable Region H-Score Z-Score
BenchmarkSingleFindOneByID ops_per_second_min 29.7481 1651.2168 Avg: 1272.6331
Med: 1254.3529
Stdev: 177.6191
0.7495 2.1314
BenchmarkBSONDeepDocumentEncoding ops_per_second_med -6.1456 61667.4889 Avg: 65705.4695
Med: 65976.1178
Stdev: 1688.2761
0.7879 -2.3918
BenchmarkBSONDeepDocumentEncoding total_bytes_allocated -5.5750 148699280.0000 Avg: 157478764.4878
Med: 158016888.0000
Stdev: 3913942.4826
0.7763 -2.2431
BenchmarkBSONDeepDocumentEncoding ns_per_op 5.2798 17151.0000 Avg: 16290.8780
Med: 16240.5000
Stdev: 421.1800
0.7567 2.0422
BenchmarkBSONDeepDocumentEncoding total_mem_allocs -5.2571 209134.0000 Avg: 220738.4512
Med: 221452.5000
Stdev: 5448.5783
0.7625 -2.1298
BenchmarkBSONFlatDocumentDecoding total_time_seconds -0.8305 1.1860 Avg: 1.1959
Med: 1.1960
Stdev: 0.0044
0.7539 -2.2696
BenchmarkBSONDeepDocumentEncoding allocated_bytes_per_op -0.3641 2139.0000 Avg: 2146.8171
Med: 2147.0000
Stdev: 1.7006
0.8998 -4.5966

For a comprehensive view of all microbenchmark results for this PR's commit, please check out the Evergreen perf task for this patch.

@matthewdale matthewdale enabled auto-merge (squash) August 11, 2025 20:53
@matthewdale matthewdale merged commit 5b79d94 into mongodb:master Aug 11, 2025
37 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug review-priority-normal Medium Priority PR for Review: within 1 business day

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants