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

Adding a telemetry log for ASB #888

Open
wants to merge 21 commits into
base: dev
Choose a base branch
from

Conversation

MariusNi
Copy link
Contributor

@MariusNi MariusNi commented Feb 27, 2025

Description

Adding a simple telemetry log for the Azure security baseline ( ASB) for Linux that logs for each individual ASB rule audit and remediation the following:

  • The friendly name of the local OS target
  • The MIM component name which is the name of the baseline (e.g. "SecurityBaseline", "Compliamce", etc)
  • The MIM object name which is the internal rule name (e.g. "auditEnsureBind9NotInstalled")
  • The result, in format "description (code)", such as: "Success (0)", "Invalid argument (22)", etc.
  • The elapsed time in microseconds for this rule audit or remediation operation.

There is more work that needs to be done here. Including adding the 'switch' for Telemetry Levels. Possibly adding more fields and more types of events, also possibly to switch from writing to this log to sending each line directly to the Cloud (TBD). For now this is the log that the cloud work can continue with in dev branch.

This can be modeled from and applied to Compliance as well.

Checklist

  • I have read the contribution guidelines.
  • All unit tests are passing.
  • I have merged the latest dev branch prior to this PR submission.
  • I ran pre-commit on my changes prior to this PR submission.
  • I submitted this PR against the dev branch.

@MariusNi MariusNi requested a review from a team as a code owner February 27, 2025 19:00
Copy link

github-actions bot commented Feb 27, 2025

Test Results

 44 files   44 suites   37m 10s ⏱️
  4 tests   4 ✅  0 💤 0 ❌
176 runs  154 ✅ 22 💤 0 ❌

Results for commit 7414d5e.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@AhmedBM AhmedBM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants