Skip to content

Commit

Permalink
test/e2e: Swap some Fatal for Error
Browse files Browse the repository at this point in the history
If we just hit timeout whilst waiting for pods to get to
into the expected state, then `Error` instead of `Fatal`
as we don't want to skip the logging and cleanup.

Signed-off-by: stevenhorsman <[email protected]>
  • Loading branch information
stevenhorsman committed Dec 20, 2024
1 parent fec1d1e commit 57891d9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/cloud-api-adaptor/test/e2e/assessment_runner.go
Original file line number Diff line number Diff line change
Expand Up @@ -285,11 +285,11 @@ func (tc *TestCase) Run() {
//Added logs for debugging nightly tests
clientset, err := kubernetes.NewForConfig(client.RESTConfig())
if err != nil {
t.Fatal(err)
t.Error(err)
}
pod, err := clientset.CoreV1().Pods(tc.pod.Namespace).Get(ctx, tc.pod.Name, metav1.GetOptions{})
if err != nil {
t.Fatal(err)
t.Error(err)
}
t.Logf("Expected Pod State: %v", tc.podState)
yamlData, err := yaml.Marshal(pod.Status)
Expand All @@ -304,7 +304,7 @@ func (tc *TestCase) Run() {
t.Log(podLogString)
t.Logf("===================\n")
}
t.Fatal(err)
t.Error(err)
}
}
}
Expand Down

0 comments on commit 57891d9

Please sign in to comment.