-
Notifications
You must be signed in to change notification settings - Fork 181
Modernize Node.js + Filebeat ingestion guide for Elastic Stack 9.0 #3422
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
Conversation
🔍 Preview links for changed docs |
|
@pierrehilbert Hi Pierre, this page contains information that is outdated in version 9.0, and we need to update it. Could you please refer me to someone from your team who can review it? |
|
@belimawr could you please review here? |
belimawr
left a comment
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.
Folks, I ended up reviewing more than just the changes. I took the time to actually follow the whole tutorial, so my comments are a mix of suggestions on the changes made and the overall structure of this tutorial.
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Show resolved
Hide resolved
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Show resolved
Hide resolved
|
@belimawr Thank you for your comments! 🫶 I've updated the file, please take a look and let me know if it looks fine now. |
...t/ingesting-data-from-applications/ingest-logs-from-nodejs-web-application-using-filebeat.md
Outdated
Show resolved
Hide resolved
colleenmcginnis
left a comment
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.
This looks great @alexandra5000! I left some minor suggestions, but none of them are blocking.
This PR updates the “Ingest logs from a Node.js web application using Filebeat” tutorial to align with Elastic Stack 9.0 GA .
Summary of changes:
Replaced the deprecated
loginput withfilestream.Updated instructions for locating the Cloud ID using the Elastic Cloud console (replacing the outdated Kibana path).
Removed obsolete setup steps for creating deployments and connected securely sections; replaced with current "Before you begin" guidance and links to centralized Cloud setup docs.
Added
::::{stepper}blocks for procedural steps.Closes #1968