From 890ef36de29e47c380b3bc06e354ce3346af1603 Mon Sep 17 00:00:00 2001 From: Tudor Golubenco Date: Thu, 6 Apr 2017 00:21:35 +0200 Subject: [PATCH] Don't stop Filebeat when modules + logstash are used together To simplify the migration from Ingest Node to Logstash, we replace the hard error ("Filebeat modules require an Elasticsearch output defined") with a warning. --- filebeat/beater/filebeat.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/filebeat/beater/filebeat.go b/filebeat/beater/filebeat.go index 8b4e0d6e42d..de1cda6e3e5 100644 --- a/filebeat/beater/filebeat.go +++ b/filebeat/beater/filebeat.go @@ -75,7 +75,11 @@ func New(b *beat.Beat, rawConfig *common.Config) (beat.Beater, error) { func (fb *Filebeat) modulesSetup(b *beat.Beat) error { esConfig := b.Config.Output["elasticsearch"] if esConfig == nil || !esConfig.Enabled() { - return fmt.Errorf("Filebeat modules configured but the Elasticsearch output is not configured/enabled") + logp.Warn("Filebeat is unable to load the Ingest Node pipelines for the configured" + + " modules because the Elasticsearch output is not configured/enabled. If you have" + + " already loaded the Ingest Node pipelines or are using Logstash pipelines, you" + + " can ignore this warning.") + return nil } esClient, err := elasticsearch.NewConnectedClient(esConfig) if err != nil {