You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by "Ragi, Sathvika via dev" <de...@cloudstack.apache.org> on 2026/08/07 17:46:06 UTC

In updatestoragepool API the provider's lifecycle method call is missing

Hello Team,

While implementing storage pool resize via the updateStoragePool API, we noticed that StorageManagerImpl.updateStoragePool()updates the database but never calls PrimaryDataStoreLifeCycle.updateStoragePool() on the storage plugin.

The interface method exists, and plugins already implement it , But the call is absent in StorageManagerImpl, making all those implementations unreachable via this API.

The call was removed in this PR: FiberChannel Multipath for KVM + Pure Flash Array and HPE-Primera Support by rg9975 · Pull Request #…<https://github.com/apache/cloudstack/pull/7889/changes#diff-cf68e132ad7711cfa9250cda5819c09f51cba9933aa595cd82ad5a02b4edbe64>

 we wanted to check if it was intentional, was the lifecycle call deliberately left out, or is this an oversight?

Regards,
sathvika ragi

Get Outlook for Mac<https://aka.ms/GetOutlookForMac>

Re: In updatestoragepool API the provider's lifecycle method call is missing

Posted by Wei ZHOU <us...@gmail.com>.
I think the call might be removed by mistake.


-Wei

On Friday, August 7, 2026, Ragi, Sathvika via dev <de...@cloudstack.apache.org>
wrote:

> Hello Team,
>
> While implementing storage pool resize via the updateStoragePool API, we
> noticed that StorageManagerImpl.updateStoragePool()updates the database
> but never calls PrimaryDataStoreLifeCycle.updateStoragePool() on the
> storage plugin.
>
> The interface method exists, and plugins already implement it , But the
> call is absent in StorageManagerImpl, making all those implementations
> unreachable via this API.
>
> The call was removed in this PR: FiberChannel Multipath for KVM + Pure
> Flash Array and HPE-Primera Support by rg9975 · Pull Request #…<
> https://github.com/apache/cloudstack/pull/7889/changes#diff-
> cf68e132ad7711cfa9250cda5819c09f51cba9933aa595cd82ad5a02b4edbe64>
>
>  we wanted to check if it was intentional, was the lifecycle call
> deliberately left out, or is this an oversight?
>
> Regards,
> sathvika ragi
>
> Get Outlook for Mac<https://aka.ms/GetOutlookForMac>
>