Skip to content

[DABs] config-remote-sync: skip stale --select-ids selectors instead of failing the batch#5980

Draft
ilyakuz-db wants to merge 4 commits into
mainfrom
configsync-fix-type5-no-deployed-id
Draft

[DABs] config-remote-sync: skip stale --select-ids selectors instead of failing the batch#5980
ilyakuz-db wants to merge 4 commits into
mainfrom
configsync-fix-type5-no-deployed-id

Conversation

@ilyakuz-db

Copy link
Copy Markdown
Contributor

Changes

bundle config-remote-sync --select-ids <type:id>,... is how DABs in the Workspace syncs edited resources — the UI batches every edited resource into one call. Previously, if any one selector matched no deployed resource in state, the command errored with no deployed <type> resource with id <id> and the whole batch failed, dropping the valid resources' edits too.

A selector that matches no deployed resource (deleted remotely, or deploy-state drift) is now skipped — a resource with no deployed state has no remote change to pull back into config. A malformed selector (bad <type>:<id> shape) is still an error.

Why

A single stale resource id was causing every sync to fail for the affected workspace, so no UI edit could be written back.

Tests

Extended the select_basic acceptance test: a batch with one stale selector still syncs the valid resource; an all-stale sync is a clean no-op; a malformed selector is still rejected.

@eng-dev-ecosystem-bot

eng-dev-ecosystem-bot commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Integration test report

Commit: 968cd77

Run: 29837325404

Env 🟨​KNOWN 💚​RECOVERED 🙈​SKIP ✅​pass 🙈​skip Time
💚​ aws-cli-is linux 4 4 314 1044 3:53
💚​ aws-cli-is windows 4 4 316 1042 4:29
💚​ azure-cli-is linux 4 4 314 1043 4:14
💚​ azure-cli-is windows 4 4 316 1041 4:31
🟨​ gcp-cli-is linux 3 1 4 313 1045 7:43
🟨​ gcp-cli-is windows 3 1 4 315 1043 8:37
8 interesting tests: 4 SKIP, 3 KNOWN, 1 RECOVERED
Test Name aws-cli-is linux aws-cli-is windows azure-cli-is linux azure-cli-is windows gcp-cli-is linux gcp-cli-is windows
💚​ TestAccept 💚​R 💚​R 💚​R 💚​R 💚​R 💚​R
🙈​ TestAccept/bundle/invariant/no_drift 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_endpoints/drift/recreated_same_name 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/bundle/resources/vector_search_indexes/recreate/embedding_dimension 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🙈​ TestAccept/ssh/connection 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S 🙈​S
🟨​ TestFetchRepositoryInfoAPI_FromRepo 💚​R 💚​R 💚​R 💚​R 🟨​K 🟨​K
🟨​ TestFetchRepositoryInfoAPI_FromRepo/root 💚​R 💚​R 💚​R 💚​R 🟨​K 🟨​K
🟨​ TestFetchRepositoryInfoAPI_FromRepo/subdir 💚​R 💚​R 💚​R 💚​R 🟨​K 🟨​K
Top 4 slowest tests (at least 2 minutes):
duration env testname
2:14 aws-cli-is windows TestAccept/bundle/deploy/mlops-stacks/DATABRICKS_BUNDLE_ENGINE=terraform
2:05 aws-cli-is windows TestAccept
2:04 gcp-cli-is windows TestAccept
2:03 azure-cli-is windows TestAccept/bundle/deploy/mlops-stacks/DATABRICKS_BUNDLE_ENGINE=terraform

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.

2 participants