Skip to content

Conversation

rcownie
Copy link
Collaborator

@rcownie rcownie commented Aug 6, 2025

The devprod tiger team effort made various good changes to mongosync evergreen config and scripts.

This commit adjusts the mongo-tools/mongodump_passthrough scripts to work against a pinned newer mongosync:

  • shell scripts have been moved from src/mongosync/evergreen/foo.sh to src/mongosync/evergreen/scripts/foo.sh

  • mongodump_passthrough tests now run with build variant "mongodump_passthru_v", to isolate the specifics of the
    hardware platform from the name used in scripts and configuration of Build Baron context rules

  • "mongodump_passthru_v" is a cheaper amazon2-arm64 instance type, as used in mongosync to reduce costs.

  • mongodump_passthrough jstestfuzz scripts are currently disabled. There's some issue with the new world that I
    don't have time to solve this week, and I'm on PTO next week, so this PR solves 3/4 of the problem and I'll file a
    follow-up ticket to fix jstestfuzz later.

@rcownie rcownie force-pushed the rcownie/TOOLS-3941a branch 3 times, most recently from bf612e8 to d985aaf Compare August 8, 2025 15:31
@rcownie rcownie changed the title change mongosync pin etc TOOLS-3941 - make mongodump_passthrough work with newer pinned mongosync Aug 8, 2025
@rcownie rcownie marked this pull request as ready for review August 8, 2025 15:58
@rcownie rcownie requested a review from a team as a code owner August 8, 2025 15:58
@rcownie rcownie requested review from mankawal and removed request for a team August 8, 2025 15:58
@rcownie rcownie assigned rcownie and unassigned rcownie Aug 8, 2025
Copy link
Collaborator

@mankawal mankawal left a comment

Choose a reason for hiding this comment

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

LGTM + a few questions

@rcownie rcownie requested a review from mankawal August 18, 2025 14:32
@rcownie rcownie force-pushed the rcownie/TOOLS-3941a branch from edee2f9 to 9fc08e9 Compare August 19, 2025 19:10
@rcownie rcownie force-pushed the rcownie/TOOLS-3941a branch from 9fc08e9 to 93e384e Compare August 19, 2025 19:17
@rcownie rcownie merged commit a704ee1 into master Aug 20, 2025
39 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