Skip to content

Add TTFR test#9

Merged
lwr20 merged 6 commits intomainfrom
ttfr
May 16, 2025
Merged

Add TTFR test#9
lwr20 merged 6 commits intomainfrom
ttfr

Conversation

@lwr20
Copy link
Member

@lwr20 lwr20 commented May 2, 2025

This "time to first response" (TTFR) test spins up a server pod on each node in the cluster, and then spins up client pods on each node in the cluster. The client pods start and send requests to the server pod, and record the amount of time it takes before they get a response. This is sometimes[1] a useful proxy for how long its taking for Calico to program the
rules for that pod (since pods start with a deny-all rule and calico-node must program the correct rules before it can talk to anything).

[1] if linuxPolicySetupTimeoutSeconds is set in the CalicoNetworkSpec in the Installation resource, then pod startup will be delayed until policy is applied. This can be handy if your application pod wants its first request to always succeed. This is a Calico-specific feature that is not part of the CNI spec. See the Calico documentation for more information on this feature and how to enable it.

@lwr20 lwr20 force-pushed the ttfr branch 7 times, most recently from 390feb6 to e907e12 Compare May 6, 2025 17:00
@lwr20 lwr20 force-pushed the ttfr branch 13 times, most recently from f781541 to a4f15fa Compare May 13, 2025 08:28
Copy link

@Glen-Tigera Glen-Tigera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

return string(b)
}

// kubectl debug -it ephemeral-demo --image=busybox:1.28 --target=ephemeral-demo

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still needed?

@lwr20 lwr20 marked this pull request as ready for review May 15, 2025 10:43
@lwr20 lwr20 changed the title Initial commit of TTFR test Add TTFR test May 15, 2025
Copy link
Member Author

@lwr20 lwr20 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs docs to be updated to explain the new test.

@lwr20 lwr20 force-pushed the ttfr branch 3 times, most recently from be076cd to b414a83 Compare May 15, 2025 14:59
@lwr20 lwr20 force-pushed the ttfr branch 4 times, most recently from 6550def to daa4282 Compare May 16, 2025 10:06
@lwr20 lwr20 merged commit 3a2dbaa into main May 16, 2025
6 checks passed
@lwr20 lwr20 deleted the ttfr branch May 16, 2025 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants