fix(commands): Skip warmup script on --dry-run

src/commands/prune.rs

@@ -40,9 +40,13 @@ impl PruneCmd {
 
         print_stats(&prune_plan.stats);
 
-        if config.global.dry_run {
+        let dry_run = config.global.dry_run;
+        if dry_run && config.global.dry_run_warmup {
             repo.warm_up(prune_plan.repack_packs().into_iter())?;
-        } else {
+        } else if !config.global.dry_run_warmup {
+            info!("Ignoring --dry-run-warmup works only in combination with --dry-run");
+        }
+        if !dry_run {
             repo.prune(&self.opts, prune_plan)?;
         }
 

src/commands/restore.rs

@@ -94,15 +94,19 @@ impl RestoreCmd {
             info!("all file contents are fine.");
         }
 
-        if dry_run {
+        if dry_run && config.global.dry_run_warmup {
             repo.warm_up(restore_infos.to_packs().into_iter())?;
-        } else {
+        } else if !dry_run && !config.global.dry_run_warmup {
             // save some memory
             let repo = repo.drop_data_from_index();
 
             let ls = repo.ls(&node, &ls_opts)?;
             repo.restore(restore_infos, &self.opts, ls, &dest)?;
             println!("restore done.");
+        } else {
+            info!(
+                "--dry-run is without warmup, --dry-run --dry-run-warmup also issues the warmup script."
+            );
         }
 
         Ok(())

src/config.rs

@@ -176,6 +176,11 @@ pub struct GlobalOptions {
     #[merge(strategy=conflate::bool::overwrite_false)]
     pub dry_run: bool,
 
+    /// Additional to dry run, but still issue warm-up command if configured
+    #[clap(long, global = true, env = "RUSTIC_DRY_RUN_WARMUP")]
+    #[merge(strategy=conflate::bool::overwrite_false)]
+    pub dry_run_warmup: bool,
+
     /// Check if index matches pack files and read pack headers if necessary
     #[clap(long, global = true, env = "RUSTIC_CHECK_INDEX")]
     #[merge(strategy=conflate::bool::overwrite_false)]

src/snapshots/rustic_rs__config__tests__default_config_display_passes.snap

@@ -5,6 +5,7 @@ expression: config
 [global]
 use-profiles = []
 dry-run = false
+dry-run-warmup = false
 check-index = false
 no-progress = false
 

src/snapshots/rustic_rs__config__tests__default_config_passes.snap

@@ -7,6 +7,7 @@ RusticConfig {
         use_profiles: [],
         group_by: None,
         dry_run: false,
+        dry_run_warmup: false,
         check_index: false,
         log_level: None,
         log_file: None,

src/snapshots/rustic_rs__config__tests__global_env_roundtrip_passes-2.snap

@@ -5,6 +5,7 @@ expression: deserialized
 [global]
 use-profiles = []
 dry-run = false
+dry-run-warmup = false
 check-index = false
 no-progress = false
 

src/snapshots/rustic_rs__config__tests__global_env_roundtrip_passes-3.snap

@@ -7,6 +7,7 @@ RusticConfig {
         use_profiles: [],
         group_by: None,
         dry_run: false,
+        dry_run_warmup: false,
         check_index: false,
         log_level: None,
         log_file: None,

src/snapshots/rustic_rs__config__tests__global_env_roundtrip_passes.snap

@@ -5,6 +5,7 @@ expression: serialized
 [global]
 use-profiles = []
 dry-run = false
+dry-run-warmup = false
 check-index = false
 no-progress = false
 

tests/snapshots/show_config__show_config_passes.snap

@@ -5,6 +5,7 @@ expression: output
 [global]
 use-profiles = []
 dry-run = false
+dry-run-warmup = false
 check-index = false
 no-progress = false
 
@@ -19,6 +20,8 @@ run-finally = []
 [global.metrics-labels]
 
 [repository]
+repository = "/tmp/repo"
+password = "test"
 no-cache = false
 warm-up = false