Tiramisu 1993

Results 17 comments of Tiramisu 1993

@liaojie Hi, any time to update this issue? Thank you

>Are these optimizations worth wild? My assumption is that the String() method isn't called particularly often, but measuring this would seem to be the first step to determine if this...

I found we already called `/bin/sh` in node init daemonset in https://github.com/cilium/cilium/blob/master/install/kubernetes/cilium/templates/cilium-nodeinit/daemonset.yaml#L57. Does this issue still available? @pchaigno @aanm

Hi @christarazi ,I'm new to cilium and I think this issue can be my starting point about cilium contribution. Can you please assign this issue to me. Thanks

I just change some code in `decision.go` and the benchmark output like: Does it make sense to you to optimize? I will commit PR if so. Thank you

> I just change some code in `decision.go` and the benchmark output like: > > Does it make sense to you to optimize? I will commit PR if so. Thank...

@christarazi Does this commit make sense to you? https://github.com/MikeLing/cilium/commit/b86b803fdd1b08411a4a4c48d808fe13c0439baa

> Yes please submit a PR. Done ;) https://github.com/cilium/cilium/pull/19822

Hi @christarazi , these 3 PRs (https://github.com/cilium/cilium/pull/19822, https://github.com/cilium/cilium/pull/19704, https://github.com/cilium/cilium/pull/19661) have been approved. Could we merge them although some CI tests failed due to container timeout? Just want to avoid too...