fixes aws#423: Populate pod list in webhook notification

Author: Abhay Verma

cmd/node-termination-handler.go

@@ -318,23 +318,22 @@ func drainOrCordonIfNecessary(interruptionEventStore *interruptioneventstore.Sto
 		runPreDrainTask(node, nodeName, drainEvent, metrics, recorder)
 	}
 
+        podNameList, err := node.FetchPodNameList(nodeName)
+        if err != nil {
+                log.Err(err).Msgf("Unable to fetch running pods for node '%s' ", nodeName)
+        }
+
 	if nthConfig.CordonOnly || (drainEvent.IsRebalanceRecommendation() && !nthConfig.EnableRebalanceDraining) {
 		err = cordonNode(node, nodeName, drainEvent, metrics, recorder)
 	} else {
 		err = cordonAndDrainNode(node, nodeName, metrics, recorder, nthConfig.EnableSQSTerminationDraining)
 	}
-
-	if err != nil {
-		podNameList, err := node.FetchPodNameList(nodeName)
-		drainEvent.Pods = podNameList
-		if err != nil {
-			log.Err(err).Msgf("Unable to fetch running pods for node '%s' ", nodeName)
-		}
-		err = node.LogPods(podNameList, nodeName)
-		if err != nil {
-			log.Err(err).Msg("There was a problem while trying to log all pod names on the node")
-		}
-	}
+        
+        drainEvent.Pods = podNameList
+        err = node.LogPods(podNameList, nodeName)
+        if err != nil {
+                log.Err(err).Msg("There was a problem while trying to log all pod names on the node")
+        }
 
 	interruptionEventStore.MarkAllAsProcessed(nodeName)
 	if nthConfig.WebhookURL != "" {