diff --git a/engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/snapshot/SnapshotObject.java b/engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/snapshot/SnapshotObject.java index a3964bd461ec..7602a142f885 100644 --- a/engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/snapshot/SnapshotObject.java +++ b/engine/storage/snapshot/src/main/java/org/apache/cloudstack/storage/snapshot/SnapshotObject.java @@ -172,10 +172,15 @@ public boolean isRevertable() { @Override public long getPhysicalSize() { long physicalSize = 0; - SnapshotDataStoreVO snapshotStore = snapshotStoreDao.findByStoreSnapshot(DataStoreRole.Image, store.getId(), snapshot.getId()); - if (snapshotStore != null) { - physicalSize = snapshotStore.getPhysicalSize(); + for (DataStoreRole role : List.of(DataStoreRole.Image, DataStoreRole.Primary)) { + logger.trace("Retrieving snapshot [{}] size from {} storage.", snapshot.getUuid(), role); + SnapshotDataStoreVO snapshotStore = snapshotStoreDao.findByStoreSnapshot(role, store.getId(), snapshot.getId()); + if (snapshotStore != null) { + return snapshotStore.getPhysicalSize(); + } + logger.trace("Snapshot [{}] size not found on {} storage.", snapshot.getUuid(), role); } + logger.warn("Snapshot [{}] reference not found in any storage. There may be an inconsistency on the database.", snapshot.getUuid()); return physicalSize; }