Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script: unthrottle ingest pipeline definition. #70272

Conversation

stu-elastic
Copy link
Contributor

@stu-elastic stu-elastic commented Mar 10, 2021

Scripts in ingest pipelines may be throttled on startup
or when beats starts up. This causes pipelines to be
stuck in a broken state.

Pipelines with many mustache templates are particularly impacted.

Creates a new context, ingest_template, with an unlimited
compilation rate limit.

The template context is used in many places so it cannot be
exempted from rate limits.

Fixes: #64595
Backport: 9370b1c

…lastic#69987)

Remove the compilation rate limit for ingest templates.

Creates a new context, `ingest_template`, with an unlimited
compilation rate limit.

The `template` context is used in many places so it cannot be
exempted from rate limits.

Fixes: elastic#64595
Backport: 9370b1c
@stu-elastic stu-elastic added >bug :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache backport v7.12.0 labels Mar 10, 2021
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Mar 10, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@stu-elastic stu-elastic changed the title Script: no compile rate limit for ingest templates (#69841) (#69987) Script: unthrottle ingest pipeline definition. Mar 10, 2021
@stu-elastic stu-elastic merged commit 58db0a3 into elastic:7.12 Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport >bug :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache Team:Core/Infra Meta label for core/infra team v7.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants