Fixes #36801: Make Redis the default cache type

theforeman · Oct 16, 2023 · fef0544 · fef0544
1 parent 623d7f6
commit fef0544
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -296,7 +296,7 @@
   Optional[Integer[0]] $foreman_service_puma_threads_min = undef,
   Integer[0] $foreman_service_puma_threads_max = 5,
   Optional[Integer[0]] $foreman_service_puma_workers = undef,
-  Hash[String, Any] $rails_cache_store = { 'type' => 'file' },
+  Hash[String, Any] $rails_cache_store = { 'type' => 'redis' },
   Boolean $keycloak = false,
   String[1] $keycloak_app_name = 'foreman-openidc',
   String[1] $keycloak_realm = 'ssl-realm',

diff --git a/spec/classes/foreman_spec.rb b/spec/classes/foreman_spec.rb
@@ -458,7 +458,13 @@
 
       describe 'with custom redis' do
         context 'with redis_url' do
-          let(:params) { super().merge(dynflow_redis_url: 'redis://127.0.0.1:4333/') }
+          let(:params) do
+            super().merge(
+              dynflow_redis_url: 'redis://127.0.0.1:4333/',
+              rails_cache_store: {type: 'redis', 'urls': ['127.0.0.1:4334']}
+            )
+          end
+
           it { should_not contain_class('redis') }
           it { should_not contain_class('redis::instance') }
         end