update CHANGELOG and CONTRIBUTORS

Author: lucabelluccini

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## x.x.x (Month Day, Year)
 
 #### Fixes
+* Update of the roles, roles mappings and users will not trigger service restart (if enabled).
 
 #### Features
 

CONTRIBUTORS

@@ -31,3 +31,4 @@ Rhommel Lamas (rhoml)
 Jose Luis Ledesma (sp-joseluis-ledesma)
 Matthias Baur (baurmatt)
 Gavin Williams (fatmcgav)
+Luca Belluccini (lucabelluccini)

manifests/instance.pp

@@ -383,39 +383,38 @@
       notify       => $notify_service,
     }
     file { "${configdir}/users_roles":
-      ensure       => 'file',
-      source       => "${elasticsearch::configdir}/users_roles",
-      purge        => $elasticsearch::purge_configdir,
-      force        => $elasticsearch::purge_configdir,
-      require      => File[$configdir],
-      before       => Elasticsearch::Service[$name],
+      ensure  => 'file',
+      source  => "${elasticsearch::configdir}/users_roles",
+      purge   => $elasticsearch::purge_configdir,
+      force   => $elasticsearch::purge_configdir,
+      require => File[$configdir],
+      before  => Elasticsearch::Service[$name],
     }
     file { "${configdir}/roles.yml":
-      ensure       => 'file',
-      source       => "${elasticsearch::configdir}/roles.yml",
-      purge        => $elasticsearch::purge_configdir,
-      force        => $elasticsearch::purge_configdir,
-      require      => File[$configdir],
-      before       => Elasticsearch::Service[$name],
+      ensure  => 'file',
+      source  => "${elasticsearch::configdir}/roles.yml",
+      purge   => $elasticsearch::purge_configdir,
+      force   => $elasticsearch::purge_configdir,
+      require => File[$configdir],
+      before  => Elasticsearch::Service[$name],
     }
     file { "${configdir}/role_mapping.yml":
-      ensure       => 'file',
-      source       => "${elasticsearch::configdir}/role_mapping.yml",
-      purge        => $elasticsearch::purge_configdir,
-      force        => $elasticsearch::purge_configdir,
-      require      => File[$configdir],
-      before       => Elasticsearch::Service[$name],
+      ensure  => 'file',
+      source  => "${elasticsearch::configdir}/role_mapping.yml",
+      purge   => $elasticsearch::purge_configdir,
+      force   => $elasticsearch::purge_configdir,
+      require => File[$configdir],
+      before  => Elasticsearch::Service[$name],
     }
     file { "${configdir}/users":
-      ensure       => 'file',
-      source       => "${elasticsearch::configdir}/users",
-      purge        => $elasticsearch::purge_configdir,
-      force        => $elasticsearch::purge_configdir,
-      require      => File[$configdir],
-      before       => Elasticsearch::Service[$name],
+      ensure  => 'file',
+      source  => "${elasticsearch::configdir}/users",
+      purge   => $elasticsearch::purge_configdir,
+      force   => $elasticsearch::purge_configdir,
+      require => File[$configdir],
+      before  => Elasticsearch::Service[$name],
     }
 
-
     # Do _not_ copy in instance directories. This avoids a) recursing
     # indefinitely by copying our own instance directory and b) copying in any
     # other potential instance directories.