chore: fix ci to run on v3 docker compose langfuse server from main branch

[marcklingen]

Important

Update CI workflow to use stable Docker Compose file for Langfuse server.

• CI Workflow:
  • Updates .github/workflows/ci.yml to use docker-compose.yml instead of docker-compose.v3preview.yml for running and logging the Langfuse server.
  • Affects steps for starting the server, logging, and running the server with specific environment variables.

^{This description was created by for b9ac219. It will automatically update as commits are pushed.}

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
langfuse-js	✅ Ready (Inspect)	Visit Preview	Dec 4, 2024 5:10pm

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

Updates CI workflow to use the standard docker-compose.yml instead of docker-compose.v3preview.yml for running Langfuse server in integration tests.

• Modified .github/workflows/ci.yml to remove v3preview configuration, indicating v3 features are now stable
• Added timeout limits (3 mins for tests, 10 mins for integration tests) to prevent long-running CI jobs
• Added Node 18 and 20 matrix testing strategy for broader version compatibility

_{💡 (2/5) Greptile learns from your feedback when you react with 👍/👎!}

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

(updates since last review)

Updates CI workflow to use the stable Docker Compose configuration for Langfuse server, removing the v3preview version and improving health check monitoring.

• Added health check step in .github/workflows/ci.yml with 10 retries and 5-second intervals to ensure server availability
• Added container logs output on health check failure for better debugging
• Simplified server startup by consolidating environment variables into a single docker compose command

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

(updates since last review)

Updates CI workflow to use a more streamlined Docker Compose configuration for the Langfuse server, focusing on stability and simplified environment setup.

• Added REDIS_HOST="" to explicitly disable Redis in server configuration
• Simplified environment variables by removing ClickHouse cluster settings and async processing flags
• Added health check step with 10 retries and 5-second intervals to verify server availability
• Added container logs output on health check failure for better debugging

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

(updates since last review)

Added environment variable to ensure consistent data storage configuration in CI tests, focusing on Postgres-only operation.

• Added LANGFUSE_RETURN_FROM_CLICKHOUSE=false in .github/workflows/ci.yml to enforce Postgres data retrieval during testing

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

(updates since last review)

Based on the latest changes to the CI workflow, here's a concise summary of the most recent updates:

Updates CI workflow to use Postgres as the primary data store for testing, with improved environment configuration and health monitoring.

• Added LANGFUSE_READ_FROM_POSTGRES_ONLY=true and LANGFUSE_READ_FROM_CLICKHOUSE_ONLY=false to ensure consistent data storage
• Added LANGFUSE_SDK_CI_SYNC_PROCESSING_ENABLED=true for synchronous processing during tests
• Improved health check monitoring with retry mechanism and detailed logging
• Removed deprecated ClickHouse and Redis configurations for simplified setup

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}