fix(deploy): restore shared-deployment namespace merge (src/__init__.py collision) - #825
Merged
Merged
Conversation
…__init__.py cockpit/langgraph/client-tools shipped an empty src/__init__.py (#642) that turned its src into a regular package; per PEP 420 that wins over every other dep's namespace portion, so c-interrupts' 'from src.aviation_tools import' raised ModuleNotFoundError at startup and every revision since Aug 7 hit DEPLOY_FAILED (production smoke timeouts were the wedged run queue). Delete the file and make the manifest generator throw if any staged dep reintroduces one. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
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.
Production smoke has been failing with all-graph run timeouts. Root cause (via LangGraph Cloud revision logs): every revision since Aug 7 is DEPLOY_FAILED with
Graph 'c-interrupts' failed to load: ModuleNotFoundError: No module named 'src.aviation_tools'— repeated failed rollouts today then wedged the run queue on the old revision.cockpit/langgraph/client-tools/python/src/__init__.py(an empty license-header file shipped in #642) turned that dep'ssrcinto a regular package. Per PEP 420 a regular package wins exclusively over all namespace portions, so on the shared deployment's aggregated sys.path every other dep'sfrom src.x import ...stopped resolving. Reproduced locally against the stageddeps/tree; after deleting the file,src.aviation_tools+src.graphimport cleanly across the merged namespace.__init__.py(nothing importssrcas a regular package; graph entrypoints load by file path)generate-shared-deployment-config.tsnow throws at manifest-generation time if any staged dep reintroduces asrc/__init__.py, so this class of breakage fails in CI instead of at LangGraph Cloud startupMerging this touches the deploy workflow's trigger paths, so
Deploy LangGraphwill roll a fresh revision; production smoke should recover once it's live.🤖 Generated with Claude Code