NFS3 protocol specific snapshot workflows by rajiv-jain-netapp · Pull Request #36 · NetApp/cloudstack

rajiv-jain-netapp · 2026-02-20T06:30:49Z

Description

This PR...

Types of changes

Breaking change (fix or feature that would cause existing functionality to change)
New feature (non-breaking change which adds functionality)
Bug fix (non-breaking change which fixes an issue)
Enhancement (improves an existing feature and functionality)
Cleanup (Code refactoring and cleanup, that may add test cases)
Build/CI
Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

Major
Minor

Bug Severity

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

rajiv-jain-netapp · 2026-02-20T06:31:03Z

I have tested

Taking a CS-volume snapshot for the data CS-volume.
Tried to take VM-level snapshot, CloudStack asked me to select a single CS-volume instead of allowing to take a VM-level snapshot. That also worked.

…iecing is enabled and memory-disk is opted out

rajiv-jain-netapp · 2026-02-21T11:18:03Z

I have tested VM level and cloudstack-volume level snapshots, all found working

VM level snapshot without memory-disk and quiecing option.
VM level snapshot with quiecing option enabled. --> did not find working and error says, unable to quiece VM since emu agent is not found working on VM
VM level snapshot with memory snapshot enabled--> working
VM level snapshot with memory-snapshot and quiecing options enabled --> working fine.
cloudstack volume level snapshots are working fine.

suryag1201 · 2026-02-23T17:25:49Z

...ain/java/org/apache/cloudstack/storage/vmsnapshot/KvmFileBasedStorageVmSnapshotStrategy.java


-        logger.error("Disk-only VM snapshot for VM [{}] failed{}.", userVm.getUuid(), answer != null ? " due to" + answer.getDetails() : "");
-        throw new CloudRuntimeException(String.format("Disk-only VM snapshot for VM [%s] failed.", userVm.getUuid()));
+        String details = answer != null ? answer.getDetails() : String.format("No answer received from host [%s]. The host may be unreachable.", hostId);


why this change is required?

I am returning the failure reasons through the exception.

suryag1201 · 2026-02-23T17:27:54Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java


    @Override
    public void takeSnapshot(SnapshotInfo snapshot, AsyncCompletionCallback<CreateCmdResult> callback) {
+        s_logger.error("temp takeSnapshot : entered with snapshot id: " + snapshot.getId() + " and name: " + snapshot.getName());


log level should not be error plus remove temp

Yes, this was intentionally kept for testing and was mistakenly overlooked during cleanup.

suryag1201 · 2026-02-23T17:29:17Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            Map<String, String> poolDetails = storagePoolDetailsDao.listDetailsKeyPairs(volumeVO.getPoolId());
+            StorageStrategy storageStrategy = Utility.getStrategyByStoragePoolDetails(poolDetails);
+
+            Map<String, String> cloudStackVolumeRequestMap = new HashMap<>();


Get Volume Request is not as per protocol

added protocol check.

suryag1201 · 2026-02-23T17:30:58Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            cloudStackVolumeRequestMap.put(Constants.VOLUME_UUID, poolDetails.get(Constants.VOLUME_UUID));
+            cloudStackVolumeRequestMap.put(Constants.FILE_PATH, volumeVO.getPath());
+            CloudStackVolume cloudStackVolume = storageStrategy.getCloudStackVolume(cloudStackVolumeRequestMap);
+            if (cloudStackVolume == null || cloudStackVolume.getFile() == null) {


this condition is also specific to file, it will fail for lun

suryag1201 · 2026-02-23T17:31:25Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+                throw new CloudRuntimeException("takeSnapshot: Failed to get source file to take snapshot");
+            }
+            long capacityBytes = storagePool.getCapacityBytes();
+            s_logger.error("temp takeSnapshot : entered after getting cloudstack volume with file path: " + cloudStackVolume.getFile().getPath() + " and size: " + cloudStackVolume.getFile().getSize());


log level should not be error

suryag1201 · 2026-02-23T17:32:04Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            long capacityBytes = storagePool.getCapacityBytes();
+            s_logger.error("temp takeSnapshot : entered after getting cloudstack volume with file path: " + cloudStackVolume.getFile().getPath() + " and size: " + cloudStackVolume.getFile().getSize());
+            long usedBytes = getUsedBytes(storagePool);
+            long fileSize = cloudStackVolume.getFile().getSize();


these all changes are related to file

suryag1201 · 2026-02-23T17:32:40Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+
+            String fileSnapshotName = volumeInfo.getName() + "-" + snapshot.getUuid();

+            int maxSnapshotNameLength = 64;


I think, we can have more character in snapshot name

moved it to contants for now, will update the length by checking it, if required.

suryag1201 · 2026-02-23T17:33:44Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+        return cloudStackVolumeRequest;
+    }
+
+    private CloudStackVolume snapshotCloudStackVolumeRequestByProtocol(Map<String, String> details,


this method is not used anywhere

I am calling this method in takeSnapshot()

suryag1201 · 2026-02-23T17:36:29Z

...volume/ontap/src/main/java/org/apache/cloudstack/storage/service/model/CloudStackVolume.java

+     *    a. Field is eligible for unified storage only.
+     *    b. It will be null for the disaggregated storage.
+     */
+    private String flexVolumeUuid;


why do we need these 2 fields here?

If you review the logic, the volume UUID is required to support file‑level snapshot workflows. The destination path will also be useful for future copy operations. In the current implementation, I am using it specifically for the clone API call.

rajiv-jain-netapp

reposting the PR with changes

rajiv-jain-netapp · 2026-02-24T03:57:59Z

...ain/java/org/apache/cloudstack/storage/vmsnapshot/KvmFileBasedStorageVmSnapshotStrategy.java


-        logger.error("Disk-only VM snapshot for VM [{}] failed{}.", userVm.getUuid(), answer != null ? " due to" + answer.getDetails() : "");
-        throw new CloudRuntimeException(String.format("Disk-only VM snapshot for VM [%s] failed.", userVm.getUuid()));
+        String details = answer != null ? answer.getDetails() : String.format("No answer received from host [%s]. The host may be unreachable.", hostId);


I am returning the failure reasons through the exception.

rajiv-jain-netapp · 2026-02-24T03:59:27Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java


    @Override
    public void takeSnapshot(SnapshotInfo snapshot, AsyncCompletionCallback<CreateCmdResult> callback) {
+        s_logger.error("temp takeSnapshot : entered with snapshot id: " + snapshot.getId() + " and name: " + snapshot.getName());


Yes, this was intentionally kept for testing and was mistakenly overlooked during cleanup.

rajiv-jain-netapp · 2026-02-24T04:06:37Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            Map<String, String> poolDetails = storagePoolDetailsDao.listDetailsKeyPairs(volumeVO.getPoolId());
+            StorageStrategy storageStrategy = Utility.getStrategyByStoragePoolDetails(poolDetails);
+
+            Map<String, String> cloudStackVolumeRequestMap = new HashMap<>();


added protocol check.

rajiv-jain-netapp · 2026-02-24T04:06:49Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            cloudStackVolumeRequestMap.put(Constants.VOLUME_UUID, poolDetails.get(Constants.VOLUME_UUID));
+            cloudStackVolumeRequestMap.put(Constants.FILE_PATH, volumeVO.getPath());
+            CloudStackVolume cloudStackVolume = storageStrategy.getCloudStackVolume(cloudStackVolumeRequestMap);
+            if (cloudStackVolume == null || cloudStackVolume.getFile() == null) {


rajiv-jain-netapp · 2026-02-24T04:06:59Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+                throw new CloudRuntimeException("takeSnapshot: Failed to get source file to take snapshot");
+            }
+            long capacityBytes = storagePool.getCapacityBytes();
+            s_logger.error("temp takeSnapshot : entered after getting cloudstack volume with file path: " + cloudStackVolume.getFile().getPath() + " and size: " + cloudStackVolume.getFile().getSize());


rajiv-jain-netapp · 2026-02-24T04:12:14Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            long capacityBytes = storagePool.getCapacityBytes();
+            s_logger.error("temp takeSnapshot : entered after getting cloudstack volume with file path: " + cloudStackVolume.getFile().getPath() + " and size: " + cloudStackVolume.getFile().getSize());
+            long usedBytes = getUsedBytes(storagePool);
+            long fileSize = cloudStackVolume.getFile().getSize();


rajiv-jain-netapp · 2026-02-24T04:13:26Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+
+            String fileSnapshotName = volumeInfo.getName() + "-" + snapshot.getUuid();

+            int maxSnapshotNameLength = 64;


moved it to contants for now, will update the length by checking it, if required.

rajiv-jain-netapp · 2026-02-24T04:15:05Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+        return cloudStackVolumeRequest;
+    }
+
+    private CloudStackVolume snapshotCloudStackVolumeRequestByProtocol(Map<String, String> details,


I am calling this method in takeSnapshot()

rajiv-jain-netapp · 2026-02-24T04:17:21Z

...volume/ontap/src/main/java/org/apache/cloudstack/storage/service/model/CloudStackVolume.java

+     *    a. Field is eligible for unified storage only.
+     *    b. It will be null for the disaggregated storage.
+     */
+    private String flexVolumeUuid;


If you review the logic, the volume UUID is required to support file‑level snapshot workflows. The destination path will also be useful for future copy operations. In the current implementation, I am using it specifically for the clone API call.

piyush5netapp · 2026-02-24T05:02:20Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

-        mapCapabilities.put(DataStoreCapabilities.STORAGE_SYSTEM_SNAPSHOT.toString(), Boolean.FALSE.toString());
-        mapCapabilities.put(DataStoreCapabilities.CAN_CREATE_VOLUME_FROM_SNAPSHOT.toString(), Boolean.FALSE.toString());
+        mapCapabilities.put(DataStoreCapabilities.STORAGE_SYSTEM_SNAPSHOT.toString(), Boolean.TRUE.toString());
+        mapCapabilities.put(DataStoreCapabilities.CAN_CREATE_VOLUME_FROM_SNAPSHOT.toString(), Boolean.TRUE.toString());


Why do we need to set this as true as we are currently not supporting volume creation from snapshot.

Without this property set to true, CloudStack initiates both the snapshot creation and the corresponding copy operation to secondary storage over the data path, as it assumes that snapshots must reside on secondary storage by default.
When only the storage-system-snapshot property is enabled, CloudStack successfully invokes the takeSnapshot() method in the plugin driver class. However, the snapshot remains on the primary storage pool and is not transferred to secondary storage.

So, with this implementation, snapshots reside in primary storage itself?

piyush5netapp · 2026-02-24T05:08:22Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+     * @param storagePoolId: primary storage pool id
+     * @param ontapSnapSize: Size of snapshot CS volume(LUN/file)
+     */
+    private void updateSnapshotDetails(long csSnapshotId, long csVolumeId, String ontapVolumeUuid, String ontapNewSnapshot, long storagePoolId, long ontapSnapSize) {


can we create a map and put whatever we need to at one time and persist in SnapshotDetailsV0 ?

We store these details as key–value pairs in the database. To facilitate efficient querying—particularly during snapshot delete or revert operations—I prefer to keep them independent rather than embedding them within a combined structure.

sandeeplocharla · 2026-02-24T04:48:15Z

...com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCreateDiskOnlyVMSnapshotCommandWrapper.java

        return new Pair<>(snapshotXml, volumeObjectToNewPathMap);
    }

+    protected void cleanupLeftoverDeltas(List<VolumeObjectTO> volumeObjectTos, Map<String, Pair<Long, String>> mapVolumeToSnapshotSizeAndNewVolumePath, KVMStoragePoolManager storagePoolMgr) {


Does this refactoring somehow help our plugin functionality?

sandeeplocharla · 2026-02-24T04:48:59Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

-        mapCapabilities.put(DataStoreCapabilities.STORAGE_SYSTEM_SNAPSHOT.toString(), Boolean.FALSE.toString());
-        mapCapabilities.put(DataStoreCapabilities.CAN_CREATE_VOLUME_FROM_SNAPSHOT.toString(), Boolean.FALSE.toString());
+        mapCapabilities.put(DataStoreCapabilities.STORAGE_SYSTEM_SNAPSHOT.toString(), Boolean.TRUE.toString());
+        mapCapabilities.put(DataStoreCapabilities.CAN_CREATE_VOLUME_FROM_SNAPSHOT.toString(), Boolean.TRUE.toString());


We are supporting volume creation from snapshot with this PR?

sandeeplocharla · 2026-02-24T04:50:17Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java


    @Override
    public void takeSnapshot(SnapshotInfo snapshot, AsyncCompletionCallback<CreateCmdResult> callback) {
+        s_logger.info("takeSnapshot : entered with snapshot id: " + snapshot.getId() + " and name: " + snapshot.getName());


Lets have an intuitive log message instead of a method entry logger?

Could you provide a few examples?
Since I am also logging the method arguments, I felt that this approach would give me better insight into the method invocation flow, rather than relying solely on static logging.

Instead of adding an entry log we can add a log post line 544. For example, it could be like
s_logger.info("takeSnapshot: Snapshot with ID: " + snapshot.getId() + " and name: " + snapshot.getName() + " requested for volume with ID: " + volumeVO.getId() + " and name: " + volumeVO.getName())
It's just an example but I feel this could give us more outlook on whats happening.

sandeeplocharla · 2026-02-24T04:52:41Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            long capacityBytes = storagePool.getCapacityBytes();
+
+            // Only proceed for NFS3 protocol
+            if (ProtocolType.NFS3.name().equalsIgnoreCase(poolDetails.get(Constants.PROTOCOL))) {


Didn't we decide not to have any conditional statements on protocols?

We did, but the current context is not related to the hierarchy of strategy classes. The implementation here is analogous to the request‑preparation helper methods we already maintain within the driver classes.

A part of the discussion was to have nothing related to protocols here, move all the necessary objects to strategy classes and have the necessary implementation there itself, according to the requirement. Will try to pick up the code for iSCSI soon, but, please take the lead in making code changes to align with the discussed approach?

sandeeplocharla · 2026-02-24T04:55:57Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+                cloudStackVolumeRequestMap.put(Constants.FILE_PATH, volumeVO.getPath());
+                cloudStackVolume = storageStrategy.getCloudStackVolume(cloudStackVolumeRequestMap);
+                if (cloudStackVolume == null || cloudStackVolume.getFile() == null) {
+                    throw new CloudRuntimeException("takeSnapshot: Failed to get source file to take snapshot");


Should we have method names only in loggers and not have them in exceptions? Management server might show this as an error on the UI?

acknowledged

sandeeplocharla · 2026-02-24T05:49:23Z

...age/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/UnifiedNASStrategy.java

+        try {
+            fileResponse = nasFeignClient.getFileResponse(authHeader, volumeUuid, filePath);
+            if (fileResponse == null || fileResponse.getRecords().isEmpty()) {
+                throw new CloudRuntimeException("File " + filePath + " not not found on ONTAP. " +


Typo "not not"

sandeeplocharla · 2026-02-24T05:50:50Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+/**
+ * VM Snapshot strategy for NetApp ONTAP managed storage.
+ *
+ * <p>This strategy handles VM-level (instance) snapshots for VMs whose volumes


Do we need comments in HTML format?

We do not have any limitations on these yet, so I kept them. BTW, I got them generated.

sandeeplocharla · 2026-02-24T05:51:09Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+ * <ol>
+ *   <li>Freeze the VM via QEMU guest agent ({@code fsfreeze})</li>
+ *   <li>For each attached volume, create a storage-level snapshot (ONTAP file clone)</li>
+ *   <li>Thaw the VM</li>


what does "Thaw the VM" mean?

sandeeplocharla · 2026-02-24T05:56:20Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+            return false;
+        }
+
+        if (!Hypervisor.HypervisorType.KVM.equals(userVm.getHypervisorType())) {


We already have checks in StoragePool and CS Volume creation workflows for Hypervisor type right? Do we need it here also?

This is inline with the snapshot workflow

sandeeplocharla · 2026-02-24T05:57:48Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+            return false;
+        }
+
+        if (!VirtualMachine.State.Running.equals(userVm.getState())) {


Wouldn't this block disk snapshots?

piyush5netapp · 2026-02-24T06:03:23Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+ *   <li>All VM volumes are on ONTAP managed NFS primary storage</li>
+ * </ul>
+ */
+public class OntapVMSnapshotStrategy extends StorageVMSnapshotStrategy {


I don't see any other vendor extending this class to implement snapshot functionality. What ways this class is helping us ? And is it being called by the framework in snapshot workflow?

I have this implementation for NFS specific workflow requirement. None of the other vendors are supporting NFS workflows via a plugin.
I added this to override the behaviour of quiceing the volume if quiecing is already done from the VM side.

piyush5netapp · 2026-02-24T10:43:24Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+                s_logger.info("takeSnapshot : entered after getting cloudstack volume with file path: " + cloudStackVolume.getFile().getPath() + " and size: " + cloudStackVolume.getFile().getSize());
+                fileSize = cloudStackVolume.getFile().getSize();
+                usedBytes += fileSize;
+            }


snapshot or clone are not of the same size as actual volume(Until spllit clone). So , indirectly we are reducing the available space of storage pool for a clone/snapshot . We can query fr snapshot or clone size and update accordingly.

piyush5netapp · 2026-02-24T10:44:26Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+
+            updateSnapshotDetails(snapshot.getId(), volumeInfo.getId(), poolDetails.get(Constants.VOLUME_UUID), cloneCloudStackVolume.getFile().getPath(), volumeVO.getPoolId(), fileSize);
+
+            snapshotObjectTo.setPath(Constants.ONTAP_SNAP_ID +"="+cloneCloudStackVolume.getFile().getPath());


Is "=" intentional ?

sandeeplocharla · 2026-02-24T11:16:34Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java

+            CloudStackVolume snapCloudStackVolumeRequest = snapshotCloudStackVolumeRequestByProtocol(poolDetails, volumeVO.getPath(), snapshotName);
+            CloudStackVolume cloneCloudStackVolume = storageStrategy.snapshotCloudStackVolume(snapCloudStackVolumeRequest);
+
+            updateSnapshotDetails(snapshot.getId(), volumeInfo.getId(), poolDetails.get(Constants.VOLUME_UUID), cloneCloudStackVolume.getFile().getPath(), volumeVO.getPoolId(), fileSize);


If iSCSI needs different things to be updated in DB, we might get an if-else on protocol here also.

piyush5netapp · 2026-02-24T11:55:50Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+            if (volume.getPoolId() == null) {
+                return false;
+            }
+            StoragePoolVO pool = storagePool.findById(volume.getPoolId());


We can optimize here -> Instead of making findById for each volume, we can use listByUuids by passing all volumes together. This will save much db calls and memory.

piyush5netapp · 2026-02-24T11:59:45Z

...me/ontap/src/main/java/org/apache/cloudstack/storage/vmsnapshot/OntapVMSnapshotStrategy.java

+            boolean quiescevm = true;
+            VMSnapshotOptions options = vmSnapshotVO.getOptions();
+            if (options != null && !options.needQuiesceVM()) {
+                logger.info("Quiesce option was set to false, but overriding to true for ONTAP managed storage " +


If we are planning to only support app consistency, why not throw an error message stating ONTAP recommends quiescing the vm instead of overriding in backend without notifying user.

rajiv-jain-netapp added 3 commits February 19, 2026 19:53

CSTACKEX-18_2: NFS3 snapshot changes

f42552b

CSTACK-18_2: fixing junit dependent changes

8894248

STACK-18_2: fixes

3f0019a

rajiv-jain-netapp added 4 commits February 20, 2026 14:12

CSTACKEX-18_2: adding VM snapshot logic

9b79f46

CSTACKEX-18_2: fix junit issues

7a0d61e

CSTACKEX-18_2: fixes for vm snapshot workflow

7c3419e

CSTACKEX-18_2: fixing the behaviour for the VM level snapshot when qu…

d2b6a27

…iecing is enabled and memory-disk is opted out

suryag1201 reviewed Feb 23, 2026

View reviewed changes

suryag1201 requested changes Feb 23, 2026

View reviewed changes

rajiv-jain-netapp commented Feb 24, 2026

View reviewed changes

rajiv-jain-netapp added 2 commits February 24, 2026 09:49

CSTACKEX-18_2: incorporating the review comments.

c5d5428

CSTACKEX-18_2: transient fixes post incorporating the comments

3f18c11

piyush5netapp reviewed Feb 24, 2026

View reviewed changes

sandeeplocharla reviewed Feb 24, 2026

View reviewed changes

piyush5netapp reviewed Feb 24, 2026

View reviewed changes

CSTACKEX-18_2: Incorporate review comments

723561b

piyush5netapp reviewed Feb 24, 2026

View reviewed changes

sandeeplocharla reviewed Feb 24, 2026

View reviewed changes

piyush5netapp reviewed Feb 24, 2026

View reviewed changes


		String fileSnapshotName = volumeInfo.getName() + "-" + snapshot.getUuid();

		int maxSnapshotNameLength = 64;


		updateSnapshotDetails(snapshot.getId(), volumeInfo.getId(), poolDetails.get(Constants.VOLUME_UUID), cloneCloudStackVolume.getFile().getPath(), volumeVO.getPoolId(), fileSize);

		snapshotObjectTo.setPath(Constants.ONTAP_SNAP_ID +"="+cloneCloudStackVolume.getFile().getPath());

Comments

Conversation

rajiv-jain-netapp commented Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Types of changes

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

Bug Severity

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

Uh oh!

rajiv-jain-netapp commented Feb 20, 2026

Uh oh!

rajiv-jain-netapp commented Feb 21, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rajiv-jain-netapp left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rajiv-jain-netapp commented Feb 20, 2026 •

edited

Loading