chore(ci): unquarantine media download test

[hassiebp]

Important

Unquarantines a media reference test and updates CI configuration for LANGFUSE_S3_EVENT_UPLOAD_ENDPOINT.

• Tests:
  • Unquarantines the test replace media reference string in object in langfuse-integration-fetch.spec.ts by removing .skip.
• CI Configuration:
  • Changes LANGFUSE_S3_EVENT_UPLOAD_ENDPOINT from http://host.docker.internal:9090 to http://0.0.0.0:9090 in ci.yml.

^{This description was created by for 5bf0271. 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 12, 2024 9:52am

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

This PR enables media download testing in the integration suite by fixing networking configuration and unquarantining a previously skipped test case.

• Changed S3 event upload endpoint from host.docker.internal:9090 to 0.0.0.0:9090 in CI workflow to fix container networking
• Added test case for media reference string replacement that verifies audio file handling through base64 encoding/decoding
• Test validates proper media reference format and consistent handling across multiple traces
• Ensures media references maintain data integrity when stored and retrieved from traces

_{2 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 configuration to use localhost:9090 instead of 0.0.0.0:9090 for the S3 media upload endpoint and adds health check verification for the Langfuse server.

• Added detailed health check step in CI workflow to verify server availability before running tests
• Changed LANGFUSE_S3_MEDIA_UPLOAD_ENDPOINT to use localhost:9090 for more reliable container networking
• Added retry logic with max 10 attempts and 5-second intervals for server health verification
• Added container logs output on health check failure for better debugging

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