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

ROX-23900: Add prometheus alert for SELinux violations #230

Merged
merged 1 commit into from
May 22, 2024

Conversation

mclasmeier
Copy link
Contributor

@mclasmeier mclasmeier commented Apr 26, 2024

This is part of https://issues.redhat.com/browse/ROX-22148.

  1. SOP is currently under review.
  2. The ACSCS release which prevents the Scanner violations is scheduled for today.
    Could we do the review now and then merge when (1) and (2) are completed?

@mclasmeier mclasmeier requested a review from a team as a code owner April 26, 2024 12:08
@mclasmeier mclasmeier requested a review from stehessel April 26, 2024 12:09
@mclasmeier mclasmeier force-pushed the mc/violation-alerts branch from b16207f to 7a0f563 Compare May 10, 2024 10:04
@mclasmeier mclasmeier force-pushed the mc/violation-alerts branch from 86f11ff to 4d20524 Compare May 13, 2024 13:07
@mclasmeier mclasmeier requested a review from stehessel May 13, 2024 13:10
Copy link
Contributor

@stehessel stehessel left a comment

Choose a reason for hiding this comment

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

LGTM assuming we have a clean base line.

@mclasmeier
Copy link
Contributor Author

Awesome, thanks.

@mclasmeier mclasmeier changed the title Add prometheus alert for SELinux violations ROX-23900: Add prometheus alert for SELinux violations May 15, 2024
@mclasmeier
Copy link
Contributor Author

image Baseline is now zero.

@mclasmeier mclasmeier merged commit 7a9693d into master May 22, 2024
3 checks passed
@mclasmeier mclasmeier deleted the mc/violation-alerts branch May 22, 2024 10:39
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.

2 participants