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

Add more metrics for AppArmor Profile #2686

Merged
merged 5 commits into from
Jan 22, 2025
Merged

Add more metrics for AppArmor Profile #2686

merged 5 commits into from
Jan 22, 2025

Conversation

ccojocar
Copy link
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

  • Implement the apparmor_profile_audit_total metric
  • Add the profile name to apparmor_profile_error_total metric
  • Use a metric label name instead of reason inside of apparmor_profile_audit_total
  • Add a metric for apparmor profile denail

Which issue(s) this PR fixes:

Fixes #2665

Does this PR have test?

N/A

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Add more metrics for AppArmor profile.

Change-Id: I53ac7c317d2a86d3cb086b189f707e492f3aff0b
Signed-off-by: Cosmin Cojocar <[email protected]>
Change-Id: I48d3f2c159da7e30e6745ba3c29a0565d5788818
Signed-off-by: Cosmin Cojocar <[email protected]>
Change-Id: I0706d004ac42d9df6c21a87b1c032c280ddae8a7
Signed-off-by: Cosmin Cojocar <[email protected]>
…audit_total

Change-Id: I449d6c7a3b1a5e1c46b70bf57c84545d73124cbe
Signed-off-by: Cosmin Cojocar <[email protected]>
Add a metric which counts the total number of operations denined for a
apparmor profile.

Change-Id: I660500bf929cdd05e08a15a01681d5444739fc5b
Signed-off-by: Cosmin Cojocar <[email protected]>
@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/feature Categorizes issue or PR as related to a new feature. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jan 22, 2025
@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jan 22, 2025
@ccojocar ccojocar requested review from saschagrunert and removed request for jhrozek and Vincent056 January 22, 2025 08:39
@ccojocar
Copy link
Contributor Author

@saschagrunert Please could you have a quick look? Thanks a lot!

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 22, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ccojocar, saschagrunert

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [ccojocar,saschagrunert]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot merged commit af2cee0 into main Jan 22, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

More Metrics for AppArmor Profile
3 participants