Revision 89cc23c09ec31bba50521495d6e352204feec72f authored by David Bimmler on 19 March 2024, 08:48:32 UTC, committed by Jussi Mäki on 05 April 2024, 12:00:13 UTC
[ upstream commit 58ad35a84faab45fa0dab5f0078287f4cceb3926 ] The endpointmanagers idallocator package was using a package global pool for its identifier allocation. That's fine for running the agent, but causes flakes in testing when multiple tests access the same pool. It's also not idiomatic Go. This patch makes the local endpoint identifier allocator a struct, and the next patch will move it into the endpointmanager package itself, as there is no other consumer. While at it, also ensure that the RemoveAll method is only called from a testing context, by taking a testing.TB as an argument. We cannot simply move the method into the _test.go files, as tests from other packages use it. Signed-off-by: David Bimmler <david.bimmler@isovalent.com> Signed-off-by: Jussi Maki <jussi@isovalent.com>
1 parent fdfbb25
| File | Mode | Size |
|---|---|---|
| bigtcp | ||
| bpf | ||
| bpf_tests | ||
| config | ||
| consul | ||
| controlplane | ||
| eks | ||
| envoy | ||
| fuzzing | ||
| ginkgo-ext | ||
| gke | ||
| helpers | ||
| k8s | ||
| l4lb | ||
| logger | ||
| nat46x64 | ||
| packet | ||
| provision | ||
| runtime | ||
| standalone | ||
| vtep | ||
| Makefile | -rw-r--r-- | 2.5 KB |
| README.md | -rw-r--r-- | 160 bytes |
| Vagrantfile | -rw-r--r-- | 11.7 KB |
| archive_test_results.sh | -rwxr-xr-x | 302 bytes |
| archive_test_results_eks.sh | -rwxr-xr-x | 303 bytes |
| get-gh-comment-info.py | -rw-r--r-- | 604 bytes |
| get-vagrant-kubeconfig.sh | -rwxr-xr-x | 304 bytes |
| kubernetes-netperftest.sh | -rwxr-xr-x | 507 bytes |
| kubernetes-test.sh | -rwxr-xr-x | 4.5 KB |
| post_build_agent.sh | -rwxr-xr-x | 376 bytes |
| print-node-ip.sh | -rwxr-xr-x | 130 bytes |
| test_suite_test.go | -rw-r--r-- | 9.6 KB |
| vagrant-ci-start.sh | -rwxr-xr-x | 1.4 KB |
| vagrant-local-create-box.sh | -rwxr-xr-x | 567 bytes |
| vagrant-local-start-runtime.sh | -rwxr-xr-x | 370 bytes |
| vagrant-local-start.sh | -rwxr-xr-x | 743 bytes |
Computing file changes ...
