-
Notifications
You must be signed in to change notification settings - Fork 125
Update installation-usage.md to clarify audit log location - OCPBUGS-59523 #2994
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
Conversation
|
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: BhargaviGudi The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Welcome @BhargaviGudi! |
|
Hi @BhargaviGudi. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
75d721d to
1ea4436
Compare
|
@BhargaviGudi thank you for the PR! Please sign the CLA for this one. |
|
/check-cla |
1ea4436 to
8890e3f
Compare
|
I have signed CLA document as individual contributor and still observing failure in this PR as |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2994 +/- ##
===========================================
- Coverage 45.50% 26.56% -18.95%
===========================================
Files 79 125 +46
Lines 7782 20921 +13139
===========================================
+ Hits 3541 5557 +2016
- Misses 4099 15080 +10981
- Partials 142 284 +142 🚀 New features to boost your workflow:
|
Have you checked this comment?
|
8890e3f to
5ee08b5
Compare
|
/check-cla |
1 similar comment
|
/check-cla |
…ation OCPBUGS-59523 - Update installation-usage.md to clarify audit log location OCPBUGS-59523 - update code block language identifiers from bash to shell OCPBUGS-59523 - replaced oc with kubectl OCPBUGS-59523 - updated Signed-off-by: Bhargavi Gudi <[email protected]>
5ee08b5 to
783778f
Compare
|
/check-cla |
|
/retest |
|
@BhargaviGudi: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/check-cla |
|
Closing this PR as I have created new PR to resolve EasyCLA authentication issue. |
What type of PR is this?
/kind bug
/documentation
What this PR does / why we need it:
This PR updates the
installation-usage.mddocumentation to clarify where the audit log file generated by the JsonEnricher is stored. Currently, the documentation does not make it clear that the log file is written to the node's file system (not inside the pod), which can lead to confusion for users monitoring audit logs.Which issue(s) this PR fixes:
Fixes documentation inconsistency observed when enabling JsonEnricher in the Security Profiles Operator.
Fixes bug: https://issues.redhat.com/browse/OCPBUGS-59523
Does this PR have test?
N/A — documentation-only change.
Special notes for your reviewer:
Does this PR introduce a user-facing change?
NONE