Skip to content

[backport 7.x] Fix/log4j routing to avoid create spurious file (#12965)#12999

Merged
andsel merged 1 commit intoelastic:7.xfrom
andsel:backport_12965_7.x
Jun 18, 2021
Merged

[backport 7.x] Fix/log4j routing to avoid create spurious file (#12965)#12999
andsel merged 1 commit intoelastic:7.xfrom
andsel:backport_12965_7.x

Conversation

@andsel
Copy link
Contributor

@andsel andsel commented Jun 17, 2021

Clean backport of #12965 to branch 7.x

Avoid the creation of log4j routing appender for log events without the pipeline.id fishtag.
In this way no spurious log file named "pipeline_${ctx:pipeline.id}.log" and logs are not duplicated with main Logstash log file.

(cherry picked from commit 1d6a3e4)

Avoid the creation of log4j routing appender for log events without the `pipeline.id` fishtag.
In this way no spurious log file named "pipeline_${ctx:pipeline.id}.log" and logs are not duplicated with main Logstash log file.

(cherry picked from commit 1d6a3e4)
Copy link
Member

@robbavey robbavey left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants