-
Notifications
You must be signed in to change notification settings - Fork 329
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
Reload config involves a loss of logs #791
Comments
It was reverted because it did not handle plugin params as expected. Things might have changed with latest fluentd versions though. I would consider making this configurable and let users experiment with it until we can get back to this and retest it thoroughly. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions! |
@sebastiendejou or anyone else still have any interest in trying gracefulReload? If yes we can implement a flag for that. |
Is your feature request related to a problem? Please describe.
When an update of the fluentd's configuration happens from another namespace, the fluentd worker process is restarted. (this way : http://127.0.0.1:24444/api/config.reload) and the problem happens : Logs are lost during reload for the first namespace.
Does logging-operator respect the multitenancy agreement ?
Describe alternatives you've considered
No one.
Describe alternatives you've considered
In a previous issue, "/api/config.gracefulReload" (#454) has been suggested. This issue has been merged, but reverted after that.
Is there a mechanism that could gracefully reload the fluentd ?
Regards
The text was updated successfully, but these errors were encountered: