-
Notifications
You must be signed in to change notification settings - Fork 7
Experimental service mapping pt.2 #164
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
Experimental service mapping pt.2 #164
Conversation
…container_id function and service_cache function. Added ServiceIdentity struct to store services relevant informations. Added Identites vector to store multiple ServiceIdentity structures
|
@siddh34 No tests needed. I'm manually closing this PR 💯 |
82f8b48
into
CortexFlow:feature/ebpf-core
|
Ahh man! No worries I will still try to test it out let you know if there are any issues if I found some issues |
|
@siddh34 I'm merging the changes into the PreReleaseMain branch right now. After that i'll push the 0.1.1 update to crates.io 🥇 |
|
Go ahead man! Will catch up to you soon All the best for release! 🚀🚀 |
Getting few errors @LorenzoTettamanti |
|
@siddh34 hi mate, thanks for the review. In the second image the error is triggered because the program cannot find that path slice in the cgroup and the other part of the triggers. For example if you use minikube the cgroup path in the container is something like sys/fs/cgroup/kubepods/... but if you use Kind the cgroup path is sys/fs/cgroup/kubelet-kubepods... In the first image the error is triggered when the program cannot find a valid pod-name ( from the kubeapi) that has a direct match with the pod uid extracted from the cgroup. I'm already thinking about a better way to extract the relevant pod info, i'll let you know very soon! |


Changes
Changes:
[[Feature]: Add command discovery for network events #150]: added extract_docker_id() unit test. Added extract_container_id function and service_cache function. Added ServiceIdentity struct to store services relevant informations. Added Identites vector to store multiple ServiceIdentity structures
Improved cfcli monitoring latencymetrics response for better output clarity
Marked experimental features with #[cfg(feature="experimental")
Updated manifests' files version (identity.yaml,metrics.yaml,agent.yaml)
Tests
Images
cortexflow-identity:0.1.1 stable release
cortexflow-identity:0.1.1-experimental
Expected output