[OPIK-256]: remove sourcemaps;

comet-ml · Feb 27, 2025 · 04d0a37 · 04d0a37
1 parent d32f639
commit 04d0a37
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 23 deletions.
diff --git a/.github/workflows/build_and_push_docker.yaml b/.github/workflows/build_and_push_docker.yaml
@@ -54,22 +54,14 @@ jobs:
           run: |
             DOCKER_IMAGE_NAME=${{env.DOCKER_REGISTRY}}/${{ inputs.image }}:${{inputs.version}}
             echo "DOCKER_IMAGE_NAME=${DOCKER_IMAGE_NAME}" | tee -a $GITHUB_ENV
-            DOCKER_BUILDKIT=1 \
-            docker build \
-            --build-arg OPIK_VERSION=${{ inputs.version }} \
-            --build-arg SENTRY_AUTH_TOKEN=${{ secrets.SENTRY_AUTH_TOKEN }} \
-            -t ${DOCKER_IMAGE_NAME} .
+            DOCKER_BUILDKIT=1 docker build --build-arg OPIK_VERSION=${{inputs.version}} -t ${DOCKER_IMAGE_NAME} .
 
         - name: Build Docker Image for Comet integration
           if: inputs.build_comet_image 
           run: |
             DOCKER_IMAGE_NAME_COMET=${{env.DOCKER_REGISTRY}}/${{inputs.image}}-comet:${{inputs.version}}
             echo "DOCKER_IMAGE_NAME_COMET=${DOCKER_IMAGE_NAME_COMET}" | tee -a $GITHUB_ENV
-            DOCKER_BUILDKIT=1 \
-            docker build \
-            --build-arg OPIK_VERSION=${{ inputs.version }} \
-            --build-arg SENTRY_AUTH_TOKEN=${{ secrets.SENTRY_AUTH_TOKEN }} \
-            -t ${DOCKER_IMAGE_NAME} .
+            DOCKER_BUILDKIT=1 docker build --build-arg OPIK_VERSION=${{inputs.version}} -t ${DOCKER_IMAGE_NAME} .
             
         - name: Login to GHCR
           uses: docker/login-action@v3

diff --git a/apps/opik-frontend/Dockerfile b/apps/opik-frontend/Dockerfile
@@ -22,19 +22,6 @@ ENV NODE_OPTIONS="--max-old-space-size=8192"
 ARG BUILD_MODE=production
 RUN npm run build -- --mode $BUILD_MODE
 
-# Upload source maps to Sentry if token is provided
-RUN if [ -n "$SENTRY_AUTH_TOKEN" ] && [ -n "$OPIK_VERSION" ]; then \
-      echo "Uploading source maps to Sentry..."; \
-      npx sentry-cli --auth-token "$SENTRY_AUTH_TOKEN" \
-        sourcemaps upload \
-        --org "$SENTRY_ORG" \
-        --project "$SENTRY_PROJECT" \
-        --release "$OPIK_VERSION" \
-        /opt/frontend/dist; \
-    else \
-      echo "Source maps for Sentry will not be uploaded. Missing required environment variables."; \
-    fi
-
 # Production stage
 FROM nginx:1.27.4-alpine3.21