Merge pull request #121554 from kaisoz/fix-job-controller-panics
Check if time is nil before logging
This commit is contained in:
		@@ -951,7 +951,7 @@ func FilterActivePods(logger klog.Logger, pods []*v1.Pod) []*v1.Pod {
 | 
				
			|||||||
		if IsPodActive(p) {
 | 
							if IsPodActive(p) {
 | 
				
			||||||
			result = append(result, p)
 | 
								result = append(result, p)
 | 
				
			||||||
		} else {
 | 
							} else {
 | 
				
			||||||
			logger.V(4).Info("Ignoring inactive pod", "pod", klog.KObj(p), "phase", p.Status.Phase, "deletionTime", p.DeletionTimestamp)
 | 
								logger.V(4).Info("Ignoring inactive pod", "pod", klog.KObj(p), "phase", p.Status.Phase, "deletionTime", klog.SafePtr(p.DeletionTimestamp))
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
	return result
 | 
						return result
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user