revert app_insight for backend (#44)

Co-authored-by: Olesia Solonko <olesia.solonko@ngi.no>

Author: solonkoolesya

README.md

@@ -29,7 +29,6 @@ When deploying in different environments (local, Kubernetes, etc.), the followin
 - `PYGEOAPI_SERVER_URL`: The external URL where pygeoapi will be accessible. This is used to generate correct links in the API responses.
   - For local development: `http://localhost:5000`
   - For Kubernetes/production: Your domain, e.g., `https://your-domain.com/api`
-- `APPLICATIONINSIGHTS_CONNECTION_STRING`: Connection string to Azure Application Insights. If not specifies, telemetry disabled, no user statistic gathered.
 
 #### Frontend:
 

docker-compose.yml

@@ -52,7 +52,6 @@ services:
       - "8080:80"
     environment:
       - API_BASE_URL=http://backend-local:5000
-      - APPLICATIONINSIGHTS_CONNECTION_STRING=InstrumentationKey=b4d987bc-1712-4026-aafb-f590fd854ab3;IngestionEndpoint=https://norwayeast-0.in.applicationinsights.azure.com/;LiveEndpoint=https://norwayeast.livediagnostics.monitor.azure.com/;ApplicationId=9fc4692c-7609-4abb-b547-49203ac14d4a
     depends_on:
       - backend-local
     profiles:

pygeoapi/docker/Dockerfile.pygeoapi

@@ -21,7 +21,6 @@ RUN apt-get update \
     zlib1g-dev \
     libprotobuf-dev \
     protobuf-compiler \
-    python3-venv \
     && git clone https://github.com/mapbox/tippecanoe.git /tmp/tippecanoe \
     && cd /tmp/tippecanoe \
     && make -j \
@@ -31,13 +30,6 @@ RUN apt-get update \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/*
 
-RUN python3 -m venv /opt/venv \
-    && /opt/venv/bin/python -m pip install --upgrade pip setuptools wheel
-ENV PATH="/opt/venv/bin:$PATH"
-
-# Install Application Insights for Python
-RUN pip install opencensus-ext-azure opencensus-ext-flask opencensus-ext-requests
-
 # Build version string
 ARG REPO_VERSION
 RUN PY_VER=$(python3 -c 'import pkg_resources;print(pkg_resources.get_distribution("pygeoapi").version)') \
@@ -46,9 +38,6 @@ RUN PY_VER=$(python3 -c 'import pkg_resources;print(pkg_resources.get_distributi
 # Create readiness indicator directory
 RUN mkdir -p /tmp/health
 
-# Add Application Insights instrumentation script
-COPY pygeoapi/docker/appinsights_init.py /appinsights_init.py
-
 # Add init script
 COPY pygeoapi/docker/init.sh /init.sh
 RUN chmod +x /init.sh
@@ -57,9 +46,6 @@ RUN chmod +x /init.sh
 COPY pygeoapi/docker/healthcheck.sh /healthcheck.sh
 RUN chmod +x /healthcheck.sh
 
-# Environment variable for Application Insights
-ENV APPLICATIONINSIGHTS_CONNECTION_STRING=""
-
 HEALTHCHECK --interval=15s --timeout=10s --start-period=180s --retries=10 CMD ["/healthcheck.sh"]
 
 ENTRYPOINT ["/init.sh"]

pygeoapi/docker/appinsights_init.py

@@ -20,14 +20,6 @@ fi
 PYGEOAPI_VERSION=$(pygeoapi --version 2>&1)
 echo "$(get_timestamp) [init] PyGeoAPI version: $PYGEOAPI_VERSION"
 
-# Check Application Insights configuration
-if [ -n "$APPLICATIONINSIGHTS_CONNECTION_STRING" ]; then
-    echo "$(get_timestamp) [init] Application Insights connection string found, enabling telemetry"
-    export PYTHONPATH="/appinsights_init.py:$PYTHONPATH"
-else
-    echo "$(get_timestamp) [init] No Application Insights connection string found, running without telemetry"
-fi
-
 # Remove any existing readiness indicator
 rm -f /tmp/health/init_complete
 echo "$(get_timestamp) [init] Removed any existing readiness indicators"
@@ -52,58 +44,6 @@ mkdir -p /tmp/health
 touch /tmp/health/init_complete
 echo "$(get_timestamp) [init] Initialization complete"
 
-# Start pygeoapi with Application Insights if configured
-if [ -n "$APPLICATIONINSIGHTS_CONNECTION_STRING" ]; then
-    echo "$(get_timestamp) [init] Starting pygeoapi service with Application Insights"
-
-    # First, generate the OpenAPI document that pygeoapi needs
-    echo "$(get_timestamp) [init] Generating OpenAPI document"
-    pygeoapi openapi generate /pygeoapi/local.config.yml --output-file /tmp/openapi.yml
-
-    # Create a startup script that integrates Application Insights
-    cat > /tmp/start_with_insights.py << 'EOF'
-import os
-import sys
-
-# Add root to Python path to import appinsights_init
-sys.path.insert(0, '/')
-
-try:
-    import appinsights_init
-    print("[start_with_insights] Application Insights module loaded")
-
-    # Import prebuilt Flask app instance
-    from pygeoapi.flask_app import APP as app
-    print("[start_with_insights] Flask app imported successfully")
-
-    # Setup Application Insights telemetry
-    appinsights_init.setup_app_insights(app)
-    print("[start_with_insights] Application Insights integrated")
-
-except ImportError as e:
-    print(f"[start_with_insights] ImportError: {e}")
-    sys.exit(1)
-
-except Exception as e:
-    print(f"[start_with_insights] Error during telemetry setup: {e}")
-    import traceback
-    traceback.print_exc()
-
-# Run the app (development mode only)
-if __name__ == '__main__':
-    print("[start_with_insights] Starting Flask app")
-    app.run(host='0.0.0.0', port=5000)
-
-EOF
-
-    # Execute the Application Insights setup (but don't start the app yet)
-    python3 /tmp/start_with_insights.py
-
-else
-    echo "$(get_timestamp) [init] Starting pygeoapi service without Application Insights"
-    echo "$(get_timestamp) [init] Generating OpenAPI document"
-    pygeoapi openapi generate /pygeoapi/local.config.yml --output-file /tmp/openapi.yml
-    # Start the pygeoapi service
-    echo "$(get_timestamp) [init] Starting pygeoapi service"
-    exec /entrypoint.sh "$@"
-fi
+# Start the pygeoapi service
+echo "$(get_timestamp) [init] Starting pygeoapi service"
+exec /entrypoint.sh "$@"