Skip to content

PYTHON-6170 [BF] AWS ECS test container cannot install uv - #3120

Merged
blink1073 merged 1 commit into
mongodb:mainfrom
blink1073:PYTHON-6170
Oct 14, 2026
Merged

blink1073 merged 1 commit into
mongodb:mainfrom
blink1073:PYTHON-6170

Conversation

@blink1073

@blink1073 blink1073 commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

PYTHON-6170

The auth-aws-ecs Evergreen task has failed on every commit since October 5: the ECS test container's Debian-based Python 3.12 image has neither a pip module nor ensurepip, so ensure_uv cannot install uv there.

Changes in this PR

  • Adds python3-pip to the apt package list in .evergreen/run-mongodb-aws-ecs-test.sh, so ensure_uv can install uv in the container.
  • Extends the comment above that line to explain why pip is needed.

Test Plan

  • Ran just lint and just typing locally; both pass.

  • The failure only reproduces inside the ECS container, so the targeted Evergreen patch build is the true end-to-end verification:

    evergreen patch --project mongo-python-driver --description "PYTHON-6170 fix ensure_uv in ECS container" --variants auth-aws-ecs-macos --tasks test-auth-aws-ecs --yes
    

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)? (Not necessary: Evergreen-only change, no user-facing behavior.)
  • Is there test coverage? (No unit test can reproduce the container environment; the auth-aws-ecs-macos Evergreen task verifies this change, see Test Plan.)
  • Is any followup work tracked in a JIRA ticket? If so, add link(s). (No followup work.)

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

…-pip

Commit 045ee2a (PYTHON-6051) made ensure_uv run inside the ECS test
container. Its Debian-based Python 3.12 image has neither a pip module
nor ensurepip, so uv cannot be installed there and setup exits non-zero.
Add python3-pip so ensure_uv's pip install --user branch works.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused dependency change addresses the reported bootstrap failure, with no unresolved review findings.

0 open findings

What changed in this PR

This PR updates the Evergreen AWS ECS test container setup so ensure_uv can install uv.

Changes:

  • Installs python3-pip alongside the existing system packages.
  • Explains in a comment why pip is required by the container image.
File Description
.evergreen/​run-mongodb-aws-ecs-test.sh Adds pip to the container setup and documents the dependency.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@blink1073
blink1073 marked this pull request as ready for review October 14, 2026 10:49
@blink1073
blink1073 requested a review from a team as a code owner October 14, 2026 10:49
@blink1073
blink1073 requested a review from sleepyStick October 14, 2026 10:49
@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sleepyStick sleepyStick left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ty! ^-^

@blink1073
blink1073 merged commit f3f19df into mongodb:main Oct 14, 2026
91 checks passed
@blink1073
blink1073 deleted the PYTHON-6170 branch October 14, 2026 17:52
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.

3 participants