Skip to content

[ML-69901] [s] Add status and logs guidance to failed Apps rollouts - #751

Draft
jamesbxwu wants to merge 1 commit into
mainfrom
codex/ml-69901-deploy-errors
Draft

jamesbxwu wants to merge 1 commit into
mainfrom
codex/ml-69901-deploy-errors

Conversation

@jamesbxwu

@jamesbxwu jamesbxwu commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

When an Apps rollout fails, agentbricks deploy streams the Apps diagnostic but ends with only Could not deploy '<name>'. A crash can suggest /logz without telling the user how to inspect the failed deployment from Agent Bricks.

Add status and log commands using the selected profile, followed by guidance to address the failure before rerunning deploy. Preserve the original error message, code, and any detail, plus the blocking streamed rollout behavior. Document the recovery in the CLI reference.

Validation: four failure-path regressions reproduce on frozen main a8d55f4 and the 0.5.1 release-tag source. All 135 focused Apps/client/deploy/auth tests pass with the fix; Ruff checks/formatting and ty pass. The related ML-69653 DEFAULT live check already verified that a real immediate startup exception exits nonzero and reports an Apps crash; no additional live App was created for this diagnostic change. Current published-wheel verification and the optional private integration runner are separate.

Tracks ML-69901.

Public CI baseline: the framework job fails the same six memory-backed parallel-history ordering cases as current main (expected last/first, observed first/last). Confirmed independently against main run https://github.com/databricks/databricks-ai-bridge/actions/runs/38662891535/job/116188048864; no forced skips. Other checks are tracked separately.

Bug-bash integration validation

The test-only combined branch at be2fa8ee8d6d3b3e011ecfa285e6455b90f9433a contains this exact PR head and eleven other reviewed heads. Its Agent Bricks unit suite passed 2,154 tests, 24 skipped. This is combined-source validation, not an isolated run of this PR; the branch must not be merged into main.

Private run 38698567494 passed all 20 live Claude cache tests at e231ca2 (OpenAI 8, LangGraph 4, generated adapters 8). Run 38701390693 at 53f9db8 passed OpenAI 8, LangGraph 4 and 3 adapter cases; the remaining 5 adapter cases returned HTTP403 because the shared test-workspace Gateway budget reached its $10 limit. Production source is identical across those runs and the final combined source; later changes affect only matrix fixtures, cleanup, regression tests and documentation. The rerun is not all green, and no budgets were changed.

The corrected Apps test passed 20/20 matrix rows, two grant snapshots, transitive-permission controls and repeat-deploy idempotency. Its evidence confirms clean source and wheel/source matching. That live run predates the final cleanup correction: the harness recorded twelve successful cleanup operations but omitted the implicit Runtime Store/role for direct-authoring Apps. #752 now includes ownership-checked cleanup for every App and safe retry/evidence handling, covered by focused regressions and the final unit suite. No new full model matrix was run for those cleanup-only changes.

Independent read-only run 38708044949 confirmed all 13 exact prior-run resources absent, including three consecutive typed404 observations for the omitted Runtime Store, dedicated database and role. One separate current-window database/role pair remains present and unattributed: SCIM did not recover its App identity and one bounded audit SELECT failed. That pair was left intact; full cleanup of the current matrix run is not established. No resources were deleted using name/time correlation alone.

The Apps matrix covers LangGraph, CLI/direct authoring, dev/deployed runtime and five tool kinds. Separate cache tests cover both LangGraph and OpenAI Agents. These checks do not establish fresh-account provisioning rights or freshly consented deployed Genie OBO acceptance.

All public checks finished. #694/#727 have 54 green; the other ten PRs have 53 green and the same framework job red. Its six memory-backed parallel-history ordering failures also occur on current main. No forced skips or weakened grant assertions were applied.

@jamesbxwu jamesbxwu changed the title [ML-69901] Add status and logs guidance to failed Apps rollouts [ML-69901] [s] Add status and logs guidance to failed Apps rollouts Oct 15, 2026

This branch has not been deployed

No deployments
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