https://flyte.org logo
Join Slack
Powered by
`flytkit` seems to have a dependency `urllib3>=...
# flyte-support
f
flytkit
seems to have a dependency 
urllib3>=1.22,<2.0.0
. Is there a way to make it work with 
urllib3 > 2
? I have another library that requires 
urllib3>=2.0.7
.
t
urllib looks like it’s mainly used for the auth cycle. it should be fine.
would you mind testing though?
we will test with our internal provider as well but more coverage is better.
f
Thanks Yee. I ended up making this work by patching the other dependency, which really did not depend on url lib 2+. But that being said, I am very interested in this getting fixed in flytekit, so I don't need to maintain my own version of that other dependency.
@thankful-minister-83577 How can I test? I actually do not need url lib for anything, it was just another dependency that was conflicting. If you have a suggested way to test, I can help with that.
t
do you use auth?
if so, would you mind checking out this branch: https://github.com/flyteorg/flytekit/pull/2431
well, you’ll probably need to install that branch, confirm urllib is on 2, and then try to run something that requires auth
let’s also just see if tests pass on the pr.
f
That's the thing, we don't rely on auth right now.
t
got it
well we’ll test internally at least, make sure it works.
Open in Slack
PreviousNext
Powered by