Skip to content

Add --dry-run flag to sync and bundle-sync commands #2657

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

anton-107
Copy link
Contributor

@anton-107 anton-107 commented Apr 4, 2025

Why

Dry-run flag allows users to preview the operations that a sync operation would do without performing the file operations.

Usage:

$ databricks sync <SRC> <DEST> --dry-run
$ databricks bundle sync --dry-run

This preview is especially handy to preview the changes when using the recently introduced (#2650) --include and --exclude flags to the sync operations

Tests

  1. Manual testing
  2. Added new acceptance tests
  3. Modified existing unit test for syncing to test dry run mode

@anton-107 anton-107 temporarily deployed to test-trigger-is April 4, 2025 13:47 — with GitHub Actions Inactive
@anton-107 anton-107 changed the title Add include/exclude flags to sync and bundle-sync commands add --dry-run flag to sync and bundle-sync commands Apr 4, 2025
@anton-107 anton-107 changed the title add --dry-run flag to sync and bundle-sync commands Add --dry-run flag to sync and bundle-sync commands Apr 4, 2025
@anton-107 anton-107 force-pushed the anton-107/sync-dryrun branch from 90b489c to 0d34783 Compare April 4, 2025 14:12
@anton-107 anton-107 temporarily deployed to test-trigger-is April 4, 2025 14:12 — with GitHub Actions Inactive
@anton-107 anton-107 marked this pull request as ready for review April 4, 2025 14:39
@anton-107 anton-107 force-pushed the anton-107/sync-dryrun branch from 0d34783 to 8907ca5 Compare April 15, 2025 07:54
@anton-107 anton-107 temporarily deployed to test-trigger-is April 15, 2025 07:54 — with GitHub Actions Inactive
@anton-107 anton-107 force-pushed the anton-107/sync-dryrun branch from 8907ca5 to c53a2f7 Compare April 15, 2025 12:43
@anton-107 anton-107 temporarily deployed to test-trigger-is April 15, 2025 12:43 — with GitHub Actions Inactive
@anton-107 anton-107 temporarily deployed to test-trigger-is April 15, 2025 12:55 — with GitHub Actions Inactive
@anton-107 anton-107 temporarily deployed to test-trigger-is April 15, 2025 13:12 — with GitHub Actions Inactive
@anton-107 anton-107 requested a review from denik April 15, 2025 14:26
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