Skip to content

Comments

Add UV_REQUEST_TIMEOUT argument/envvar to building CI/PROD images#38467

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:add-uv-request-timeout-parameter
Mar 25, 2024
Merged

Add UV_REQUEST_TIMEOUT argument/envvar to building CI/PROD images#38467
potiuk merged 1 commit intoapache:mainfrom
potiuk:add-uv-request-timeout-parameter

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Mar 25, 2024

UV has smaller timeout than PIP and in some cases it might be too low. This PR adds flag with ENVVAR that allows to set custom value for the envvar while building the image.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added area:dev-tools area:production-image Production image improvements and fixes kind:documentation labels Mar 25, 2024
@potiuk potiuk requested a review from uranusjr March 25, 2024 17:40
Copy link
Member

@uranusjr uranusjr left a comment

Choose a reason for hiding this comment

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

A couple of nits. lgtm.

@potiuk potiuk force-pushed the add-uv-request-timeout-parameter branch from 9351add to 6cb669c Compare March 25, 2024 21:03
UV has smaller timeout than PIP and in some cases it might be too
low. This PR adds flag with ENVVAR that allows to set custom
value for the envvar while building the image.
@potiuk potiuk force-pushed the add-uv-request-timeout-parameter branch from 6cb669c to 1f0a18d Compare March 25, 2024 21:03
@potiuk potiuk merged commit 0942ae0 into apache:main Mar 25, 2024
@potiuk potiuk deleted the add-uv-request-timeout-parameter branch March 25, 2024 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants