Tonitzpp commented on code in PR #12367:
URL: https://github.com/apache/cloudstack/pull/12367#discussion_r2668034676


##########
server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java:
##########
@@ -2199,7 +2201,7 @@ public Volume changeDiskOfferingForVolumeInternal(Long 
volumeId, Long newDiskOff
                 if (volumeMigrateRequired) {
                     logger.warn(String.format("Volume change offering 
operation succeeded for volume ID: %s but volume resize operation failed, so 
please try resize volume operation separately", volume.getUuid()));
                 } else {
-                    throw new CloudRuntimeException(String.format("Volume 
change offering operation failed for volume ID: %s due to resize volume 
operation failed", volume.getUuid()));
+                    throw new CloudRuntimeException(String.format("Volume disk 
offering change operation failed for volume ID [%s] because the volume resize 
operation failed.", volume.getUuid()));

Review Comment:
   Using [ ] will make it easier for users or operators to identify the ID. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to