Closed tmwoodruff closed 9 months ago
@tmwoodruff Just FYI I've noticed this PR and think it's interesting, but I think the problem causing the tests to fail needs to be properly resolved before progressing
@gchazot - Added fixes for the tests. This required
cython<3
for https://github.com/yaml/pyyaml/issues/601requests
and urllib3
for https://github.com/docker/docker-py/issues/3113For https://github.com/docker/docker-py/issues/3113, we could alternatively upgrade the docker
lib to 6.1.0, but that looks like a much bigger change.
https://github.com/EDITD/kubetools/pull/159 @tmwoodruff I've extracted the dependencies fixes and documented them in a separate PR
I've force pushed a rebase of this on top of master
now that #159 was merged
Nice one @tmwoodruff Thanks for the reviews @DilaraOflaz
I will release this now
Test failures are unrelated to this change. I ran into this issue a couple of weeks ago. Seems to be an issue with new versions of cython and pyaml: https://github.com/yaml/pyyaml/issues/601
Haven't had time to look into solutions.