libs/sync: extract FileList to decouple file listing from syncing by pietern · Pull Request #5947 · databricks/cli

pietern · 2026-07-16T15:57:12Z

Why

The code in libs/sync conflated listing the files selected for sync (git-aware + include/exclude) with syncing them (snapshot diff, upload, progress). Callers that only want the list had to reach through the syncer: #5254 faked a partial Sync to call the listing method, and validate:files_to_sync builds a full Sync via sync.New, running side effects a validation pass shouldn't (writes .databricks/.gitignore, can MkdirsByPath the remote path). This extracts the real primitive so listing no longer depends on a syncer.

What

Add FileList (libs/sync/filelist.go): owns the filesets and the (git ∪ include) − exclude merge. NewFileList(...) builds it, Files(ctx) lists.
Sync composes a *FileList; (*Sync).GetFileList delegates to it.
Remove the free GetFileList; its one caller (snapshot zip builder) builds a FileList directly.
Drop the implicit ["."] default: empty paths now select nothing, matching the default engine. Every real caller already passes paths explicitly.

Behavior

Pure refactor, except an immutable-folder deploy with sync: {paths: []} now zips zero files instead of the whole root, aligning it with the default engine.

This pull request and its description were written by Isaac.

The code in libs/sync conflated listing the files selected for sync (git-aware + include/exclude) with syncing them (snapshot diff, upload, progress). Callers that only want the list had to reach through the syncer: #5254 faked a partial Sync to call the listing method, and validate:files_to_sync builds a full Sync via sync.New, running side effects a validation pass shouldn't (writes .databricks/.gitignore, can MkdirsByPath the remote path). This extracts the real primitive so listing no longer depends on a syncer. - Add FileList (libs/sync/filelist.go): owns the filesets and the (git union include) minus exclude merge. NewFileList(...) builds it, Files(ctx) lists. - Sync composes a *FileList; (*Sync).GetFileList delegates to it. - Remove the free GetFileList; its one caller (snapshot zip builder) builds a FileList directly. - Drop the implicit ["."] default: empty paths now select nothing, matching the default engine. Every real caller already passes paths explicitly. Pure refactor, except an immutable-folder deploy with sync: {paths: []} now zips zero files instead of the whole root, aligning it with the default engine. Co-authored-by: Isaac

Group each stanza as fetch, error-check, apply with a blank line between sections; the git stanza was missing the blank line the include/exclude stanzas already had. Co-authored-by: Isaac

eng-dev-ecosystem-bot · 2026-07-16T16:28:48Z

Integration test report

Commit: 4e6ec12

Run: 29513952846

	Env	🔄flaky	💚RECOVERED	🙈SKIP	✅pass	🙈skip	Time
🔄	aws linux	1	4	4	226	1119	9:43
💚	aws windows		4	4	229	1117	6:41
💚	aws-ucws linux		4	4	316	1035	6:24
💚	aws-ucws windows		4	4	318	1033	7:54
💚	azure linux		4	4	227	1118	5:17
💚	azure windows		4	4	229	1116	6:47
🔄	azure-ucws linux	1	3	4	318	1032	8:14
💚	azure-ucws windows		4	4	320	1030	8:11
💚	gcp linux		4	4	226	1120	4:54
💚	gcp windows		4	4	228	1118	6:49

9 interesting tests: 4 SKIP, 3 RECOVERED, 2 flaky

	Test Name	aws linux	aws windows	aws-ucws linux	aws-ucws windows	azure linux	azure windows	azure-ucws linux	azure-ucws windows	gcp linux	gcp windows
💚	TestAccept	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R
🙈	TestAccept/bundle/invariant/no_drift	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S
🙈	TestAccept/bundle/resources/vector_search_endpoints/drift/recreated_same_name	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S
🙈	TestAccept/bundle/resources/vector_search_indexes/recreate/embedding_dimension	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S
🙈	TestAccept/ssh/connection	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S	🙈S
🔄	TestClustersGet	🔄f	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p
🔄	TestFetchRepositoryInfoAPI_FromRepo	💚R	💚R	💚R	💚R	💚R	💚R	🔄f	💚R	💚R	💚R
💚	TestFetchRepositoryInfoAPI_FromRepo/root	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R
💚	TestFetchRepositoryInfoAPI_FromRepo/subdir	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R	💚R

Top 11 slowest tests (at least 2 minutes):

duration	env	testname
6:52	azure-ucws windows	TestAccept
6:46	aws linux	TestSecretsPutSecretStringValue
6:35	aws-ucws windows	TestAccept
6:23	gcp windows	TestAccept
6:22	azure windows	TestAccept
6:17	aws windows	TestAccept
2:57	azure linux	TestAccept
2:53	gcp linux	TestAccept
2:51	azure-ucws linux	TestAccept
2:51	aws linux	TestAccept
2:48	aws-ucws linux	TestAccept

eng-dev-ecosystem-bot · 2026-07-17T08:08:37Z

Integration test report

Commit: 4c07b27

Run: 29559059195

	Env	❌FAIL	🟨KNOWN	🔄flaky	💚RECOVERED	🙈SKIP	✅pass	🙈skip	Time
❌	aws linux	6	1	2	3	4	518	1048	48:00
🔄	aws windows			4	4	4	479	1061	65:20
🔄	aws-ucws linux			5	5	2	999	857	90:18
💚	aws-ucws windows				6	2	945	875	108:02
🔄	azure linux			3	3	4	518	1049	42:48
💚	azure windows				4	4	477	1062	55:01
🔄	azure-ucws linux			3	4	2	897	892	79:36
🔄	azure-ucws windows			3	5	2	838	910	99:23
❌	gcp linux	507	2			4	28	1019	3:26
❌	gcp windows	464	2			4	28	1032	8:04

526 interesting tests: 515 FAIL, 4 flaky, 3 RECOVERED, 2 KNOWN, 2 SKIP

	Test Name	aws linux	aws windows	aws-ucws linux	aws-ucws windows	azure linux	azure windows	azure-ucws linux	azure-ucws windows	gcp linux	gcp windows
🟨	TestAccept	🟨K	💚R	🔄f	💚R	🔄f	💚R	🔄f	🔄f	🟨K	🟨K
❌	TestAccept/bundle/apps/compute_size	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/apps/compute_size/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/apps/compute_size/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/config-remote-sync/cli_defaults	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/cli_defaults/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/cli_defaults/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/config_edits	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/config_edits/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/config_edits/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/dashboard_etag	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/dashboard_etag/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/dashboard_etag/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/flushed_cache	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/flushed_cache/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/flushed_cache/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/formatting_preserved	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/formatting_preserved/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/formatting_preserved/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/job_multiple_tasks	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/job_multiple_tasks/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/job_multiple_tasks/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/job_pipeline_task	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/job_pipeline_task/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/job_pipeline_task/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/multiple_files	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/multiple_files/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/multiple_files/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/output_json	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/output_json/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/output_json/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/output_no_changes	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/output_no_changes/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/output_no_changes/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/policy_injected_cluster_fields	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/policy_injected_cluster_fields/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/policy_injected_cluster_fields/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/select_basic	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/select_basic/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/select_basic/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/select_multiple	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/select_multiple/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/select_multiple/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/target_override	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/target_override/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/target_override/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/task_rename_revert	✅p	🙈s	✅p	🙈s	✅p	🙈s	✅p	🙈s	❌F	🙈s
❌	TestAccept/bundle/config-remote-sync/task_rename_revert/DATABRICKS_BUNDLE_ENGINE=direct	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/config-remote-sync/task_rename_revert/DATABRICKS_BUNDLE_ENGINE=terraform	✅p		✅p		✅p		✅p		❌F
❌	TestAccept/bundle/deploy/empty-bundle	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/empty-bundle/DATABRICKS_BUNDLE_ENABLE_EXPERIMENTAL_YAML_SYNC=/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/empty-bundle/DATABRICKS_BUNDLE_ENABLE_EXPERIMENTAL_YAML_SYNC=/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/empty-bundle/DATABRICKS_BUNDLE_ENABLE_EXPERIMENTAL_YAML_SYNC=true/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/empty-bundle/DATABRICKS_BUNDLE_ENABLE_EXPERIMENTAL_YAML_SYNC=true/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/files/no-snapshot-sync	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/files/no-snapshot-sync/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/files/no-snapshot-sync/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/snapshot-comparison	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/snapshot-comparison/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/spark-jar-task	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/alert	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/alert/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/alert/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/cluster	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/cluster/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/cluster/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/dashboard	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/dashboard/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/dashboard/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/dashboard/recreation	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/dashboard/recreation/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/dashboard/recreation/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/experiment	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/experiment/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/experiment/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/generate-and-bind	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/generate-and-bind/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/generate-and-bind/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/job-abort-bind	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/job-abort-bind/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/job-abort-bind/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/job-spark-python-task	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/job-spark-python-task/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/job/job-spark-python-task/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/model-serving-endpoint	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/model-serving-endpoint/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/model-serving-endpoint/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/pipelines/recreate	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/pipelines/recreate/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/bind/pipelines/recreate/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/unbind/permissions	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/unbind/permissions/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/deployment/unbind/permissions/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/destroy/jobs-and-pipeline	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/destroy/jobs-and-pipeline/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/destroy/jobs-and-pipeline/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/alert	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/alert/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/alert/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/auto-bind	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/auto-bind/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/pipeline_and_deploy	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/pipeline_and_deploy/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/pipeline_and_deploy/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/python_job_and_deploy	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/python_job_and_deploy/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/generate/python_job_and_deploy/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
💚	TestAccept/bundle/invariant/no_drift	🙈S	🙈S	💚R	💚R	🙈S	🙈S	💚R	💚R	🙈S	🙈S
❌	TestAccept/bundle/local_state_staleness	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/local_state_staleness/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/local_state_staleness/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/alerts/basic	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/alerts/basic/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/alerts/basic/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/alerts/with_file	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/alerts/with_file/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/alerts/with_file/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/inline_config	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/inline_config/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started-omitted	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started-omitted/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started-terraform-error	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started-terraform-error/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started-toggle	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/data_security_mode	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/data_security_mode/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/data_security_mode/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/local_ssd_count	🙈s	🙈s	🙈s	🙈s	🙈s	🙈s	🙈s	🙈s	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/local_ssd_count/DATABRICKS_BUNDLE_ENGINE=direct									❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/local_ssd_count/DATABRICKS_BUNDLE_ENGINE=terraform									❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/simple	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/simple/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/simple/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/update-after-create	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/update-after-create/DATABRICKS_BUNDLE_ENGINE=direct	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/deploy/update-after-create/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/lifecycle-started	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/lifecycle-started-terraform-error	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F
❌	TestAccept/bundle/resources/clusters/lifecycle-started-terraform-error/DATABRICKS_BUNDLE_ENGINE=terraform	✅p	✅p	✅p	✅p	✅p	✅p	✅p	✅p	❌F	❌F

Top 50 slowest tests (at least 2 minutes):

duration	env	testname
14:23	azure-ucws linux	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
13:38	aws-ucws linux	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
12:55	aws linux	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
11:58	azure linux	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
10:42	aws-ucws linux	TestAccept/bundle/config-remote-sync/multiple_resources/DATABRICKS_BUNDLE_ENGINE=terraform
9:43	aws linux	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=terraform
8:58	aws linux	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct
8:35	aws-ucws linux	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
8:08	aws-ucws windows	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
8:06	azure windows	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
8:00	aws linux	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
7:53	azure windows	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
7:51	aws linux	TestAccept/bundle/resources/clusters/deploy/update-after-create/DATABRICKS_BUNDLE_ENGINE=terraform
7:51	azure-ucws windows	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
7:47	aws windows	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
7:35	azure-ucws windows	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
7:12	aws-ucws windows	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
7:11	azure-ucws windows	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
7:07	aws linux	TestAccept/bundle/resources/clusters/resize-terminated-fallback/DATABRICKS_BUNDLE_ENGINE=direct
7:07	aws-ucws windows	TestAccept/bundle/invariant/no_drift/DATABRICKS_BUNDLE_ENGINE=direct/INPUT_CONFIG=cluster.yml.tmpl/READPLAN=1
7:00	aws windows	TestAccept/bundle/resources/clusters/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
6:59	aws-ucws windows	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
6:59	aws-ucws linux	TestAccept/bundle/resources/clusters/deploy/simple/DATABRICKS_BUNDLE_ENGINE=direct
6:57	azure-ucws linux	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
6:55	aws-ucws linux	TestAccept/bundle/resources/clusters/deploy/data_security_mode/DATABRICKS_BUNDLE_ENGINE=direct
6:54	azure linux	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
6:53	azure-ucws linux	TestAccept/bundle/resources/apps/lifecycle-started/DATABRICKS_BUNDLE_ENGINE=direct
6:50	aws-ucws windows	TestAccept/bundle/invariant/no_drift/DATABRICKS_BUNDLE_ENGINE=direct/INPUT_CONFIG=cluster.yml.tmpl/READPLAN=
6:44	azure-ucws linux	TestAccept/bundle/config-remote-sync/multiple_resources/DATABRICKS_BUNDLE_ENGINE=terraform
6:40	azure windows	TestAccept
6:39	aws windows	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
6:36	aws-ucws windows	TestAccept
6:32	azure-ucws linux	TestAccept/bundle/resources/clusters/deploy/update-after-create/DATABRICKS_BUNDLE_ENGINE=terraform
6:18	aws-ucws linux	TestAccept/bundle/resources/clusters/deploy/update-after-create/DATABRICKS_BUNDLE_ENGINE=direct
5:59	aws-ucws linux	TestAccept/bundle/config-remote-sync/multiple_resources/DATABRICKS_BUNDLE_ENGINE=direct
5:48	azure windows	TestAccept/bundle/resources/clusters/deploy/update-after-create/DATABRICKS_BUNDLE_ENGINE=direct
5:20	aws-ucws linux	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
5:17	aws windows	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=terraform
5:07	azure windows	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
5:03	aws linux	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
4:54	aws-ucws linux	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct
4:51	aws windows	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct
4:36	azure-ucws windows	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=terraform
4:14	azure-ucws linux	TestAccept/bundle/config-remote-sync/multiple_resources/DATABRICKS_BUNDLE_ENGINE=direct
4:11	azure linux	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=terraform
4:09	azure windows	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct
4:03	azure linux	TestAccept/bundle/resources/clusters/lifecycle-started-toggle/DATABRICKS_BUNDLE_ENGINE=direct
3:57	azure-ucws windows	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct
3:55	azure linux	TestAccept/bundle/deploy/spark-jar-task/DATABRICKS_BUNDLE_ENGINE=direct
3:33	azure-ucws linux	TestSQLExecScalar

(381 table rows omitted to keep the report under 60000 bytes)

pietern added 2 commits July 16, 2026 15:55

libs/sync: consistent blank-line grouping in FileList.Files

4e6ec12

Group each stanza as fetch, error-check, apply with a blank line between sections; the git stanza was missing the blank line the include/exclude stanzas already had. Co-authored-by: Isaac

pietern requested review from andrewnester and janniklasrose July 16, 2026 16:06

andrewnester approved these changes Jul 16, 2026

View reviewed changes

pietern added this pull request to the merge queue Jul 17, 2026

Merged via the queue into main with commit 4c07b27 Jul 17, 2026
25 checks passed

pietern deleted the extract-sync-filelist branch July 17, 2026 06:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

libs/sync: extract FileList to decouple file listing from syncing#5947

libs/sync: extract FileList to decouple file listing from syncing#5947
pietern merged 2 commits into
mainfrom
extract-sync-filelist

pietern commented Jul 16, 2026

Uh oh!

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

Uh oh!

Uh oh!

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

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

pietern commented Jul 16, 2026

Why

What

Behavior

Uh oh!

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

Integration test report

Uh oh!

Uh oh!

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

Integration test report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants