Skip to content

Conversation

blink1073
Copy link
Member

  • Break up the test matrix to cover all server versions, python versions, platforms, topologies, and auth/ssl with as little repetition as possible
  • Break up sync and async tests on MacOS and Windows

Before: 1026 tasks across 39 build variants
After: 495 tasks across 37 build variants

image

@blink1073 blink1073 requested a review from NoahStapp October 15, 2024 15:38
@blink1073
Copy link
Member Author

I'll need to update the GitHub PR aliases once this is merged.

@blink1073
Copy link
Member Author

I need to add coverage...

Copy link
Contributor

@NoahStapp NoahStapp left a comment

Choose a reason for hiding this comment

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

One clarification, otherwise looks great! This will be so nice for testing going forward

- name: .sharded_cluster
display_name: Test Win32 py3.9 NoAuth NoSSL Sync
run_on:
- windows-64-vsMulti-small
Copy link
Contributor

Choose a reason for hiding this comment

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

We run 32-bit python on a 64-bit machine?

Copy link
Member Author

Choose a reason for hiding this comment

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

32-bit versions of Python. No one runs 32 bit machines anymore lol.

Copy link
Member Author

Choose a reason for hiding this comment

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

But some apps still require 32 bit

Copy link
Contributor

Choose a reason for hiding this comment

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

Just wanted to make sure, makes sense lol

@blink1073
Copy link
Member Author

evergreen retry

2 similar comments
@blink1073
Copy link
Member Author

evergreen retry

@blink1073
Copy link
Member Author

evergreen retry

@blink1073 blink1073 merged commit 1b6c0d3 into mongodb:master Oct 15, 2024
53 of 70 checks passed
@blink1073 blink1073 deleted the PYTHON-4868 branch October 15, 2024 18:33
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