Skip to content

[fix] Resolve corrupted docker files#4355

Merged
junaway merged 2 commits into
release/v0.99.10from
fix/resolve-corrupted-docker-files
May 18, 2026
Merged

[fix] Resolve corrupted docker files#4355
junaway merged 2 commits into
release/v0.99.10from
fix/resolve-corrupted-docker-files

Conversation

@junaway
Copy link
Copy Markdown
Contributor

@junaway junaway commented May 18, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 18, 2026 16:16
@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment May 18, 2026 4:19pm

Request Review

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label May 18, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 18, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b247cc2a-a99f-45b1-8a9a-aaccde64a43a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/resolve-corrupted-docker-files

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reverts the SDK/client injection workaround by making the Python api and services images build from the repo root, which matches how their Dockerfile.gh files already reference ./api/..., ./services/..., ./sdks/python, and ./clients/python. This removes the brittle copy-before-build step in CI and local helpers.

Changes:

  • Switch all gh/gh.local/gh.ssl/ee.gh compose files and the Railway build script to use the repo root as build context with the api/... / services/... Dockerfile path prefix.
  • Remove the SDK/client copy + cleanup trap logic from run.sh and build-and-push-images.sh.
  • Update OSS and EE README docs to describe the new root-context build model.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
hosting/railway/oss/scripts/build-and-push-images.sh Drops SDK/client copy + trap; builds api & services with repo root context.
hosting/railway/oss/README.md Documents that no SDK injection step is needed.
hosting/docker-compose/run.sh Removes the gh.local SDK/client copy block and cleanup trap.
hosting/docker-compose/oss/docker-compose.gh.yml Updates commented-out build sections to root context.
hosting/docker-compose/oss/docker-compose.gh.ssl.yml Switches api/services builds to root context.
hosting/docker-compose/oss/docker-compose.gh.local.yml Switches api/services builds to root context.
hosting/docker-compose/tmp/docker-compose.ee.gh.override.yml Switches EE api/services overrides to root context.
hosting/docker-compose/ee/docker-compose.gh.local.yml Switches EE api/services builds to root context.
hosting/docker-compose/ee/README.md Documents repo-root build context for API & Services.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@junaway junaway merged commit 430b7a1 into release/v0.99.10 May 18, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants