Filter out user errors from GetDisk error returned from ControllerPublishVolume

amacaskill · k8s-infra-cherrypick-robot · commit 6001b9758759 · 2023-08-02T20:51:53.000Z
diff --git a/pkg/gce-pd-csi-driver/controller.go b/pkg/gce-pd-csi-driver/controller.go
@@ -584,7 +584,7 @@ func (gceCS *GCEControllerServer) executeControllerPublishVolume(ctx context.Con
 		if gce.IsGCENotFoundError(err) {
 			return nil, status.Errorf(codes.NotFound, "Could not find disk %v: %v", volKey.String(), err.Error()), diskType
 		}
-		return nil, status.Errorf(codes.Internal, "Failed to getDisk: %v", err.Error()), diskType
+		return nil, common.LoggedError("Failed to getDisk: ", err), diskType
 	}
 	instanceZone, instanceName, err := common.NodeIDToZoneAndName(nodeID)
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -584,7 +584,7 @@ func (gceCS *GCEControllerServer) executeControllerPublishVolume(ctx context.Con`
`584`	`584`	`if gce.IsGCENotFoundError(err) {`
`585`	`585`	`return nil, status.Errorf(codes.NotFound, "Could not find disk %v: %v", volKey.String(), err.Error()), diskType`
`586`	`586`	`}`
`587`		`- return nil, status.Errorf(codes.Internal, "Failed to getDisk: %v", err.Error()), diskType`
	`587`	`+ return nil, common.LoggedError("Failed to getDisk: ", err), diskType`
`588`	`588`	`}`
`589`	`589`	`instanceZone, instanceName, err := common.NodeIDToZoneAndName(nodeID)`
`590`	`590`	`if err != nil {`