Copilot commented on code in PR #11625:
URL: https://github.com/apache/cloudstack/pull/11625#discussion_r2344320725
##########
plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java:
##########
@@ -3037,7 +3037,7 @@ public Answer copyVolumeFromPrimaryToPrimary(CopyCommand
cmd) {
destVolumeName = volumeName + "." +
destFormat.getFileExtension();
// Update path in the command for reconciliation
- if (destData.getPath() == null) {
+ if (StringUtils.isBlank(destVolumePath)) {
Review Comment:
The condition check has changed from `destData.getPath() == null` to
`StringUtils.isBlank(destVolumePath)`. This changes the logic from checking if
the destination data path is null to checking if the destination volume path is
blank (null or empty). Ensure this change is intentional and that
`destVolumePath` is the correct variable to check here, as this could affect
the reconciliation path update behavior.
--
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: commits-unsubscr...@cloudstack.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
