On Tue, Mar 28, 2023 at 10:40 AM Martin KaFai Lau <martin.lau@xxxxxxxxx> wrote: > > Hi YiFei and Stan, it is observed that the cg_stroage_* tests fail from time to > time. A recent example is > https://github.com/kernel-patches/bpf/actions/runs/4543867424/jobs/8009943115?pr=3924 > > Could you help to take a look? may be run it under netns and also have better > filtering by ip/port when counting packets? Error: #43/2 cg_storage_multi/isolated test_isolated:PASS:skel-load 0 nsec test_isolated:PASS:parent-egress1-cg-attach 0 nsec test_isolated:PASS:parent-egress2-cg-attach 0 nsec test_isolated:PASS:parent-ingress-cg-attach 0 nsec test_isolated:PASS:first-connect-send 0 nsec test_isolated:FAIL:first-invoke invocations=2 Error: #43/3 cg_storage_multi/shared test_shared:PASS:skel-load 0 nsec test_shared:PASS:parent-egress1-cg-attach 0 nsec test_shared:PASS:parent-egress2-cg-attach 0 nsec test_shared:PASS:parent-ingress-cg-attach 0 nsec test_shared:PASS:first-connect-send 0 nsec test_shared:FAIL:first-invoke invocations=2 Probably because we're using tcp? And race with syn vs syn+ack (invocatoins=1 vs invocations=2)? YiFei, maybe we should count only pure syns? > Thanks!