Remove WORKSPACE from s3 image path

dnum-mi · May 22, 2024 · c4e66d1 · c4e66d1
1 parent 426235c
commit c4e66d1
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 106 deletions.
diff --git a/.github/workflows/deploy.yml b/.github/workflows/deploy.yml
diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml
@@ -62,7 +62,6 @@ jobs:
     container:
       image: ghcr.io/datalab-mi/basegun/basegun-backend:${{ github.head_ref }}
       env:
-        WORKSPACE: dev
         AWS_REGION: gra
         AWS_DEFAULT_REGION: gra
         AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}

diff --git a/backend/src/config.py b/backend/src/config.py
@@ -105,7 +105,7 @@ def get_base_logs(user_agent, user_id: str) -> dict:
 # Object storage
 S3_URL_ENDPOINT = os.environ["S3_URL_ENDPOINT"]
 S3_BUCKET_NAME = os.environ["S3_BUCKET_NAME"]
-S3_PREFIX = os.path.join("uploaded-images/", os.environ["WORKSPACE"])
+S3_PREFIX = os.path.join("uploaded-images/")
 
 S3 = boto3.resource("s3", endpoint_url=S3_URL_ENDPOINT, verify=False)
 

diff --git a/backend/src/utils.py b/backend/src/utils.py
@@ -7,9 +7,7 @@
 
 def upload_image(content: bytes, image_key: str):
     """Uploads an image to s3 bucket
-        path uploaded-images/WORKSPACE/img_name
-        where WORKSPACE is dev, preprod or prod
-
+        path uploaded-images/img_name
     Args:
         content (bytes): file content
         image_key (str): path we want to have

diff --git a/backend/tests/test_api.py b/backend/tests/test_api.py
@@ -66,8 +66,7 @@ def check_log_base(self, log):
 
     def test_upload(self):
         """Checks that the file upload works properly"""
-        if os.environ["WORKSPACE"] == "dev":
-            create_bucket()
+        create_bucket()
         path = os.path.join(os.path.dirname(os.path.abspath(__file__)), "revolver.jpg")
 
         with open(path, "rb") as f:

diff --git a/docker-compose.yml b/docker-compose.yml
@@ -19,7 +19,6 @@ services:
       - http_proxy
       - https_proxy
       - no_proxy
-      - WORKSPACE=dev
       - REQUESTS_CA_BUNDLE=$CACERT_LOCATION
     ports:
       - 5000:5000