Skip to content

Commit

Permalink
Merge pull request #60 from razo7/pod-name-e2e
Browse files Browse the repository at this point in the history
Log Pod Name on Error
  • Loading branch information
openshift-merge-robot authored Jun 29, 2023
2 parents 0ed0614 + 0560ed6 commit 69ba62b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/e2e/far_e2e_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -250,10 +250,10 @@ func checkFarLogs(logString string) {
if apiErrors.IsNotFound(err) {
// If FAR pod was running in testNodeName, then after reboot it was recreated in another node, and with a new name.
// Thus the "old" pod's name prior to this eventually won't link to a running pod, since it was already evicted by the reboot
log.Error(err, "failed to get logs. FAR pod might have been recreated due to rebooting the node it was resided. Might try again", "pod", pod)
log.Error(err, "failed to get logs. FAR pod might have been recreated due to rebooting the node it was resided. Might try again", "pod", pod.Name)
return ""
}
log.Error(err, "failed to get logs. Might try again", "pod", pod)
log.Error(err, "failed to get logs. Might try again", "pod", pod.Name)
return ""
}
return logs
Expand Down

0 comments on commit 69ba62b

Please sign in to comment.