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

Doc: Add shared attribute for messaging ecs default info #13083

Merged
merged 5 commits into from
Oct 1, 2021

Conversation

karenzone
Copy link
Contributor

@karenzone karenzone commented Jul 19, 2021

Release notes

[rn:skip]

What does this PR do?

Adds and links shared attribute file for accurately stating ecs default status.

Design intent/usage

We can add/update text related to ecs defaults (or other info we want to write once and use multiple places). Then, we can reference it by calling the attribute. In this case {ecs-default}.

Important: This work requires a corresponding change for the Versioned Plugin Reference.

To do (in this order):

  • Merge and backport this PR.
  • Add shared attribute include file to the Versioned Plugin Reference guide, and test.
  • Implement the attribute in plugin docs. We can't start implementing the attribute in plugin docs until the framework is fully implemented. Otherwise, we will break the doc build.

@karenzone karenzone requested a review from yaauie August 11, 2021 18:12
@karenzone
Copy link
Contributor Author

Hi @yaauie I would appreciate your comments on the usefulness and content of this PR.
Related: elastic/logstash-docs#1128

@karenzone karenzone force-pushed the shared-ecs-attr branch 2 times, most recently from 0129521 to 13b71de Compare September 2, 2021 00:41
karenzone and others added 2 commits September 27, 2021 17:21
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
@karenzone karenzone requested a review from yaauie September 27, 2021 23:41
Copy link
Member

@yaauie yaauie left a comment

Choose a reason for hiding this comment

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

👍🏼 LGTM

@karenzone karenzone merged commit 2d8abc4 into elastic:master Oct 1, 2021
@karenzone karenzone deleted the shared-ecs-attr branch October 1, 2021 17:15
karenzone added a commit to karenzone/logstash that referenced this pull request Oct 1, 2021
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
karenzone added a commit to karenzone/logstash that referenced this pull request Oct 1, 2021
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
karenzone added a commit to karenzone/logstash that referenced this pull request Oct 1, 2021
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
karenzone added a commit that referenced this pull request Oct 1, 2021
Backports #13083 to 7.x
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
karenzone added a commit that referenced this pull request Oct 1, 2021
Backports #13083 to 7.15
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
karenzone added a commit that referenced this pull request Oct 1, 2021
Backports #13083 to 7.14
Co-authored-by: Ry Biesemeyer <yaauie@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants