-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
x-pack/filebeat/input/{websocket=>streaming}: rename package #40421
Conversation
983417c
to
7c7c924
Compare
7c7c924
to
b902459
Compare
Pinging @elastic/security-service-integrations (Team:Security-Service Integrations) |
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the inclusion point of this file need changed to keep the input list sorted alphabetically?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -156,4 +156,4 @@ include::inputs/input-udp.asciidoc[] | |||
|
|||
include::inputs/input-unix.asciidoc[] | |||
|
|||
include::../../x-pack/filebeat/docs/inputs/input-websocket.asciidoc[] | |||
include::../../x-pack/filebeat/docs/inputs/input-streaming.asciidoc[] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this line needs moved up to ~L150.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I miss mechanically generated documentation rendering.
Proposed commit message
See title.
This has been tested by injecting an
agentbeat
compiled with this version into a running 8.15 elastic-package stack and then running the proofpoint_on_demand integration, an integration that depends on the websocket input.Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Disruptive User Impact
Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs