Skip to content

Commit 360ca33

Browse files
Merge pull request #88 from martinkunc/add-log-when-tests-fail
Report logs when checkPods is going to fail
2 parents 508da9e + d4605f1 commit 360ca33

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

test/integration/basic_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ func TestOptOutOptIn(t *testing.T) {
5656
resourceVersion := latestSecret.GetResourceVersion()
5757
pullSecret.SetResourceVersion(resourceVersion) // need to update the version, otherwise operation is not permitted
5858

59-
errConfig := wait.PollImmediate(5*time.Second, 5*time.Minute, func() (bool, error) {
59+
errConfig := wait.PollImmediate(5*time.Second, 10*time.Minute, func() (bool, error) {
6060
objs := map[string]interface{}{}
6161
errUnmarshals := json.Unmarshal([]byte(pullSecret.Data[".dockerconfigjson"]), &objs)
6262
if errUnmarshals != nil {
@@ -79,7 +79,7 @@ func TestOptOutOptIn(t *testing.T) {
7979

8080
// Check if reports are uploaded - Logs show that insights-operator is enabled and reports are uploaded
8181
restartInsightsOperator(t)
82-
errDisabled := wait.PollImmediate(1*time.Second, 10*time.Minute, func() (bool, error) {
82+
errDisabled := wait.PollImmediate(1*time.Second, 20*time.Minute, func() (bool, error) {
8383
insightsDisabled := isOperatorDisabled(t, clusterOperatorInsights())
8484
if insightsDisabled {
8585
return false, nil

test/integration/bugs_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ func TestUnreachableHost(t *testing.T) {
9696
t.Fatal(err.Error())
9797
}
9898
// Check the operator is not degraded anymore
99-
errDegraded := wait.PollImmediate(1*time.Second, 10*time.Minute, func() (bool, error) {
99+
errDegraded := wait.PollImmediate(1*time.Second, 20*time.Minute, func() (bool, error) {
100100
insightsDegraded := isOperatorDegraded(t, clusterOperatorInsights())
101101
if insightsDegraded {
102102
return false, nil

test/integration/main_test.go

+4-3
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ func restartInsightsOperator(t *testing.T) {
102102

103103
for _, pod := range pods.Items {
104104
clientset.CoreV1().Pods("openshift-insights").Delete(pod.Name, &metav1.DeleteOptions{})
105-
err := wait.PollImmediate(1*time.Second, 10*time.Minute, func() (bool, error) {
105+
err := wait.PollImmediate(1*time.Second, 20*time.Minute, func() (bool, error) {
106106
_, err := clientset.CoreV1().Pods("openshift-insights").Get(pod.Name, metav1.GetOptions{})
107107
if err == nil {
108108
t.Logf("the pod is not yet deleted: %v\n", err)
@@ -150,7 +150,7 @@ func checkPodsLogs(t *testing.T, kubeClient *kubernetes.Clientset, message strin
150150
panic(err.Error())
151151
}
152152

153-
errLog := wait.PollImmediate(5*time.Second, 15*time.Minute, func() (bool, error) {
153+
errLog := wait.PollImmediate(5*time.Second, 30*time.Minute, func() (bool, error) {
154154
req := kubeClient.CoreV1().Pods(pod.Namespace).GetLogs(pod.Name, &corev1.PodLogOptions{})
155155
podLogs, err := req.Stream()
156156
if err != nil {
@@ -168,6 +168,7 @@ func checkPodsLogs(t *testing.T, kubeClient *kubernetes.Clientset, message strin
168168
result := strings.Contains(log, message)
169169
if result == false {
170170
t.Logf("No %s in logs\n", message)
171+
t.Logf("Logs for verification: ****\n%s", log)
171172
return false, nil
172173
}
173174

@@ -193,7 +194,7 @@ func TestMain(m *testing.M) {
193194
func waitForOperator(kubeClient *kubernetes.Clientset) error {
194195
depClient := kubeClient.AppsV1().Deployments("openshift-insights")
195196

196-
err := wait.PollImmediate(1*time.Second, 10*time.Minute, func() (bool, error) {
197+
err := wait.PollImmediate(1*time.Second, 20*time.Minute, func() (bool, error) {
197198
_, err := depClient.Get("insights-operator", metav1.GetOptions{})
198199
if err != nil {
199200
fmt.Printf("error waiting for operator deployment to exist: %v\n", err)

0 commit comments

Comments
 (0)