Repository navigation
Stage examples/ and compat/ in app deploy scripts - #500
sundarleads wants to merge 1 commit into
Conversation
setup.py needs both folders to build the wheel, but deploy_app.sh and deploy_app.ps1 only copied src, demo, integration_tests and databricks_app. The wheel build in start.sh then fails on missing examples/json and examples/yml, and the app crashes right after deploy. Added both folders to the list in each script and a test that checks the lists against setup.py.
|
Thank you for the contribution and for adding coverage for the staging allow-lists. Could you please retarget this PR to the There is one additional issue to address before this can resolve the startup failure: both Please also remove the following exclusions from -compat/
-examples/It would be useful to extend the new staging test to verify that every directory required by the wheel build is not excluded by the staged |
The app crashes on start after deploying with scripts/deploy_app.sh. start.sh builds the wheel with setup.py, and setup.py now needs examples/ (for the MCP templates) and compat/. The deploy scripts don't copy either folder, so the build fails with:
This adds both folders to the list in deploy_app.sh and deploy_app.ps1, plus a small test that checks the lists against setup.py so this doesn't drift again.
Tested by deploying the app before and after the change. I haven't run the .ps1 on Windows, but it's the same two entries.