Skip to content

PYTHON-5931 PYTHON-5935 Fix auth-aws-ecs and kms test failures#2933

Merged
caseyclements merged 7 commits into
mongodb:mainfrom
blink1073:PYTHON-5931
Jul 17, 2026
Merged

PYTHON-5931 PYTHON-5935 Fix auth-aws-ecs and kms test failures#2933
caseyclements merged 7 commits into
mongodb:mainfrom
blink1073:PYTHON-5931

Conversation

@blink1073

@blink1073 blink1073 commented Jul 10, 2026

Copy link
Copy Markdown
Member

PYTHON-5931 / PYTHON-5935

Changes in this PR

Fixes the auth-aws-ecs and kms (Azure/GCP) Evergreen tests, which were both failing on their remote/ephemeral test hosts.

Test Plan

Verified with a passing Evergreen patch covering all affected tasks (test-auth-aws-ecs, test-gcpkms, test-azurekms, and the auth_oidc remote variants): https://spruce.corp.mongodb.com/patch/6a518b361480fd00075f1a1d

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)?
  • Is there test coverage?
  • Is any followup work tracked in a JIRA ticket? If so, add link(s).

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?

Fixes dependency resolution failure when uv needs git to fetch
the mockupdb dependency.
@blink1073
blink1073 marked this pull request as ready for review July 10, 2026 13:47
@blink1073
blink1073 requested a review from a team as a code owner July 10, 2026 13:47
@blink1073
blink1073 requested review from aclark4life and Copilot July 10, 2026 13:47

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.

Pull request overview

Updates the Evergreen ECS-host test setup for MONGODB-AWS authentication by ensuring the test container has the missing git dependency required for the auth-aws-ecs test to run successfully.

Changes:

  • Install git alongside build-essential in the ECS test bootstrap script.

@blink1073
blink1073 marked this pull request as draft July 10, 2026 14:03
@blink1073

Copy link
Copy Markdown
Member Author

This is the wrong approach - we need to pass UV_NO_LOCK to the ecs and kms environments, that's what we were missing from #2881

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Both tests fail because a local-dev-only dependency sync step runs
unguarded on remote/ephemeral hosts, forcing resolution of a git-based
test dependency that isn't needed there.
@blink1073 blink1073 changed the title PYTHON-5931 Add git to auth-aws-ecs test container PYTHON-5931 PYTHON-5935 Fix auth-aws-ecs and kms test failures Jul 10, 2026
Comment thread pyproject.toml
blink1073 and others added 5 commits July 10, 2026 18:03
Replaces the duplicated bash date computation in configure-env.sh,
justfile, setup-tests.sh, and run-tests.sh with a single
exclude-newer setting, so the cooldown also applies to ad-hoc uv
invocations that bypass those wrapper scripts.
The ecs container has no git, so it can't resolve mockupdb's git
dependency itself. Generate the lock on the host instead (where git
is available) and have the container install from it frozen.
@aclark4life
aclark4life marked this pull request as ready for review July 17, 2026 19:32

@aclark4life aclark4life 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.

LGTM

@aclark4life
aclark4life requested a review from sleepyStick July 17, 2026 19:54

@caseyclements caseyclements 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.

LGTM

@caseyclements
caseyclements merged commit c8d65e1 into mongodb:main Jul 17, 2026
33 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.

5 participants