config-remote-sync: report skipped/failed status in JSON output and exit gracefully - #6383
config-remote-sync: report skipped/failed status in JSON output and exit gracefully#6383ilyakuz-db wants to merge 2 commits into
Conversation
…xit gracefully Co-authored-by: Isaac <no-reply@databricks.com>
Approval status: pending
|
…tput Co-authored-by: Isaac <no-reply@databricks.com>
|
An authorized user can trigger integration tests manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
|
Superseded by #6384 — recreated from a branch in databricks/cli (not a fork) so CI can access OIDC/JFrog secrets. Fork PRs fail setup at the OIDC token step. |
Changes
bundle config-remote-syncno longer hard-fails when there is nothing to sync. In JSON output mode it now emits astatusfield (success/skipped/failed) plus anerrormessage and exits 0:--select-idsselector matched a deployed resourceText output mode is unchanged and still fails loudly (non-zero exit).
Why
The command is consumed by DABs in the Workspace through its JSON output. Treating "nothing to sync yet" (not deployed, or a stale selector) as a hard command failure is indistinguishable from a real error to the caller. Reporting it as
skippedwith a structured status lets the caller handle it gracefully. Telemetry still records the error message and category on every non-success path.Tests
Added acceptance coverage for the skipped (missing state, no selector match) and failed JSON outcomes, plus a telemetry test confirming the error payload is still reported on the graceful exit-0 path. Existing JSON goldens updated.
This pull request and its description were written by Isaac.