Skip to content
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

NAS-130498 / 24.10 / Omit SMB audit results from audit.query output by default #14163

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

anodos325
Copy link
Contributor

This is an optimization to improve memory consumption by default queries used by the webui. Initially we included SMB audit results by default with the expectation that user would specify the exact audited service they wanted to query. Since SMB audit logs can span upwards to hundreds of thousands of entries in some extreme cases we need to ensure that UI / API consumers use optimized SQL queryfilters with appropriate pagination.

@anodos325 anodos325 added the WIP label Aug 7, 2024
This is an optimization to improve memory consumption by default
queries used by the webui. Initially we included SMB audit results
by default with the expectation that user would specify the exact
audited service they wanted to query. Since SMB audit logs can
span upwards to hundreds of thousands of entries in some extreme
cases we need to ensure that UI / API consumers use optimized
SQL queryfilters with appropriate pagination.
@anodos325 anodos325 force-pushed the make-audit-form-usable branch from 76e3f30 to 1f431c9 Compare August 8, 2024 11:42
@anodos325 anodos325 added jira and removed WIP labels Aug 8, 2024
@anodos325 anodos325 requested review from a team and mgrimesix August 8, 2024 12:21
@bugclerk bugclerk changed the title Omit SMB audit results from audit.query output by default NAS-130498 / 24.10 / Omit SMB audit results from audit.query output by default Aug 8, 2024
@bugclerk
Copy link
Contributor

bugclerk commented Aug 8, 2024

@anodos325 anodos325 merged commit 0de447b into master Aug 8, 2024
2 of 3 checks passed
@anodos325 anodos325 deleted the make-audit-form-usable branch August 8, 2024 15:03
@bugclerk
Copy link
Contributor

bugclerk commented Aug 8, 2024

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Aug 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants