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.

Sign up for GitHub

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

Jump to bottom

/sys/fs/bpf hierarchy reflecting sensors/programs/etc. #408

Closed
kkourt opened this issue Sep 15, 2022 · 1 comment
Closed

/sys/fs/bpf hierarchy reflecting sensors/programs/etc. #408

kkourt opened this issue Sep 15, 2022 · 1 comment
Assignees
@kkourt @olsajiri

Comments

@kkourt
Copy link
Contributor

kkourt commented Sep 15, 2022

tetragon bpf fs hierarchy
kkourt added a commit that referenced this issue Sep 15, 2022
@kkourt
sensors: add function for creating bpf fs paths
68fae5b 
This is meant to be used by sensors to create paths for their objects
under /sys/fs/bpf. Ideally, we would want a directory hierarchy under
each sensor, as described in #408.

This is currently not easy to do, however. Instead, use a function that
adds "-" instead of "/", leading to a flat hiearachy that has unique
names. This function can be changed to filepath.Join once everything is
in place.

The purpose of the function is to document the above intention.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
@kkourt kkourt mentioned this issue Sep 15, 2022
Merged
@kkourt kkourt self-assigned this Sep 15, 2022
kkourt added a commit that referenced this issue Sep 15, 2022
@kkourt
sensors: add function for creating bpf fs paths
c472ebc 
This is meant to be used by sensors to create paths for their objects
under /sys/fs/bpf. Ideally, we would want a directory hierarchy under
each sensor, as described in #408.

This is currently not easy to do, however. Instead, use a function that
adds "-" instead of "/", leading to a flat hierarchy that has unique
names. This function can be changed to filepath.Join once everything is
in place.

The purpose of the function is to document the above intention.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
kkourt added a commit that referenced this issue Sep 16, 2022
@kkourt
sensors: add function for creating bpf fs paths
1a6e259 
This is meant to be used by sensors to create paths for their objects
under /sys/fs/bpf. Ideally, we would want a directory hierarchy under
each sensor, as described in #408.

This is currently not easy to do, however. Instead, use a function that
adds "-" instead of "/", leading to a flat hierarchy that has unique
names. This function can be changed to filepath.Join once everything is
in place.

The purpose of the function is to document the above intention.

Signed-off-by: Kornilios Kourtis <kornilios@isovalent.com>
@kkourt kkourt changed the title /sys/fs/bpf hierracy reflecting sensors/programs/etc. /sys/fs/bpf hierarchy reflecting sensors/programs/etc. Sep 30, 2022
@olsajiri olsajiri self-assigned this Feb 13, 2024
@kkourt kkourt mentioned this issue Jul 22, 2024
Open
@kkourt
Copy link
Contributor Author

kkourt commented Nov 21, 2024

Done by #2128 :tad

@kkourt kkourt closed this as completed Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kkourt kkourt

@olsajiri olsajiri

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kkourt @olsajiri