From 9eae3c4285d742f946af2b173e32a9e6378da8f3 Mon Sep 17 00:00:00 2001
From: Benedikt Trefzer <benedikt.trefzer@cirrax.com>
Date: Sat, 11 Dec 2021 13:56:24 +0100
Subject: [PATCH] redis::instance: do not exec cp if source and destination are
 equal

---
 manifests/instance.pp | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/manifests/instance.pp b/manifests/instance.pp
index 87bde750..50b78f43 100644
--- a/manifests/instance.pp
+++ b/manifests/instance.pp
@@ -447,12 +447,6 @@
     if $title != 'default' {
       $real_service_ensure = $service_ensure == 'running'
       $real_service_enable = $service_enable
-
-      if $notify_service {
-        Exec["copy ${redis_file_name_orig} to ${redis_file_name}"] ~> Service["${service_name}.service"]
-      } else {
-        Exec["copy ${redis_file_name_orig} to ${redis_file_name}"] -> Service["${service_name}.service"]
-      }
     } else {
       $real_service_ensure = undef
       $real_service_enable = undef
@@ -613,10 +607,20 @@
     ),
   }
 
-  exec { "copy ${redis_file_name_orig} to ${redis_file_name}":
-    path        => '/usr/bin:/bin',
-    command     => "cp -p ${redis_file_name_orig} ${redis_file_name}",
-    subscribe   => File[$redis_file_name_orig],
-    refreshonly => true,
+  if $redis_file_name_orig != $redis_file_name {
+    exec { "copy ${redis_file_name_orig} to ${redis_file_name}":
+      path        => '/usr/bin:/bin',
+      command     => "cp -p ${redis_file_name_orig} ${redis_file_name}",
+      subscribe   => File[$redis_file_name_orig],
+      refreshonly => true,
+    }
+
+    if $title != 'default' and $manage_service_file {
+      if $notify_service {
+        Exec["copy ${redis_file_name_orig} to ${redis_file_name}"] ~> Service["${service_name}.service"]
+      } else {
+        Exec["copy ${redis_file_name_orig} to ${redis_file_name}"] -> Service["${service_name}.service"]
+      }
+    }
   }
 }