Multi-Host support for NFS3 and iSCSI by sandeeplocharla · Pull Request #64 · NetApp/cloudstack

sandeeplocharla · 2026-06-18T09:53:45Z

Description

This PR...
Provides support to Multi-Host for NFS3 and iSCSI

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?

Only 1 Host present in the Cluster

Created an NFS3 StoragePool with Cluster scope

ONTAP volume created for the SP

Export Policy rule and client

Added a new host

Updated client in the Export Policy

Removed the host from the cluster

The respective client has been removed from the Export Policy

Test Done	Result	Comments
Create an instance by specifying the host and disk offering pointed to NFS3 Primary Storage Pool (Cluster scoped with min of 2 hosts)	PASS
Create an instance specifying host and disk offering pointed to iSCSI Primary Storage Pool (Cluster scoped with min of 2 hosts)	PASS
Create an instance in a cluster with atleast 2 hosts in the cluster by not specifying the host and disk offering pointed to NFS3 Primary Storage Pool (Cluster scoped with min of 2 hosts)	PASS
Create an instance in a cluster with atleast 2 hosts in the cluster by not specifying the host and disk offering pointed to iSCSI Primary Storage Pool (Cluster scoped with min of 2 hosts)	PASS
Create an instance by specifying the host and disk offering pointed to 2 NFS3 Primary Storage Pools (Cluster scoped + Zone scoped)	FAIL	StoragePoolAllocator is coming as empty and the random strategy to allocate was failing.
Create an instance by specifying the host and disk offering pointed to 2 iSCSI Primary Storage Pools (Cluster scoped + Zone scoped)	FAIL	StoragePoolAllocator is coming as empty and the random strategy to allocate was failing.
Create an instance by specifying the host and disk offering pointed to NFS3 Primary Storage Pool (Zone scoped with min of 2 hosts)	FAIL	Zone scoped instance creation failing (CSTACKEX-188)
Create an instance specifying host and disk offering pointed to iSCSI Primary Storage Pool (Zone scoped with min of 2 hosts)	FAIL	Zone scoped instance creation failing (CSTACKEX-188)
Create an instance in a cluster with atleast 2 hosts in the cluster by not specifying the host and disk offering pointed to NFS3 Primary Storage Pool (Zone scoped with min of 2 hosts)	FAIL	Zone scoped instance creation failing (CSTACKEX-188)
Create an instance in a cluster with atleast 2 hosts in the cluster by not specifying the host and disk offering pointed to iSCSI Primary Storage Pool (Zone scoped with min of 2 hosts)	FAIL	Zone scoped instance creation failing (CSTACKEX-188)
Power off VM on Host-1 and start it on Host-2 (same cluster) with disk offering pointed to NFS3 primary storage pool (cluster scoped)	PASS
Power off and on a VM with 'last known host' selected. Create the instance in a cluster, without specifying a host.	PASS	Last known host was selected
Power off and on a VM with 'last known host' NOT selected. Create the instance in a cluster, without specifying a host.	PASS	VM got hosted on the chosen host.
Power off and on a VM with 'last known host' selected but choose a different host. Create the instance in a cluster, without specifying a host.	PASS	VM got hosted on the chosen host.
Insufficient resources in the storage pool. Instance creation should fail.	PASS	Generic error is being displayed instead of proper error regarding Insufficient resources.
One host in the cluster (min. 2 hosts) loaded, resulting in insufficient CPU. Disk Offering pointed to Cluster scoped primary storage pool.	PASS
Both host in the cluster (min. 2 hosts) loaded, resulting in insufficient CPU. Disk Offering pointed to Cluster scoped primary storage pool. Instance creation should fail but disks should first get created and then destroyed.

github-actions · 2026-06-23T12:10:10Z

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

Copilot

Pull request overview

Adds multi-host lifecycle handling for the ONTAP volume plugin by updating NFS export policy rules as hosts connect to / are removed from storage pools, and by extending the AccessGroup model/strategy API to support add/remove semantics.

Changes:

Implemented UnifiedNASStrategy.updateAccessGroup to add/remove host client matches in an existing NFS export policy.
Added HostRuleAction to AccessGroup and wired host-connect / host-removal flows to update NFS3 export policy rules.
Tightened/adjusted model and API surface (e.g., StorageStrategy.updateAccessGroup visibility; ExportRule.ProtocolsEnum JSON annotations).

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
plugins/storage/volume/ontap/src/test/java/org/apache/cloudstack/storage/service/StorageStrategyTest.java	Updates test scaffolding to match new abstract method visibility; import/method formatting adjustments.
plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/UnifiedNASStrategy.java	Implements export-policy mutation logic for NFS host add/remove (multi-host enablement).
plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/StorageStrategy.java	Makes `updateAccessGroup` explicitly `public abstract` for consistent overriding/call sites.
plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/service/model/AccessGroup.java	Adds `HostRuleAction` (ADD/REMOVE) to drive update semantics.
plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/listener/OntapHostListener.java	Updates host lifecycle handling to update NFS3 export policy rules on connect and before removal.
plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/feign/model/ExportRule.java	Adjusts enum JSON serialization/deserialization behavior for export rule protocols.
plugins/storage/volume/ontap/src/main/java/org/apache/cloudstack/storage/driver/OntapPrimaryDatastoreDriver.java	Import reordering / formatting-only change.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

github-actions · 2026-07-10T04:59:07Z

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

Copilot

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated 4 comments.

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

rajiv-jain-netapp · 2026-07-13T14:47:25Z

    }

+    @Override
+    public boolean hostAboutToBeRemoved(long hostId) {


Thinking aloud, do we not have any usecase where this method returns false?

If export policy rule hasn't been removed, we could catch the exception and return false

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Copilot

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

rajiv-jain-netapp

In the testing section, I need results for

User created an NFS pool for cluster scope; validate the export policy on the ONTAP for the rules and client added.
Now add a new host to this cluster and validate whether the export policy is getting updated for this new host or not.
Similar to #2, test for host removal as well.

Note: I noticed the term "Live migrate" in your current test case capture. Just to remind you, we do not support live migration yet; these details will lead to a wrong interpretation in audiences. Please correct them accordingly.

sandeeplocharla · 2026-07-14T04:16:15Z

In the testing section, I need results for

User created an NFS pool for cluster scope; validate the export policy on the ONTAP for the rules and client added.

Now add a new host to this cluster and validate whether the export policy is getting updated for this new host or not.

Similar to Feignconfiguration and volume feignClient along with desired POJOs #2, test for host removal as well.

Note: I noticed the term "Live migrate" in your current test case capture. Just to remind you, we do not support live migration yet; these details will lead to a wrong interpretation in audiences. Please correct them accordingly.

Updated the Testing done with the relevant screenshots.
Also, the test section mentions "Live migrate a VM between two hosts in the same cluster". As per my understanding, we currently don't support VM's storage migration i.e., between 2 storage pools, which the test case is not claiming to be. If the language could be made better, please suggest an alternative, will update it accordingly.

suryag1201

Please address the comments

sandeeplocharla requested review from piyush5netapp, rajiv-jain-netapp and suryag1201 as code owners June 18, 2026 09:53

sandeeplocharla self-assigned this Jun 18, 2026

sandeeplocharla requested review from piyush5netapp, rajiv-jain-netapp and suryag1201 and removed request for piyush5netapp, rajiv-jain-netapp and suryag1201 June 18, 2026 09:54

rajiv-jain-netapp reviewed Jun 22, 2026

View reviewed changes

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

rajiv-jain-netapp reviewed Jun 22, 2026

View reviewed changes

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

github-actions Bot added the status:has-conflicts label Jun 23, 2026

sandeeplocharla force-pushed the feature/CSTACKEX-192 branch from 45b634f to 8323d52 Compare June 29, 2026 17:25

github-actions Bot removed the status:has-conflicts label Jun 29, 2026

sandeeplocharla force-pushed the feature/CSTACKEX-192 branch from 8323d52 to 1f0e3c1 Compare June 30, 2026 07:40

rajiv-jain-netapp reviewed Jul 1, 2026

View reviewed changes

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

Comment thread ...age/volume/ontap/src/main/java/org/apache/cloudstack/storage/listener/OntapHostListener.java Outdated

Copilot AI review requested due to automatic review settings July 8, 2026 05:33

sandeeplocharla force-pushed the feature/CSTACKEX-192 branch from 1f0e3c1 to 8a26a59 Compare July 8, 2026 05:33

Copilot started reviewing on behalf of sandeeplocharla July 8, 2026 05:33 View session

Copilot AI reviewed Jul 8, 2026

View reviewed changes

github-actions Bot added the status:has-conflicts label Jul 10, 2026

Copilot AI review requested due to automatic review settings July 10, 2026 06:13

sandeeplocharla force-pushed the feature/CSTACKEX-192 branch from 8a26a59 to f8a887c Compare July 10, 2026 06:13

github-actions Bot removed the status:has-conflicts label Jul 10, 2026

Copilot started reviewing on behalf of sandeeplocharla July 10, 2026 06:13 View session

Copilot AI reviewed Jul 10, 2026

View reviewed changes

Copilot AI review requested due to automatic review settings July 10, 2026 06:40

sandeeplocharla added 6 commits July 13, 2026 15:36

Addressed comments

891d1ea

Addressed comments

d7651f3

Revert unwanted changes in file

47868b9

Revert unwanted changes in file

30412d4

Fixed review comments

26892e9

Fixed review comments

d1ed46a

Copilot AI review requested due to automatic review settings July 13, 2026 10:06

sandeeplocharla force-pushed the feature/CSTACKEX-192 branch from 8179fdb to d1ed46a Compare July 13, 2026 10:07

Copilot started reviewing on behalf of sandeeplocharla July 13, 2026 10:07 View session

Fixed more review comments

8df4bc9

Copilot AI reviewed Jul 13, 2026

View reviewed changes

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

Copilot AI review requested due to automatic review settings July 13, 2026 10:09

Copilot started reviewing on behalf of sandeeplocharla July 13, 2026 10:10 View session

Copilot AI reviewed Jul 13, 2026

View reviewed changes

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

Comment thread ...volume/ontap/src/test/java/org/apache/cloudstack/storage/service/UnifiedNASStrategyTest.java

rajiv-jain-netapp reviewed Jul 13, 2026

View reviewed changes

Fixed few more review comments

76bdf24

Copilot AI review requested due to automatic review settings July 13, 2026 16:57

Copilot started reviewing on behalf of sandeeplocharla July 13, 2026 16:58 View session

Copilot AI reviewed Jul 13, 2026

View reviewed changes

Removed a UT which no longer is necessary

6af0e37

Copilot AI review requested due to automatic review settings July 14, 2026 02:59

Copilot started reviewing on behalf of sandeeplocharla July 14, 2026 03:00 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

rajiv-jain-netapp reviewed Jul 14, 2026

View reviewed changes

rajiv-jain-netapp approved these changes Jul 16, 2026

View reviewed changes

suryag1201 reviewed Jul 21, 2026

View reviewed changes

Comment thread ...age/volume/ontap/src/main/java/org/apache/cloudstack/storage/listener/OntapHostListener.java

Comment thread ...age/volume/ontap/src/main/java/org/apache/cloudstack/storage/listener/OntapHostListener.java

piyush5netapp approved these changes Jul 22, 2026

View reviewed changes

Comment thread ...age/volume/ontap/src/main/java/org/apache/cloudstack/storage/listener/OntapHostListener.java

Comment thread ...age/volume/ontap/src/main/java/org/apache/cloudstack/storage/listener/OntapHostListener.java

suryag1201 approved these changes Jul 23, 2026

View reviewed changes

suryag1201 reviewed Jul 23, 2026

View reviewed changes

Uh oh!

Conversation

sandeeplocharla commented Jun 18, 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?

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jun 23, 2026

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Jul 10, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rajiv-jain-netapp Jul 13, 2026

Choose a reason for hiding this comment

Uh oh!

sandeeplocharla Jul 13, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

rajiv-jain-netapp left a comment

Choose a reason for hiding this comment

Uh oh!

sandeeplocharla commented Jun 18, 2026 •

edited

Loading