determined-ai / determined

Determined is an open-source machine learning platform that simplifies distributed training, hyperparameter tuning, experiment tracking, and resource management. Works with PyTorch and TensorFlow.
https://determined.ai
Apache License 2.0
3k stars 350 forks source link

fix: docker version bump to unpin requests #9614

Closed MikhailKardash closed 2 months ago

MikhailKardash commented 3 months ago

Ticket

https://hpe-aiatscale.atlassian.net/browse/MD-415

Description

Python requests library update broke the docker python library. docker has since fixed the issue. See: https://github.com/docker/docker-py/pull/3257

Test Plan

CI passes

Checklist

netlify[bot] commented 3 months ago

Deploy Preview for determined-ui canceled.

Name Link
Latest commit 10b363b2703e91377a236d9f3956976731b7e411
Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/6688633bd56a830008ab6537
codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.88%. Comparing base (0a57cde) to head (10b363b).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #9614 +/- ## ========================================== - Coverage 52.89% 52.88% -0.01% ========================================== Files 1255 1255 Lines 153086 153086 Branches 3230 3230 ========================================== - Hits 80971 80959 -12 - Misses 71964 71976 +12 Partials 151 151 ``` | [Flag](https://app.codecov.io/gh/determined-ai/determined/pull/9614/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=determined-ai) | Coverage Δ | | |---|---|---| | [backend](https://app.codecov.io/gh/determined-ai/determined/pull/9614/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=determined-ai) | `43.99% <ø> (-0.03%)` | :arrow_down: | | [harness](https://app.codecov.io/gh/determined-ai/determined/pull/9614/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=determined-ai) | `72.76% <ø> (ø)` | | | [web](https://app.codecov.io/gh/determined-ai/determined/pull/9614/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=determined-ai) | `51.30% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=determined-ai#carryforward-flags-in-the-pull-request-comment) to find out more. [see 4 files with indirect coverage changes](https://app.codecov.io/gh/determined-ai/determined/pull/9614/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=determined-ai)