-
Notifications
You must be signed in to change notification settings - Fork 387
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
[Tomcat] Deprecate Tomcat package #6091
[Tomcat] Deprecate Tomcat package #6091
Conversation
🌐 Coverage report
|
|
||
This integration is for [Tomcat device's](https://tomcat.apache.org/tomcat-10.0-doc/logging.html) logs. It includes the following | ||
datasets for receiving logs over syslog or read from a file: | ||
|
||
- `log` dataset: supports Apache Tomcat logs. | ||
|
||
Note: | ||
- To collect Apache Tomcat Logs and Metrics please use ``Apache Tomcat`` integration since ``Tomcat NetWitness Logs`` integration will be deprecated soon. |
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.
Once we have the Apache Tomcat integration published, we can provide the link to that integration here
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, but for now we have kept just name of the integration.
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!
Looks good, but lets merge this PR once the Apache Tomcat Pr's are merged |
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.
Looks good, but lets merge this PR once the Apache Tomcat PR's are merged.
@@ -1,10 +1,13 @@ | |||
# Tomcat integration | |||
# Tomcat NetWitness Logs integration (To be deprecated soon) |
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.
Should we provide a link to the new integration to be used instead here?
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.
@lalit-satapathy : Please refer to this comment
Package tomcat - 1.8.2 containing this change is available at https://epr.elastic.co/search?package=tomcat |
What does this PR do?
Tomcat
integration toTomcat NetWitness Logs
.Checklist
changelog.yml
file.How to test this PR locally
Related issues