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

[processor/k8sattributes] Store only necessary data in the pod/replicaset/namespace informers #23226

Closed
dmitryax opened this issue Jun 7, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request processor/k8sattributes k8s Attributes processor

Comments

@dmitryax
Copy link
Member

dmitryax commented Jun 7, 2023

Apply the same optimization as done in SumoLogic/sumologic-otel-collector#593 to all the informers in order to reduce the memory footprint per suggestion from @swiatekm-sumo.

Hopefully, it'll make the heap usage less than before introducing cluster-wise replicaset informers, as reported in #23067.

@swiatekm
Copy link
Contributor

Are we sure we want to do namespaces? Relatively speaking, there tends to be a small number of them in the cluster, and they don't carry a lot of data. Maybe the memory savings aren't worth the additional complexity in their case?

@dmitryax
Copy link
Member Author

Are we sure we want to do namespaces? Relatively speaking, there tends to be a small number of them in the cluster, and they don't carry a lot of data. Maybe the memory savings aren't worth the additional complexity in their case?

I agree. Let's keep the namespaces informer as is.

Closing this issue. Thank you, @swiatekm-sumo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request processor/k8sattributes k8s Attributes processor
Projects
None yet
Development

No branches or pull requests

2 participants