Michał Siwiński

Results 24 comments of Michał Siwiński

Even though PR has approval, it's still missing release label to be mergable. From what I see, `release-note/minor` applies here. Any chance @glibsm that you could add the missing label...

Well, business reason would be that we are offering managed clusters with Hubble based observability and we want to have SLOs for user-facing functionalities that we offer. Bad latency (or...

Seems like some integration / conformance tests are failing, but I don't think it's because of this PR.

Sure, thanks for the info. Let me know when it will be fine to rebase again.

I agree that it would be very useful to have such information on the number of events dropped by rate limiter, though I think it's better to add it as...

Thanks for the comments everyone. I'll try to follow up on your reviews either this week or early next week. From all the follow-up tasks, I will try to implement...

Alright, I resolved the comments that were actionable. Couple of points what has changed and what has not. I tried implementing configuration through file, but as I went deeper with...

@kaworu could you have another look at this? I think all your requested changes were addressed and I'd love to push this PR forward if possible.