Yuvipanda has submitted this change and it was merged.
Change subject: dynamicproxy: Don't try to restart redis when config changes
......................................................................
dynamicproxy: Don't try to restart redis when config changes
Is a stateful service, should be done manually
Change-Id: I6a0e60bf3ddeb2a7dea67b040f134080584d98f9
---
M modules/dynamicproxy/manifests/init.pp
1 file changed, 1 insertion(+), 6 deletions(-)
Approvals:
Yuvipanda: Looks good to me, approved
jenkins-bot: Verified
diff --git a/modules/dynamicproxy/manifests/init.pp
b/modules/dynamicproxy/manifests/init.pp
index 2731b3f..da74ff0 100644
--- a/modules/dynamicproxy/manifests/init.pp
+++ b/modules/dynamicproxy/manifests/init.pp
@@ -49,13 +49,8 @@
maxmemory => $redis_maxmemory,
slaveof => $slaveof,
dir => '/var/lib/redis',
- }
+ },
}
-
- # The redis module intentionally does not restart the redis
- # service if the configuration changes, so we have to do this
- # explicitly here.
- File['/etc/redis/redis.conf'] ~> Service['redis-server']
class { 'nginx':
variant => 'extras',
--
To view, visit https://gerrit.wikimedia.org/r/253364
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6a0e60bf3ddeb2a7dea67b040f134080584d98f9
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Yuvipanda <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
