Remove unused audit log file rotation settings from docs#8771
Open
Remove unused audit log file rotation settings from docs#8771
Conversation
From Mattermost server v11.4, the following ExperimentalAuditSettings file rotation settings were removed as unused (mattermost/mattermost#35170): - FileMaxSizeMB - FileMaxAgeDays - FileMaxBackups - FileCompress File rotation for audit logs is now configured via AdvancedLoggingJSON. Updated files: - environment-configuration-settings.rst: Remove 4 rotation setting sections - experimental-configuration-settings.rst: Remove 4 rotation setting sections; update description to direct admins to AdvancedLoggingJSON for file rotation - logging.rst: Remove rotation settings from Cloud defaults list - telemetry.rst: Remove rotation settings from telemetry data list Co-authored-by: Combs7th <Combs7th@users.noreply.github.com>
Combs7th
added
1: Dev Review
Contributor
Author
|
@hanzei - Tagging you for dev review. Do the changes above look accurate? If so, I'll tag Eric for editor review. |
Contributor
|
Newest code from mattermost has been published to preview environment for Git SHA f141ce0 |
Member
|
@Combs7th This is also scheduled for v11.6 (April release). |
hanzei
requested changes
Mar 5, 2026
| :start-after: :nosearch: | ||
|
|
||
| When audit logging is enabled in a self-hosted instance, you can specify size, backup interval, compression, maximium age to manage file rotation, and timestamps for audit logging, as defined below. You can specify these settings independently for audit events and AD/LDAP events. | ||
| When audit logging is enabled in a self-hosted instance, you can specify a file name and queue size for audit logging, as defined below. To configure file rotation for audit logs, use the :ref:`AdvancedLoggingJSON <administration-guide/configure/environment-configuration-settings:output audit logs to multiple targets>` setting. You can specify these settings independently for audit events and AD/LDAP events. |
Contributor
There was a problem hiding this comment.
Are you sure that you can set the queue side without advanced logging?
Contributor
There was a problem hiding this comment.
We don't do telemetry any longer. Can we just drop the file?
Contributor
Author
|
@claude - Please revise the docs update based on the PR evidence and developer feedback. Before drafting:
What to do:
|
Contributor
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove four unused audit log file rotation settings from documentation to reflect Mattermost server v11.4+ behavior.
Engineering PR mattermost/mattermost#35170 removes these settings from
ExperimentalAuditSettings:FileMaxSizeMBFileMaxAgeDaysFileMaxBackupsFileCompressFile rotation for audit logs is now configured via
AdvancedLoggingJSON.Closes #8769
Generated with Claude Code