Skip to content

Commit e2ea274

Browse files
authored
Merge pull request kubernetes#94729 from dashpole/flaky_metrics
Add retry for creating metrics grabber to fix test flakes
2 parents 2cb951d + 20863a8 commit e2ea274

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

test/e2e/instrumentation/monitoring/metrics_grabber.go

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,9 @@ package monitoring
1818

1919
import (
2020
"context"
21+
"fmt"
2122
"strings"
23+
"time"
2224

2325
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
2426
clientset "k8s.io/client-go/kubernetes"
@@ -40,8 +42,16 @@ var _ = instrumentation.SIGDescribe("MetricsGrabber", func() {
4042
c = f.ClientSet
4143
ec = f.KubemarkExternalClusterClientSet
4244
framework.ExpectNoError(err)
43-
grabber, err = e2emetrics.NewMetricsGrabber(c, ec, true, true, true, true, true)
44-
framework.ExpectNoError(err)
45+
gomega.Eventually(func() error {
46+
grabber, err = e2emetrics.NewMetricsGrabber(c, ec, true, true, true, true, true)
47+
if err != nil {
48+
return fmt.Errorf("failed to create metrics grabber: %v", err)
49+
}
50+
if !grabber.HasControlPlanePods() {
51+
return fmt.Errorf("unable to get find control plane pods")
52+
}
53+
return nil
54+
}, 5*time.Minute, 10*time.Second).Should(gomega.BeNil())
4555
})
4656

4757
ginkgo.It("should grab all metrics from API server.", func() {

0 commit comments

Comments
 (0)