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

Support dynamic loading of policies #3239

Open
12 of 19 tasks
geyslan opened this issue Jun 14, 2023 · 6 comments
Open
12 of 19 tasks

Support dynamic loading of policies #3239

geyslan opened this issue Jun 14, 2023 · 6 comments
Assignees
Milestone

Comments

@geyslan
Copy link
Member

geyslan commented Jun 14, 2023

Context: #3209 (comment)

@geyslan

This comment was marked as outdated.

@geyslan

This comment was marked as outdated.

@geyslan geyslan modified the milestones: v0.16.0, v0.17.0 Jun 19, 2023
@geyslan
Copy link
Member Author

geyslan commented Jun 19, 2023

Postponing this since it requires further discussion on how to sync policies in bpf and userland.

@geyslan geyslan modified the milestones: v0.17.0, v0.18.0 Jul 19, 2023
@yanivagman yanivagman changed the title Policies require concurrency control Support dynamic loading of policies Sep 4, 2023
@geyslan geyslan modified the milestones: v0.18.0, v0.19.0 Sep 6, 2023
@geyslan geyslan added the EPIC label Sep 14, 2023
@yanivagman yanivagman modified the milestones: v0.19.0, v0.20.0 Sep 28, 2023
@geyslan geyslan modified the milestones: v0.20.0, v0.21.0 Jan 16, 2024
@yanivagman yanivagman removed this from the v0.21.0 milestone Apr 16, 2024
@yanivagman yanivagman added this to the v0.22.0 milestone Apr 16, 2024
@yanivagman
Copy link
Collaborator

@geyslan I thought about the following API to be added:

AddPolicy(Policy) error
UpdatePolicy(Policy) error
DeletePolicy(Policy) error
ListPolicies()

WDYT?

@yanivagman yanivagman mentioned this issue May 8, 2024
4 tasks
@geyslan
Copy link
Member Author

geyslan commented May 17, 2024

@yanivagman considering that this API should be to PolicyManager, perhaps:

DeletePolicy(name) error

instead of the policyObject as parameter.

@yanivagman
Copy link
Collaborator

yanivagman commented May 19, 2024

@yanivagman considering that this API should be to PolicyManager, perhaps:

DeletePolicy(name) error

instead of the policyObject as parameter.

Yes, I agree.
Getting into more details:
AddPolicy(policy) error - Add a new policy to Tracee. A full policy (struct parsed from the yaml) is provided. If a policy with the same name already exists, an error should be returned.
UpdatePolicy(policy) error - for now, we won't support providing deltas in the update, and full description of the new policy (as a struct) must be provided. We should decide what to do in case a policy with the same name doesn't exist. We can either return an error or simply add the new policy.
DeletePolicy(name) error
ListPolicies() policies

WDYT?

@yanivagman yanivagman modified the milestones: v0.22.0, v0.23.0 Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants