Skip to content

NAS-135620 / 25.10 / Simplify middleware audit startup #16371

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

anodos325
Copy link
Contributor

Delay certain expensive parts of middlewared auditing startup (an initial scan for file changes and removing unnecessary refreservations on old audit datasets) until after the system READY event, and only perform it on first boot after upgrade or fresh install.

@anodos325 anodos325 added the WIP label Apr 29, 2025
@anodos325 anodos325 force-pushed the simplify-audit-startup branch 2 times, most recently from 4378919 to 9671e0e Compare April 29, 2025 14:50
@anodos325 anodos325 removed the WIP label Apr 29, 2025
Delay certain expensive parts of middlewared auditing startup (an
initial scan for file changes and removing unnecessary refreservations
on old audit datasets) until after the system READY event, and only
perform it on first boot after upgrade or fresh install.

This commit also removes the AuditSetupAlertClass. It could only be
called during the first boot for a boot environment and would never
be dismissed. Removal decreases complexity of operations during
system ready events on first boot.
@anodos325 anodos325 force-pushed the simplify-audit-startup branch from 6e32007 to 0794429 Compare April 29, 2025 17:59
@anodos325 anodos325 requested a review from yocalebo April 29, 2025 17:59
@bugclerk bugclerk changed the title Simplify middleware audit startup NAS-135620 / 25.10 / Simplify middleware audit startup Apr 29, 2025
@bugclerk
Copy link
Contributor

@anodos325 anodos325 requested a review from yocalebo April 29, 2025 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants