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

fix(ebpf): use debug error level instead of error #3985

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

geyslan
Copy link
Member

@geyslan geyslan commented Apr 17, 2024

1. Explain what the PR does

d621c36 fix(ebpf): use log debug level instead of error...

... for internal requirements.

Considering that if GetLast() failed t.config.Policies is nil, this also
removes the use of t.config.Policies. Furthermore, the goal is to
concetrate all Policies access in the PolicyManager.

2. Explain how to test it

3. Other comments

... for internal requirements.

Considering that if GetLast() failed t.config.Policies is nil, this also
removes the use of t.config.Policies. Furthermore, the goal is to
concetrate all Policies access in the PolicyManager.
@geyslan geyslan marked this pull request as ready for review April 17, 2024 20:14
@geyslan geyslan requested a review from yanivagman April 17, 2024 20:14
@geyslan
Copy link
Member Author

geyslan commented Apr 17, 2024

It's green in E2E 1465.

@geyslan
Copy link
Member Author

geyslan commented Apr 17, 2024

Merging it to unblock E2E.

@geyslan geyslan merged commit e8d7a35 into aquasecurity:main Apr 17, 2024
32 checks passed
@geyslan geyslan deleted the fix-mem-dump branch June 28, 2024 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant