Skip to content

Conversation

@ngopalak-redhat
Copy link
Contributor

What type of PR is this?

/kind documentation

What this PR does / why we need it:

There are multiple steps to configure json log enricher. The users need to be guided to achieve it.

Which issue(s) this PR fixes:

Does this PR have test?

NA

Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

@k8s-ci-robot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@k8s-ci-robot k8s-ci-robot added release-note-none Denotes a PR that doesn't merit a release note. kind/documentation Categorizes issue or PR as related to documentation. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Sep 10, 2025
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 10, 2025
@ngopalak-redhat ngopalak-redhat marked this pull request as ready for review September 10, 2025 06:46
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 10, 2025
@ngopalak-redhat
Copy link
Contributor Author

/assign @saschagrunert

@codecov-commenter
Copy link

codecov-commenter commented Sep 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.56%. Comparing base (11d77f4) to head (58f215b).
⚠️ Report is 1003 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3001       +/-   ##
===========================================
- 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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ngopalak-redhat
Copy link
Contributor Author

/retest

@ngopalak-redhat
Copy link
Contributor Author

/test all

@ngopalak-redhat
Copy link
Contributor Author

/test e2e-fedora

@k8s-ci-robot
Copy link
Contributor

@ngopalak-redhat: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test pull-security-profiles-operator-build
/test pull-security-profiles-operator-build-image
/test pull-security-profiles-operator-test-e2e
/test pull-security-profiles-operator-test-unit
/test pull-security-profiles-operator-verify

Use /test all to run all jobs.

In response to this:

/test e2e-fedora

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.

@ngopalak-redhat
Copy link
Contributor Author

@saschagrunert Some weird failures. Unable to re-trigger them. After your review, I can rebase and push.

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

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ngopalak-redhat, 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:

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 added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 11, 2025
@saschagrunert
Copy link
Member

Let's merge, that's just docs. Great work, thank you!

@saschagrunert saschagrunert merged commit d919cb2 into kubernetes-sigs:main Sep 11, 2025
29 of 33 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/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants