-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[8.0] Remove @kbn/legacy-logging #50660
Comments
Pinging @elastic/kibana-platform (Team:Platform) |
Regarding log rotation and retention: We are in the planning phase for revamping Audit Logging in #52125, and we will need log rotation in place to support that. Log rotation has been added in #12259, but it only allows logs to be retained based on number of files. Industry standards generally require that organizations retain an audit trail based on a time period (e.g, 30 days, 90 days, etc.) This is currently supported in Elasticsearch, it would be great if Kibana supported this as well. |
Pinging @elastic/kibana-core (Team:Core) |
Logging service in the legacy platform based on our fork of
@hapi/good
and internal logic https://github.com/elastic/kibana/blob/8e9a8a84dccfa7965ce8a22362885e6cdef8b51f/src/legacy/server/logging/log_interceptor.jsPreparation steps:
Removing legacy logging:
@kbn/legacy-logging
transformLogging
logic frompackages/kbn-config/src/legacy/legacy_object_to_config_adapter.ts
@hapi/good
/@hapi/good-squeeze
/@hapi/podium
from kibana dependencies [core.logging] Remove dependency on @hapi/good, @hapi/good-squeeze, @hapi/podium #82240src/core/server/legacy/logging/appenders
logging.root.appenders
to have adefault
(this can now be optional and populated with the default value if nothing is explicitly configured) https://github.com/elastic/kibana/blob/master/src/core/server/logging/logging_config.ts#L79-L81src/legacy/server/logging
src/cli/serve/serve.js
to remove references to remove deprecated flags (quiet
,log-file
), and remove references to deprecated legacy config (used insideverbose
andsilent
)The text was updated successfully, but these errors were encountered: