-
Notifications
You must be signed in to change notification settings - Fork 456
use debian11-small and install UV
#2163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The debian11 distro provides a python3 system binary with virtualenv support.
Instead of restoring old install scripts, suggest implementing the following testazurekms-task-specific workaround prior to dependent scripts and commands:
# Temporary workaround for lack of uv on `debian11`.
uv_dir="$(mktemp -d)"
python3 -m virtualenv "${uv_dir:?}"
(. "${uv_dir:?}/bin/activate" && python -m pip install uv)
PATH="${uv_dir:?}/bin:${PATH:-}"
command -V uv # uv is hashed (/tmp/.../bin/uv)
uv --version # 0.9.5There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated (and thank you for the much simpler suggestion!).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor suggestion; otherwise, LGTM.
Co-authored-by: Ezra Chung <88335979+eramongodb@users.noreply.github.com>
Summary
Switch
testazurekms-taskfromdebian11-latest-smalltodebian11-smallto fix set-up failure.Tested with this patch build: https://spruce.mongodb.com/version/690221a09c315e0007b2bcd1
Background & Motivation
#2127 switched the
testazurekms-taskfromdebian11-smalltodebian11-latest-small. Theaztool is too old on thedebian11-latest-small, triggering a failure (example):Upgrading
azis requested in DEVPROD-23011. This PR proposes reverting todebian11-smalland restoring the UV install scripts removed in #2127 in the short term. A newerazis also needed for Azure testing for OIDC.