Skip to content

(new page) Logstash Diag #1726

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

Merged
merged 7 commits into from
Jun 24, 2025
Merged

(new page) Logstash Diag #1726

merged 7 commits into from
Jun 24, 2025

Conversation

stefnestor
Copy link
Contributor

👋 howdy, team!

Same as ES Diag & KB diag, this adds a diagnostic page for the remaining Logstash product to be used with the https://github.com/elastic/support-diagnostics/ .

This is my first bigger PR in this new repo, so kindly confirm YAML and link validity in particular 🙏

@stefnestor stefnestor requested a review from a team as a code owner June 12, 2025 20:44
@stefnestor stefnestor added the enhancement New feature or request label Jun 12, 2025
Copy link

github-actions bot commented Jun 12, 2025

🔍 Preview links for changed docs:

🔔 The preview site may take up to 3 minutes to finish building. These links will become live once it completes.

@colleenmcginnis
Copy link
Contributor

Because you're introducing a new page, you need to specify where it should go in the left hand nav. You'll need to add this file (using - file: ingest/logstash/diagnostic.md) to troubleshoot/toc.yml... probably somewhere around here.

@stefnestor stefnestor requested a review from a team as a code owner June 13, 2025 20:29
Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

First pass. Good work!

stefnestor and others added 2 commits June 24, 2025 08:08
Co-authored-by: Fabrizio Ferri-Benedetti <[email protected]>
Co-authored-by: Fabrizio Ferri-Benedetti <[email protected]>
@stefnestor
Copy link
Contributor Author

Cheers, @theletterf ! Ready for you again 🙏

Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Ready to go! Weeee!

@stefnestor stefnestor merged commit ee0b964 into elastic:main Jun 24, 2025
7 checks passed
stefnestor added a commit that referenced this pull request Jun 24, 2025
👋 @theletterf, removing the tabs from just created [LS Diag](https://www.elastic.co/docs/troubleshoot/ingest/logstash/diagnostic) page from #1726 because it sub-content'd wrong but afterwards if it's okay I decided I like not tabs for searchability 🙂
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants