Skip to content

Commit

Permalink
Make app dependencies more explicit (#160)
Browse files Browse the repository at this point in the history
* Remove tornado dependency

* Switch to PyPI version of kombu

* Ensure pip gets re-built if venv is deleted
  • Loading branch information
c-w authored Mar 15, 2019
1 parent 00b3c53 commit 90de4c2
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 11 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,5 @@ venv*/
.coverage
cover/
.mypy_cache/
requirements.txt.out

serviceprincipal.json
2 changes: 1 addition & 1 deletion docker/app/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ RUN groupadd -r opwen \
USER opwen

ENV TESTING_UI="False"
ENV CONNEXION_SERVER="tornado"
ENV CONNEXION_SERVER="flask"
ENV CONNEXION_SPEC="SET_ME"
ENV SERVER_WORKERS="1"
ENV QUEUE_WORKERS="1"
Expand Down
12 changes: 6 additions & 6 deletions makefile
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,13 @@ py_env=venv
.PHONY: venv tests
default: ci

requirements.txt.out: requirements.txt requirements-dev.txt requirements-prod.txt
if [ ! -d $(py_env) ]; then python3 -m venv $(py_env) && $(py_env)/bin/pip install -U pip wheel | tee requirements.txt.out; fi
$(py_env)/bin/pip install -r requirements.txt | tee requirements.txt.out
$(py_env)/bin/pip install -r requirements-dev.txt | tee requirements.txt.out
$(py_env)/bin/pip install -r requirements-prod.txt | tee requirements.txt.out
$(py_env)/requirements.txt.out: requirements.txt requirements-dev.txt requirements-prod.txt
if [ ! -d $(py_env) ]; then python3 -m venv $(py_env) && $(py_env)/bin/pip install -U pip wheel | tee $(py_env)/requirements.txt.out; fi
$(py_env)/bin/pip install -r requirements.txt | tee $(py_env)/requirements.txt.out
$(py_env)/bin/pip install -r requirements-dev.txt | tee $(py_env)/requirements.txt.out
$(py_env)/bin/pip install -r requirements-prod.txt | tee $(py_env)/requirements.txt.out

venv: requirements.txt.out
venv: $(py_env)/requirements.txt.out

tests: venv
$(py_env)/bin/coverage run -m nose2 && $(py_env)/bin/coverage report
Expand Down
1 change: 0 additions & 1 deletion requirements-prod.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,2 @@
azure-servicebus==0.50.0
gunicorn==19.9.0
tornado==6.0.1
4 changes: 2 additions & 2 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,14 @@ https://github.com/CatalystCode/libcloud/archive/1037_fix-azurite.zip
applicationinsights==0.11.7
beautifulsoup4==4.7.1
cached-property==1.5.1
connexion[swagger-ui]==2.2.0
connexion[flask,swagger-ui]==2.2.0
environs==4.1.0
msgpack==0.6.1
pyzmail36==1.0.4
requests==2.21.0
sendgrid==5.6.0
typing-extensions==3.7.2
typing==3.6.6
https://github.com/celery/kombu/archive/483cadced77d82a6ecd0be553b91ce92f04f9617.zip
kombu==4.4.0
celery==4.2.1
xtarfile[zstd]==0.0.2

0 comments on commit 90de4c2

Please sign in to comment.