Skip to content
This repository has been archived by the owner on Dec 27, 2022. It is now read-only.

Commit

Permalink
Merge pull request #346 from lusis/fix_service_restart
Browse files Browse the repository at this point in the history
Fix service restart
  • Loading branch information
paulczar committed Sep 6, 2014
2 parents 1092113 + 1c83271 commit 88312e3
Show file tree
Hide file tree
Showing 12 changed files with 301 additions and 294 deletions.
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,4 +5,5 @@ vendor/
.kitchen/
.kitchen.local.yml
.bundle/
bin/
bin/
Gemfile.lock
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ This file is used to list changes made in each version of chef-logstash.


## 0.9.2:
* fix PAX headers on community site
* update to fix PAX header issue on community site

## 0.9.1:

Expand Down
248 changes: 0 additions & 248 deletions Gemfile.lock

This file was deleted.

3 changes: 2 additions & 1 deletion providers/config.rb
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ def load_current_resource
@service_name = new_resource.service_name || @instance
end

use_inline_resources

action :create do
conf = conf_vars
# Chef::Log.info("config vars: #{conf.inspect}")
Expand All @@ -40,7 +42,6 @@ def load_current_resource
group conf[:group]
mode conf[:mode]
variables conf[:variables]
notifies :restart, "logstash_service[#{conf[:service_name]}]"
action :create
end
new_resource.updated_by_last_action(tp.updated_by_last_action?)
Expand Down
Loading

1 comment on commit 88312e3

@paulczar
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it was auto-starting so I removed the test ... but that might not have been the right thing to do.

Please sign in to comment.