Skip to content

fix(builder): the uvicorn logging files are written into the project, where the build context is - #100

Merged
antoinetoussaint-byte merged 1 commit into
mainfrom
fix/probe-filter-files-come-from-the-project
Oct 21, 2026
Merged

antoinetoussaint-byte merged 1 commit into
mainfrom
fix/probe-filter-files-come-from-the-project

Conversation

@antoinetoussaint-byte

Copy link
Copy Markdown
Contributor

#99 merged its first revision: it wrote uvicorn-log-config.json and codefly_uvicorn_logging.py into the render directory and COPY'd them from the context root, but the build context is the service directory (or the assembled copy of it), so every image build on main now fails at that COPY. This writes both files into the project before any context is assembled — exactly where a native start writes them — and copies them from code/ beside src.

Verified locally with the four image-building tests (TestBuildShipsSiblingPackages, TestAssembledContextBuildsTheImage, TestImageSBOMInventoriesTheBuiltImage, TestWorkspaceMemberBuildsTheImage) and the uvicorn tests; the full suite was green on the branch #99 was meant to merge.

🤖 Generated with Claude Code

… where the build context is

#99 wrote uvicorn-log-config.json and codefly_uvicorn_logging.py into the render directory and COPY'd them from the context root; the build context is the service directory (or the assembled copy of it), so every image build failed at that COPY. They are now written into the project before any context is assembled — exactly where a native start writes them — and copied from code/ beside src.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@antoinetoussaint-byte
antoinetoussaint-byte merged commit e1a2c10 into main Oct 21, 2026
4 checks passed
@antoinetoussaint-byte
antoinetoussaint-byte deleted the fix/probe-filter-files-come-from-the-project branch October 21, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant