From 39d8bcad80a688680c9d7dfd53dc09ed2aa003f3 Mon Sep 17 00:00:00 2001 From: Deimos Date: Mon, 14 Jan 2019 16:33:12 -0700 Subject: [PATCH] Install Sentry SDK with other dependencies Previously I was using Salt to install the Sentry SDK (previously known as "raven") only on the production server, but that's not really necessary. This will just install it everywhere, and then we'll only actually integrate it in production. --- salt/salt/raven.sls | 6 ------ salt/salt/top.sls | 1 - tildes/requirements-to-freeze.txt | 1 + tildes/requirements.txt | 1 + 4 files changed, 2 insertions(+), 7 deletions(-) delete mode 100644 salt/salt/raven.sls diff --git a/salt/salt/raven.sls b/salt/salt/raven.sls deleted file mode 100644 index e2183d1..0000000 --- a/salt/salt/raven.sls +++ /dev/null @@ -1,6 +0,0 @@ -{% from 'common.jinja2' import app_dir, venv_dir %} - -raven-pip-install: - pip.installed: - - name: raven - - bin_env: {{ venv_dir }} diff --git a/salt/salt/top.sls b/salt/salt/top.sls index 3e189c4..216ba3f 100644 --- a/salt/salt/top.sls +++ b/salt/salt/top.sls @@ -30,7 +30,6 @@ base: 'prod': - nginx.shortener-config - nginx.static-sites-config - - raven - prod-config 'monitoring': - nginx diff --git a/tildes/requirements-to-freeze.txt b/tildes/requirements-to-freeze.txt index eabeba2..79a097a 100644 --- a/tildes/requirements-to-freeze.txt +++ b/tildes/requirements-to-freeze.txt @@ -30,6 +30,7 @@ pytest pytest-mock PyYAML # needs to be installed separately for webassets qrcode +sentry-sdk SQLAlchemy SQLAlchemy-Utils stripe diff --git a/tildes/requirements.txt b/tildes/requirements.txt index c07f815..3287b41 100644 --- a/tildes/requirements.txt +++ b/tildes/requirements.txt @@ -80,6 +80,7 @@ redis==3.0.1 repoze.lru==0.7 requests==2.21.0 requirements-detector==0.6 +sentry-sdk==0.6.6 setoptconf==0.2.0 simplejson==3.16.0 six==1.12.0