Fabio Falzoi

Results 42 comments of 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