add volume ID to CreateSnapshot error message

hungnguyen243 · hungnguyen243 · commit eb3e6eb3c0c6 · 2024-05-24T22:39:01.000Z
diff --git a/pkg/gce-pd-csi-driver/controller.go b/pkg/gce-pd-csi-driver/controller.go
@@ -1160,7 +1160,7 @@ func (gceCS *GCEControllerServer) CreateSnapshot(ctx context.Context, req *csi.C
 
 	volumeIsMultiZone := isMultiZoneVolKey(volKey)
 	if gceCS.multiZoneVolumeHandleConfig.Enable && volumeIsMultiZone {
-		return nil, status.Errorf(codes.InvalidArgument, "Snapshots are not supported with the multi-zone PV volumeHandle feature")
+		return nil, status.Errorf(codes.InvalidArgument, "CreateSnapshot for volume %v failed. Snapshots are not supported with the multi-zone PV volumeHandle feature", volumeID)
 	}
 
 	if acquired := gceCS.volumeLocks.TryAcquire(volumeID); !acquired {

Original file line number	Diff line number	Diff line change
`@@ -1160,7 +1160,7 @@ func (gceCS GCEControllerServer) CreateSnapshot(ctx context.Context, req csi.C`
`1160`	`1160`
`1161`	`1161`	`volumeIsMultiZone := isMultiZoneVolKey(volKey)`
`1162`	`1162`	`if gceCS.multiZoneVolumeHandleConfig.Enable && volumeIsMultiZone {`
`1163`		`- return nil, status.Errorf(codes.InvalidArgument, "Snapshots are not supported with the multi-zone PV volumeHandle feature")`
	`1163`	`+ return nil, status.Errorf(codes.InvalidArgument, "CreateSnapshot for volume %v failed. Snapshots are not supported with the multi-zone PV volumeHandle feature", volumeID)`
`1164`	`1164`	`}`
`1165`	`1165`
`1166`	`1166`	`if acquired := gceCS.volumeLocks.TryAcquire(volumeID); !acquired {`