Build fixes (#65)

* Switch to building psycopg2. The binary wheels don't work well with docker + apple silicon * Update issue URL to work with multi-region * Fix test * Fix lint
getsentry · Dec 7, 2023 · c25225c · c25225c
1 parent 238068e
commit c25225c
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/backend-py/Dockerfile b/backend-py/Dockerfile
@@ -1,5 +1,8 @@
 FROM python:3.9-slim as backend-py
 
+RUN apt update && \
+  apt install -y python3-dev gcc libpq-dev
+
 WORKDIR /backend-py
 # Re-create the requirements layer if the requirements change
 COPY requirements.txt .

diff --git a/backend-py/requirements.txt b/backend-py/requirements.txt
@@ -19,7 +19,7 @@ packaging==21.3
 pathspec==0.9.0
 platformdirs==2.5.1
 pluggy==1.0.0
-psycopg2-binary==2.9.3
+psycopg2==2.9.3
 py==1.11.0
 pycodestyle==2.8.0
 pyflakes==2.4.0

diff --git a/backend-py/src/api/endpoints/items.py b/backend-py/src/api/endpoints/items.py
@@ -22,7 +22,9 @@ def add_sentry_api_data(
     for item in items:
         if item["sentryId"]:
             # Use the numerical ID to fetch the short ID
-            sentry_data = sentry.get(f"/issues/{item['sentryId']}/")
+            sentry_data = sentry.get(
+                f"/organizations/{organization.external_slug}/issues/{item['sentryId']}/"
+            )
             short_id = sentry_data.json().get("shortId")
             # Replace the numerical ID with the short ID
             if short_id:

diff --git a/backend-py/tests/api/items/test_index.py b/backend-py/tests/api/items/test_index.py
@@ -41,7 +41,7 @@ def test_index_with_sentry_api(self):
 
         responses.add(
             responses.GET,
-            f"{SENTRY_URL}/api/0/issues/{sentry_id}/",
+            f"{SENTRY_URL}/api/0/organizations/organization/issues/{sentry_id}/",
             body=json.dumps({"shortId": short_id}),
         )
 
@@ -57,7 +57,7 @@ def test_index_with_failing_sentry_api(self):
 
         responses.add(
             responses.GET,
-            f"{SENTRY_URL}/api/0/issues/{sentry_id}/",
+            f"{SENTRY_URL}/api/0/organizations/organization/issues/{sentry_id}/",
             body=json.dumps({}),
         )