Comment 4 for bug 1688464

Revision history for this message
Michele Baldessari (michele) wrote : Re: new puppet-redis ulimit change broke ci

So as soon as the puppet-redis issue is fixed, we likely need to do the following:
1) Remove the file limit the redis pcmk profile:
diff --git a/manifests/profile/pacemaker/database/redis.pp b/manifests/profile/pacemaker/database/redis.pp
index 3ef6815..4ede81b 100644
--- a/manifests/profile/pacemaker/database/redis.pp
+++ b/manifests/profile/pacemaker/database/redis.pp
@@ -60,12 +60,12 @@ class tripleo::profile::pacemaker::database::redis (
     # we best explicitely set the file limit only in the pacemaker profile
     # (the base profile does not need it as it is using systemd which has
     # the limits set there)
- file { '/etc/security/limits.d/redis.conf':
- content => inline_template("redis soft nofile <%= @redis_file_limit %>\nredis hard nofile <%= @redis_file_limit %>\n"),
- owner => '0',
- group => '0',
- mode => '0644',
- }

As it would create a duplicate resource

2) Set the newly introduced hiera key in tht:
diff --git a/puppet/services/pacemaker/database/redis.yaml b/puppet/services/pacemaker/database/redis.yaml
index e702d28..8229fc2 100644
--- a/puppet/services/pacemaker/database/redis.yaml
+++ b/puppet/services/pacemaker/database/redis.yaml
@@ -37,5 +37,6 @@ outputs:
           - get_attr: [RedisBase, role_data, config_settings]
           - redis::service_manage: false
             redis::notify_service: false
+ redis::managed_by_cluster_manager: true
       step_config: |
         include ::tripleo::profile::pacemaker::database::redis