diff options
| author | AbhishekKr <abhikumar163@gmail.com> | 2016-06-28 02:22:28 +0530 |
|---|---|---|
| committer | AbhishekKr <abhikumar163@gmail.com> | 2016-06-28 02:22:38 +0530 |
| commit | ed070cc97df6cf244afd4f4f4d0b49fee973dc8f (patch) | |
| tree | 0736ed48ce247d42afe97743cf94bd7e0fd4a18d | |
| parent | 945f23be7a43d90ae7ed402d05363b3ff0c11bff (diff) | |
| download | chef-ed070cc97df6cf244afd4f4f4d0b49fee973dc8f.tar.gz | |
service - upstart restart does not uses latest job config, need start/stop
latest upstart job config is not used by /sbin/restart but retains oiginal
http://upstart.ubuntu.com/cookbook/#restart
| -rw-r--r-- | lib/chef/provider/service/upstart.rb | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/chef/provider/service/upstart.rb b/lib/chef/provider/service/upstart.rb index 3ac5ff51da..268324b8c7 100644 --- a/lib/chef/provider/service/upstart.rb +++ b/lib/chef/provider/service/upstart.rb @@ -186,9 +186,13 @@ class Chef super # Upstart always provides restart functionality so we don't need to mimic it with stop/sleep/start. # Older versions of upstart would fail on restart if the service was currently stopped, check for that. LP:430883 + # But for safe working of latest upstart job config being loaded, 'restart' can't be used as per link + # http://upstart.ubuntu.com/cookbook/#restart (it doesn't uses latest jon config from disk but retains old) else if @current_resource.running - shell_out_with_systems_locale!("/sbin/restart #{@job}") + stop_command + sleep 1 + start_command else start_service end |