Skip to content

Commit 6b46434

Browse files
author
Branden Timm
committed
Changes for version 0.0.2
1 parent db1f1a8 commit 6b46434

6 files changed

Lines changed: 19 additions & 16 deletions

File tree

CHANGELOG.rst

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
logstash-forwarder formula
22
================
33

4+
0.0.2 (2014-09-02)
5+
- Added managed init files for Debian and RedHat
6+
- Support turning off logstash-forwarder log messages using
7+
log_to_syslog pillar variable
8+
49
0.0.1 (2014-08-19)
510
- Initial commit

README.rst

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,15 @@ Available states
2121

2222
Install the ``logstash-forwarder`` package, set up configuration file,
2323
optionally set up the lumberjack certificate, and enable the service. This
24-
formula currently supports Debian and RedHat based distributions, however the
25-
service init file for RedHat distributions is currently broken, see
26-
<https://github.com/elasticsearch/logstash-forwarder/pull/196>. Due to this,
27-
the formula provides it's own init script until such a time when the above
28-
issue is resolved.
29-
24+
formula currently supports Debian and RedHat based distributions. Although the
25+
packages for Debian and RedHat provide service init files, we manage it within
26+
this formula for two reasons:
27+
28+
* The RedHat service init file is currently broken, see <https://github.com/elasticsearch/logstash-forwarder/pull/196>
29+
* By default, init files for both distro families enables 'log-to-syslog' when invoking
30+
logstash-forwarder, which can pollute your syslog with unnecessary noise. By default
31+
this feature will remain on, but it can be turned off by setting logstash_forwarder:log_to_syslog
32+
to false in your pillar data.
3033

3134
Usage
3235
=====

VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
0.0.1
1+
0.0.2

logstash_forwarder/files/logstash-forwarder.deb.init

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,13 @@
11
{%- from 'logstash_forwarder/map.jinja' import logstash_forwarder with context %}
22
#! /bin/sh
33
### BEGIN INIT INFO
4-
# Provides: skeleton
4+
# Provides: logstash-forwarder
55
# Required-Start: $remote_fs $syslog
66
# Required-Stop: $remote_fs $syslog
77
# Default-Start: 2 3 4 5
88
# Default-Stop: 0 1 6
9-
# Short-Description: Example initscript
10-
# Description: This file should be used to construct scripts to be
11-
# placed in /etc/init.d.
129
### END INIT INFO
1310

14-
# Author: Jordan Sissel <jordan.sissel@dreamhost.com>
15-
1611
# PATH should only include /usr/* if it runs after the mountnfs.sh script
1712
PATH=/sbin:/usr/sbin:/bin:/usr/bin
1813
DESC="log shipper"

logstash_forwarder/init.sls

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
1-
{%- from 'logstash_forwarder/map.jinja' import logstash_forwarder with context %}
2-
31
# Do nothing unless the target is RedHat or Debian based
4-
52
{%- if grains['os_family'] == 'RedHat' or grains['os_family'] == 'Debian' %}
3+
{%- from 'logstash_forwarder/map.jinja' import logstash_forwarder with context %}
4+
65
include:
76
- .repo
87

pillar.example

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ logstash_forwarder:
1818
fields:
1919
type: nginx-error
2020
cert_path: /etc/ssl/certs/logstash-forwarder.crt
21+
log_to_syslog: true
2122
cert_contents: |
2223
-----BEGIN CERTIFICATE-----
2324
MIIDBzCCAe+gAwIBAgIJAImyMODCMdTFMA0GCSqGSIb3DQEBBQUAMBoxGDAWBgNV

0 commit comments

Comments
 (0)