Merge pull request #128498 from googs1025/flake/TestReconcilerAPIServerLeaseMultiCombined

flake(TestReconcilerAPIServerLeaseMultiCombined): fix TestReconcilerAPIServerLeaseMultiCombined flake
This commit is contained in:
Kubernetes Prow Robot 2024-11-07 16:10:43 +00:00 committed by GitHub
commit a660b51201
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -24,7 +24,6 @@ import (
"net/http" "net/http"
"reflect" "reflect"
"strings" "strings"
"sync"
"testing" "testing"
"time" "time"
@ -440,22 +439,16 @@ func testReconcilersAPIServerLease(t *testing.T, leaseCount int, apiServerCount
instanceOptions := kubeapiservertesting.NewDefaultTestServerOptions() instanceOptions := kubeapiservertesting.NewDefaultTestServerOptions()
wg := sync.WaitGroup{}
// 1. start apiServerCount api servers // 1. start apiServerCount api servers
for i := 0; i < apiServerCount; i++ { for i := 0; i < apiServerCount; i++ {
// start count api server // start count api server
wg.Add(1) server := kubeapiservertesting.StartTestServerOrDie(t, instanceOptions, []string{
go func(i int) { "--endpoint-reconciler-type", "master-count",
defer wg.Done() "--advertise-address", fmt.Sprintf("10.0.1.%v", i+1),
server := kubeapiservertesting.StartTestServerOrDie(t, instanceOptions, []string{ "--apiserver-count", fmt.Sprintf("%v", apiServerCount),
"--endpoint-reconciler-type", "master-count", }, etcd)
"--advertise-address", fmt.Sprintf("10.0.1.%v", i+1), apiServerCountServers[i] = server
"--apiserver-count", fmt.Sprintf("%v", apiServerCount),
}, etcd)
apiServerCountServers[i] = server
}(i)
} }
wg.Wait()
// 2. verify API Server count servers have registered // 2. verify API Server count servers have registered
if err := wait.PollImmediate(3*time.Second, 2*time.Minute, func() (bool, error) { if err := wait.PollImmediate(3*time.Second, 2*time.Minute, func() (bool, error) {
@ -476,18 +469,14 @@ func testReconcilersAPIServerLease(t *testing.T, leaseCount int, apiServerCount
// 3. start lease api servers // 3. start lease api servers
for i := 0; i < leaseCount; i++ { for i := 0; i < leaseCount; i++ {
wg.Add(1) options := []string{
go func(i int) { "--endpoint-reconciler-type", "lease",
defer wg.Done() "--advertise-address", fmt.Sprintf("10.0.1.%v", i+10),
options := []string{ }
"--endpoint-reconciler-type", "lease", server := kubeapiservertesting.StartTestServerOrDie(t, instanceOptions, options, etcd)
"--advertise-address", fmt.Sprintf("10.0.1.%v", i+10), leaseServers[i] = server
}
server := kubeapiservertesting.StartTestServerOrDie(t, instanceOptions, options, etcd)
leaseServers[i] = server
}(i)
} }
wg.Wait()
defer func() { defer func() {
for i := 0; i < leaseCount; i++ { for i := 0; i < leaseCount; i++ {
leaseServers[i].TearDownFn() leaseServers[i].TearDownFn()