Conversation
…ve MinIO dependencies
There was a problem hiding this comment.
Pull request overview
Updates the self-hosting/deployment configuration to v1.28.9, including changes to the LiveKit recording stack (removing the custom webhook flow and introducing new S3/RustFS wiring), plus updates to Synapse logging and new runtime environment variables.
Changes:
- Bump deployed WorkAdventure version to
v1.28.9(wa.version,.env.example,.env). - Rework meeting recording infrastructure: remove
webhook/service andegress/bin/start.sh, add RustFS + init job, and adjust egress startup/config + LiveKit template. - Adjust Synapse logging configuration and bump Synapse image version.
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
webhook/server.js |
Removes the custom LiveKit webhook HTTP server used to trigger egress. |
webhook/package.json |
Removes webhook service Node dependencies. |
webhook/Dockerfile |
Removes webhook service container build. |
wa.version |
Version bump to v1.28.9. |
synapse/log.config.yaml |
Switches to buffered logging targeting the file handler; changes SQL logger verbosity. |
livekit/livekit-config.template.yaml |
Removes LiveKit webhook configuration block. |
egress/entrypoint.sh |
Starts PulseAudio before launching egress; simplifies template substitution. |
egress/egress-config.template.yaml |
Adds cpu_cost configuration; removes S3 file_output block. |
egress/bin/start.sh |
Removes script that started track egress jobs via lk egress start. |
docker-compose.yaml |
Removes webhook + MinIO; adds RustFS + init; adjusts egress privileges and adds new env vars and routing. |
.env.example |
Adds new recording S3 env vars, P2P bandwidth env vars, and conversation bubble settings; bumps VERSION. |
.env |
Updates encrypted configuration values and adds new variables aligned with the updated compose/env example. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.