Faidon has submitted this change and it was merged.

Change subject: Ceph: fix start/stop/restart for radosgw Service
......................................................................


Ceph: fix start/stop/restart for radosgw Service

Puppet calls /sbin/start 'radosgw id=radosgw', which of course fails.
Puppet also doesn't seem to support actions to services. Change to
custom start/stop/restart...

Change-Id: Ie91d93f93febc6e4b45479481358a1eb23dc60c1
---
M modules/ceph/manifests/radosgw.pp
1 file changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Faidon: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ceph/manifests/radosgw.pp 
b/modules/ceph/manifests/radosgw.pp
index b4ffa99..e762b30 100644
--- a/modules/ceph/manifests/radosgw.pp
+++ b/modules/ceph/manifests/radosgw.pp
@@ -23,9 +23,11 @@
 
     service { 'radosgw id=radosgw':
         ensure     => 'running',
+        status     => '/sbin/status radosgw id=radosgw',
+        start      => '/sbin/start  radosgw id=radosgw',
+        stop       => '/sbin/stop   radosgw id=radosgw',
+        restart    => '/sbin/start  radosgw id=radosgw',
         hasrestart => true,
-        # upstart status is broken with id= ...
-        status     => '/usr/bin/pgrep radosgw',
         provider   => 'upstart',
         require    => File['/var/lib/ceph/radosgw/ceph-radosgw/done'],
     }

-- 
To view, visit https://gerrit.wikimedia.org/r/61160
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie91d93f93febc6e4b45479481358a1eb23dc60c1
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Faidon <[email protected]>
Gerrit-Reviewer: Faidon <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to