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

ECS Templates: stack alignment #1062

Merged
merged 5 commits into from
Jan 13, 2022

Conversation

yaauie
Copy link
Contributor

@yaauie yaauie commented Dec 22, 2021

Resolution of #1060.

With ECS 8.0, ECS is becoming Stack-aligned, and no longer generates template artifacts for other Elasticsearch majors. This is causing some paths to change in the upstream project from which we vendor our templates.

They continue to generate a monolith "legacy" template, for use with the Elasticsearch "legacy" template API (a.k.a., v1), but this plugin automatically uses the v2 template endpoint when persisting templates to Elasticsearch 8.

Historically, this plugin has provided ECS v1 templates for (unreleased) Elasticsearch 8 by transforming them into the v2 format prior to vendoring. This changeset builds upon that work, renaming the relevant methods to be more descriptive and separating the concerns of transforming the template from those of defining which versions top vendor.

It also includes a number of build fixes.

Rakefile Outdated Show resolved Hide resolved
Rakefile Outdated Show resolved Hide resolved
@yaauie yaauie marked this pull request as ready for review January 12, 2022 23:36
@yaauie yaauie requested a review from andsel January 12, 2022 23:38
@kares kares linked an issue Jan 13, 2022 that may be closed by this pull request
Copy link
Contributor

@andsel andsel left a comment

Choose a reason for hiding this comment

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

Nice,
LGTM!

@yaauie yaauie merged commit 9b16071 into logstash-plugins:main Jan 13, 2022
@yaauie yaauie deleted the ecs-stack-alignment branch January 13, 2022 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ILM integration specs failing against 8.1-SNAPSHOT
3 participants