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

Writeable datasource types #3759

Merged

Conversation

NDStrahilevitz
Copy link
Collaborator

1. Explain what the PR does

Add types for PR #3725
Fix #3757 (opportunistically)

dfa6fe5 feature(types): add WritableDatasource interface
ef3bcd6 feat(ebpf): synchronize ready callback to pipeline
c831127 fix(grpc): cleanup listen address

2. Explain how to test it

For the bugfix: run tracee with --grpc-listen-addr set to something (tested with both unix and tcp) close tracee and run again. Note that tracee will not fail anymore.

Make sure listening address is cleaned up wether gracefully or on
tracee's restart.
Make sure the event pipeline starts before the ready callback is called.
This is so the ready callback can reliably use tracee methods which may
only work post pipeline initialization.
Copy link
Contributor

@rafaeldtinoco rafaeldtinoco left a comment

Choose a reason for hiding this comment

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

LGTM

@rafaeldtinoco rafaeldtinoco merged commit e796975 into aquasecurity:main Dec 13, 2023
31 checks passed
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.

Cannot reuse the same gRPC unix socket path on restart of Tracee
2 participants