Log and aggregate errors

Author: Qqkyu

kubetest2-gke/deployer/down.go

@@ -17,6 +17,7 @@ limitations under the License.
 package deployer
 
 import (
+	"errors"
 	"fmt"
 	"sync"
 
@@ -55,19 +56,27 @@ func (d *Deployer) Down() error {
 		klog.V(1).Infof("Deleted %d network firewall rules", numDeletedFWRules)
 	}
 
-	if err := d.CleanupNat(); err != nil {
-		klog.Errorf("Error cleaning-up nat: %v", err)
+	errCleanNat := d.CleanupNat()
+	if errCleanNat != nil {
+		klog.Errorf("Error cleaning up NAT: %v", errCleanNat)
 	}
 
-	if err := d.TeardownNetwork(); err != nil {
-		klog.Errorf("Error tearing-down network: %v", err)
+	errTeardownNetwork := d.TeardownNetwork()
+	if errTeardownNetwork != nil {
+		klog.Errorf("Error tearing down network: %v", errTeardownNetwork)
 	}
 
-	if err := d.DeleteSubnets(d.retryCount); err != nil {
-		klog.Errorf("Error deleting subnets: %v", err)
+	errDeleteSubnets := d.DeleteSubnets(d.retryCount)
+	if errDeleteSubnets != nil {
+		klog.Errorf("Error deleting subnets: %v", errDeleteSubnets)
 	}
 
-	return d.DeleteNetwork()
+	errDeleteNetwork := d.DeleteNetwork()
+	if errDeleteNetwork != nil {
+		klog.Errorf("Error deleting network: %v", errDeleteNetwork)
+	}
+
+	return errors.Join(errCleanFirewalls, errCleanNat, errTeardownNetwork, errDeleteSubnets, errDeleteNetwork)
 }
 
 func (d *Deployer) DeleteClusters(retryCount int) {