Skip to content

Commit

Permalink
[Requirements] Extras addition followup fixes (#632)
Browse files Browse the repository at this point in the history
  • Loading branch information
Hedingber committed Dec 31, 2020
1 parent bebfe4d commit 10cd91c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion dockerfiles/jupyter/Dockerfile
Expand Up @@ -23,7 +23,7 @@ COPY ./dockerfiles/jupyter/requirements.txt /tmp
RUN python -m pip install -r /tmp/requirements.txt && rm -f /tmp/requirements.txt

COPY . /tmp/mlrun
RUN cd /tmp/mlrun && python -m pip install ".[complete]" && cd /tmp && rm -rf mlrun
RUN cd /tmp/mlrun && python -m pip install ".[complete-api]" && cd /tmp && rm -rf mlrun

RUN chown -R $NB_UID:$NB_GID $HOME

Expand Down
5 changes: 0 additions & 5 deletions dockerfiles/mlrun-api/requirements.txt
@@ -1,9 +1,4 @@
uvicorn~=0.12.0
# dask-kubernetes 0.11.0 has distributed>=2.5.2, but after 2.30.1 they moved to CalVer and released 2020.12.0
# so without our limitation to <3, 2020.12.0 is installed which is incompatible since it has dask>=2020.12.0 while ours
# is ~=2.12
# TODO: dask-kubernetes will probably release 0.11.1 with a fix for this soon and this could be removed
distributed>=2.5.2, <3
dask-kubernetes~=0.11.0
# 3.0 iguazio system is running k8s 1.17 so ideally we would use 17.X, but kfp limiting to <12.0
kubernetes-asyncio~=11.0
Expand Down
5 changes: 5 additions & 0 deletions requirements.txt
Expand Up @@ -29,6 +29,11 @@ mergedeep~=1.3
v3io-frames~=0.8.5
semver~=2.13
dask~=2.12
# dask-kubernetes 0.11.0 has distributed>=2.5.2, but after 2.30.1 they moved to CalVer and released 2020.12.0
# so without our limitation to <3, 2020.12.0 is installed which is incompatible since it has dask>=2020.12.0 while ours
# is ~=2.12
# TODO: dask-kubernetes will probably release 0.11.1 with a fix for this soon and this could be removed
distributed>=2.5.2, <3
# 3.0 iguazio system is running k8s 1.17 so ideally we would use 17.X, but kfp limiting to <12.0
kubernetes~=11.0
# TODO: move to API requirements (shouldn't really be here, the sql run db using the API sqldb is preventing us from
Expand Down

0 comments on commit 10cd91c

Please sign in to comment.