[fix] Resolve absolute binaries in railway#3968
[fix] Resolve absolute binaries in railway#3968junaway wants to merge 2 commits intofix/local-web-testsfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dosubot
bot
added
the
size:XXL
dosubot
bot
added
size:M
There was a problem hiding this comment.
Pull request overview
This PR adjusts Railway OSS deployment wrappers to use absolute executable paths (e.g., /opt/venv/bin/python, /opt/venv/bin/gunicorn, /usr/local/bin/supercronic) to avoid redeploy/start failures caused by command resolution issues, and simplifies the web start command to rely on the base image entrypoint.
Changes:
- Update API/services/workers to invoke
/opt/venv/bin/*binaries directly (python/gunicorn). - Update cron to invoke supercronic via
/usr/local/bin/supercronic. - Simplify web startup to
node /app/oss/server.js(avoids invoking/app/entrypoint.shexplicitly).
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| hosting/railway/oss/api/Dockerfile | Use /opt/venv/bin/gunicorn explicitly for Railway. |
| hosting/railway/oss/services/Dockerfile | Use /opt/venv/bin/gunicorn explicitly for Railway. |
| hosting/railway/oss/worker-webhooks/Dockerfile | Use /opt/venv/bin/python explicitly for Railway. |
| hosting/railway/oss/worker-tracing/Dockerfile | Use /opt/venv/bin/python explicitly for Railway. |
| hosting/railway/oss/worker-events/Dockerfile | Use /opt/venv/bin/python explicitly for Railway. |
| hosting/railway/oss/worker-evaluations/Dockerfile | Use /opt/venv/bin/python explicitly for Railway. |
| hosting/railway/oss/cron/Dockerfile | Use /usr/local/bin/supercronic explicitly. |
| hosting/railway/oss/alembic/Dockerfile | Use /bin/sh -c plus absolute venv python paths for migration runner. |
| hosting/railway/oss/scripts/deploy-from-images.sh | Generate wrapper Dockerfiles with absolute executable paths. |
| hosting/railway/oss/web/Dockerfile | Run node /app/oss/server.js directly (base image entrypoint handles setup). |
| hosting/railway/oss/web/railway.json | Align Railway start command to node /app/oss/server.js. |
| hosting/railway/oss/README.md | Document absolute-path requirement and the Railway exec lookup behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Railway Preview Environment
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 12 out of 12 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
No description provided.