Rdb save incremental fsync

spec/classes/redis_spec.rb

@@ -1680,7 +1680,7 @@
         }
       end
 
-      describe 'test rdb-save-incremental-fsync for redis6' do
+      describe 'test rdb-save-incremental-fsync enabled for redis6' do
         let(:params) do
           {
             rdb_save_incremental_fsync: true,
@@ -1690,6 +1690,26 @@
         it { is_expected.to contain_file(config_file_orig).with('content' => %r{^rdb-save-incremental-fsync yes$}) }
       end
 
+      describe 'test rdb-save-incremental-fsync disabled for redis6' do
+        let(:params) do
+          {
+            rdb_save_incremental_fsync: false,
+          }
+        end
+
+        it { is_expected.to contain_file(config_file_orig).with('content' => %r{^rdb-save-incremental-fsync no$}) }
+      end
+
+      describe 'test rdb-save-incremental-fsync Undef for redis6' do
+        let(:params) do
+          {
+            rdb_save_incremental_fsync: Undef,
+          }
+        end
+
+        it { is_expected.not_to contain_file(config_file_orig).with('content' => %r{^rdb-save-incremental-fsync}) }
+      end
+
       describe 'test systemd service timeouts' do
         let(:params) do
           {

templates/redis.conf.epp

@@ -1030,8 +1030,9 @@ aof-rewrite-incremental-fsync <%= bool2str($aof_rewrite_incremental_fsync, 'yes'
 # the file will be fsync-ed every 32 MB of data generated. This is useful
 # in order to commit the file to the disk more incrementally and avoid
 # big latency spikes.
-<% if $rdb_save_incremental_fsync { -%>rdb-save-incremental-fsync <%= $rdb_save_incremental_fsync ? {true => 'yes', default => 'no'} %><% } -%>
-
+<% unless $rdb_save_incremental_fsync =~ Undef{ -%>
+rdb-save-incremental-fsync <%= bool2str($rdb_save_incremental_fsync, 'yes', 'no') -%>
+<% } %>
 # Redis Cluster Settings
 <% if $cluster_enabled { -%>
 cluster-enabled yes