Add metrics for lru cache in pkg/observer/data.go

[olsajiri]

would be good to add metrics for lru cache in pkg/observer/data.go, like we do in pkg/process/cache.go

so we can eventually watch and suggest better value for the cache

[rajaSahil]

@kkourt Can I start working on this issue?

[kkourt]

@rajaSahil I'm not aware of anyone working on it, so yes :)

[rajaSahil]

@kkourt Thank you! Can you please assign it to me?

[kkourt]

@kkourt Thank you! Can you please assign it to me?

Sure thing! Let me know if you need any help.

[Neilblaze]

@rajaSahil are you still on it? If not we can collaborate. @kkourt If possible can you assign both of us? 😄

[Neilblaze]

Update: As a newcomer, I've been trying my level best to get familiar with the codebase. Here's my approach & the following changes can be observed here. Lemme know if I need to update anything.

cc: @kkourt

[YTGhost]

Perhaps I can take on the job if the previous two have moved on from the issue.

[Neilblaze]

@YTGhost fyi, I already had started working on it. You can inspect my previous comment for reference.

I'm just waiting for feedback from @kkourt.

[Neilblaze]

@YTGhost But yeah, no worries! you can go ahead & raise a PR if you want.

[mtardy]

Update: As a newcomer, I've been trying my level best to get familiar with the codebase. Here's my approach & the following changes can be observed here. Lemme know if I need to update anything.

cc: @kkourt

your links are private, please create a PR or a draft PR if you want someone to review your changes.

[Neilblaze]

@mtardy Oh no! I just realized that I mistakenly force-pushed all the master content into the working branch, thus overriding the entire data. 😓

I'll have to check if I can recover it, but in the meantime, YTGhost can work on it.

[mtardy]

Ok I'll unassign this for now. If anyone wants to take this.

[AshishNaware]

question - what kind of metrics are expected to be added here? Is it about adding error metrics after cache eviction?