Fabio Falzoi
Fabio Falzoi
I left another question inline regarding the default. Thanks for the thorough description of the problem in the commit message :100: :+1:
Great work! :+1: I've completed a first pass and left some (minor) comments, I'll do a second pass tomorrow.
Another potential hit [here](https://github.com/cilium/cilium/actions/runs/7395578181/job/20119008822) ``` [=] Test [no-interrupted-connections] [-] Scenario [no-interrupted-connections/no-interrupted-connections] đĨ Pod test-conn-disrupt-client-647c46854d-rr9bp flow was interrupted (restart count does not match 0 != 1) ``` [Full logs](https://github.com/cilium/cilium/files/13820112/logs_1987253.zip) Sysdump too...
Hit another one [here](https://github.com/cilium/cilium/actions/runs/6012006894/job/16306521008) Matrix entry: (5, 5.15-20230810.091425, iptables, true, disabled, dsr, true, true, true) ``` [=] Test [client-egress-l7-named-port] ............ âšī¸ đ Applying CiliumNetworkPolicy 'client-egress-only-dns' to namespace 'cilium-test'.. âšī¸ đ...
Another hit [here](https://github.com/cilium/cilium/actions/runs/6651360038/job/18073278255): ``` [=] Test [client-egress-l7-named-port] ............. âšī¸ đ Applying CiliumNetworkPolicy 'client-egress-only-dns' to namespace 'cilium-test'.. âšī¸ đ Applying CiliumNetworkPolicy 'client-egress-only-dns' to namespace 'cilium-test'.. âšī¸ đ Applying CiliumNetworkPolicy 'client-egress-l7-http-named-port' to...
I realized I missed some minor changes, so I'll temporarily mark this as not ready to be merged.
Hey @derailed I added a couple of commits to make bugtool easily extendable for specific environments. Do you mind taking another look? Compare to the previous version you already reviewed,...
- https://github.com/cilium/cilium/actions/runs/8967430105/job/24624952244?pr=32253 hit https://github.com/cilium/cilium/issues/13071