Skip to content

Conversation

@kevinAlbs
Copy link
Contributor

@kevinAlbs kevinAlbs commented Dec 4, 2025

PYTHON-5664

Summary

Avoid using tarfile module to workaround DEVPROD-24663.

Changes in this PR

Run tar in the shell rather than tarfile module to extract the libmongocrypt tarball.

Testing Plan

Tested with this patch build: https://spruce.mongodb.com/version/693198b85d553300078e54c1. Remaining failures appear unrelated, example:

urllib.error.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1017)>

Checklist

Checklist for Author

- [ ] Did you update the changelog (if necessary)?
- [ ] Is the intention of the code captured in relevant tests?
- [ ] If there are new TODOs, has a related JIRA ticket been created?

Checklist for Reviewer {@primary_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?)
  • Have you checked for spelling & grammar errors?
  • Is all relevant documentation (README or docstring) updated?

Avoid using `tarfile` module to workaround DEVPROD-24663.
@kevinAlbs kevinAlbs changed the title extract using tar command PYTHON-5665 extract using tar command Dec 4, 2025
@kevinAlbs kevinAlbs changed the title PYTHON-5665 extract using tar command PYTHON-5664 extract using tar command Dec 4, 2025
@kevinAlbs kevinAlbs requested a review from NoahStapp December 4, 2025 15:46
@kevinAlbs kevinAlbs marked this pull request as ready for review December 4, 2025 15:46
@kevinAlbs kevinAlbs requested a review from a team as a code owner December 4, 2025 15:46
@kevinAlbs kevinAlbs merged commit 3093a7c into mongodb:master Dec 4, 2025
78 of 79 checks passed
blink1073 pushed a commit to blink1073/mongo-python-driver that referenced this pull request Dec 4, 2025
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