1. 15 Jan, 2019 1 commit
  2. 16 Jun, 2018 2 commits
  3. 11 Jul, 2017 3 commits
  4. 07 Jul, 2017 1 commit
    • Peter Souter's avatar
      Adds defined type for mult-instancing (#200) · 385d4f0c
      Peter Souter authored
      * Move the config logic into a defined type
      * This will allow us to have multiple instances
      of redis on one machine
      * Example code:
      ```
      # Install redis package
      class { '::redis':
        default_install => false,
      }
      
      redis::instance {'redis1':
        port     => '7777',
      }
      
      redis::instance {'redis2':
        port     => '8888',
      }
      ```
      385d4f0c
  5. 05 Jul, 2017 2 commits
  6. 03 Jul, 2017 2 commits
  7. 01 Jul, 2017 1 commit
  8. 19 May, 2017 8 commits
  9. 18 May, 2017 2 commits
    • Peter Souter's avatar
      Adds log to not manage /var/run/redis with systemd (#204) · 63251bcf
      Peter Souter authored
      * The existence of /var/run/redis causes the startup of redis-server via
      systemd to fail on debian systems with redis-server >= 3.2.8-2:
      
      ```
      May 18 19:17:54 debian-8 systemd[1]: Starting Advanced key-value store...
             May 18 19:17:54 debian-8 systemd[803]: Failed at step RUNTIME_DIRECTORY spawning /bin/run-parts: File exists
             May 18 19:17:54 debian-8 systemd[804]: Failed at step RUNTIME_DIRECTORY spawning /usr/bin/redis-server: File exists
             May 18 19:17:54 debian-8 systemd[1]: redis-server.service: control process exited, code=exited status=233
             May 18 19:17:54 debian-8 systemd[1]: Failed to start Advanced key-value store.
             May 18 19:17:54 debian-8 systemd[1]: Unit redis-server.service entered failed state.
             May 18 19:17:54 debian-8 systemd[1]: redis-server.service holdoff time over, scheduling restart.
      ```
      * It will actually eventually start, but the first run will fail
      * The default unit file for redis-server in debian jessie-backports
      contains `RuntimeDirectory=redis`, see [this commit](https://github.com/lamby/pkg-redis/commit/1cecea5abcb2ce05beacd4347977634d06c59cef).
      * Adds logic to not manage the /var/run/redis directory if systemd present
      63251bcf
    • Alfredo Moralejo's avatar
      Ignore selinux default context for /etc/systemd/system/redis.service.d (#202) · 464a04b9
      Alfredo Moralejo authored
      When creating new files/directory/symlink, puppet asumes that it's a file
      to get the default context. This is wrong as the default context depends
      on the file type. See https://tickets.puppetlabs.com/browse/PUP-7559
      
      For the redis case, when creating /etc/systemd/system/redis.service.d
      it's setting the context redis_unit_file_t, which is wrong. It should
      be systemd_unit_file_t. This has two efects, it's setting the directory
      to a wrong context on first run and it's breaking the idempotency of
      puppet-redis.
      
      With this patch, puppet-redis is ignoring selinux context for the directory.
      Note that the default context is applied automatically by the OS, so it
      will be set to the right one.
      464a04b9
  10. 11 May, 2017 2 commits
  11. 05 May, 2017 2 commits
    • Peter Souter's avatar
      Update sort to specify key (#199) · b4cfc1b1
      Peter Souter authored
      * With the previous approach, error on 1.8.7 Ruby:
      ```
      Error: Failed to parse template redis/redis.conf.2.4.10.erb:
        Filepath: /etc/puppet/modules/redis/templates/redis.conf.2.4.10.erb
        Line: 87
        Detail: undefined method `<=>' for nil:NilClass
       at /etc/puppet/modules/redis/manifests/config.pp:91 on node master
      ```
      * For ruby 1.8 compatibility
      * Should fix older Puppet on systems that have 
      1.8.7 Ruby (RHEL6/CentOS6)
      * We're not officially supporting 1.8.7 as it's 
      EOL, but since this is a minor change, isn't too 
      much of a maintenance cost, lets do it! 💃
      b4cfc1b1
    • Peter Souter's avatar
      Adds tests for when Redis service is unmanaged (#198) · 965a8889
      Peter Souter authored
      * If the redis service is unmanaged, like so:
      ```
      redis::managed_by_cluster_manager": true
      redis::service_manage": false
      redis::notify_service": false
      ```
      then you get a compilation failure as it tries to
      look for the redis service to notify:
      ```
      But now an 'include ::redis' gives me:
      Error: Could not find dependent Service[redis] for Augeas[Systemd redis ulimit] at /etc/puppet/modules/redis/manifests/ulimit.pp:44
      ```
      * In fact, we don't need any service notification 
      in the redis::ulimit class, as it's already in the
      redis::config class, which notifies the service 
      class anyways
      * Adds spec for this use-case to avoid regression
      
      Closes #197
      965a8889
  12. 04 May, 2017 4 commits
    • Peter Souter's avatar
      Improves ulimit configuration (#192) · f69f2dda
      Peter Souter authored
      * Changing Travis back to Trusty
      
      * Improves ulimit configuration
      
      * Moves to a dedicated class to DRY up config.pp
      * Adds support for systemd systems:
       - /etc/systemd/system/redis.service.d/limit.conf
      * Add support for redis when used with a cluster
      manager such as pacemaker or rgmanager
      * New parameter `managed_by_cluster_manager`
      * This manages the file under:
      ```
      /etc/security/limits.d/redis.conf
      ```
      * Closes #130
      f69f2dda
    • Peter Souter's avatar
      Changing Travis back to Trusty (#194) · 9d42a488
      Peter Souter authored
      9d42a488
    • Peter Souter's avatar
      Remove service notification (#193) · fe5964d8
      Peter Souter authored
      * This is done at a class level in init.pp
      * Do not need to specify individual service
      fe5964d8
    • Peter Souter's avatar
      Bump to version 3.0.0-RC (#186) · ab9b4c31
      Peter Souter authored
      Release candidate for the 3.0.0 release
      
      This will be the last release that supports 
      Puppet 3.X
      
      Version 4.x of the module onward will remove 3.X support
      ab9b4c31
  13. 03 May, 2017 10 commits