Skip to content

Commit ba85b4d

Browse files
committed
Merge release branch 4.18 to 4.19
* 4.18: linstor: deleteAsync fallback to volume UUID if path not set yet (#9325) linstor: do not use the same static ApiClient for all calls (#9326)
2 parents 374ea7d + eb1a0ef commit ba85b4d

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

plugins/storage/volume/linstor/src/main/java/org/apache/cloudstack/storage/datastore/driver/LinstorPrimaryDataStoreDriverImpl.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,9 @@ public void deleteAsync(DataStore dataStore, DataObject dataObject, AsyncComplet
271271
case VOLUME:
272272
{
273273
final VolumeInfo volumeInfo = (VolumeInfo) dataObject;
274-
final String rscName = LinstorUtil.RSC_PREFIX + volumeInfo.getPath();
274+
// if volume creation wasn't completely done .setPath wasn't called, so we fallback to vol.getUuid()
275+
final String volUuid = volumeInfo.getPath() != null ? volumeInfo.getPath() : volumeInfo.getUuid();
276+
final String rscName = LinstorUtil.RSC_PREFIX + volUuid;
275277
deleteResourceDefinition(storagePool, rscName);
276278

277279
long usedBytes = storagePool.getUsedBytes();

plugins/storage/volume/linstor/src/main/java/org/apache/cloudstack/storage/datastore/util/LinstorUtil.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818

1919
import com.linbit.linstor.api.ApiClient;
2020
import com.linbit.linstor.api.ApiException;
21-
import com.linbit.linstor.api.Configuration;
2221
import com.linbit.linstor.api.DevelopersApi;
2322
import com.linbit.linstor.api.model.ApiCallRc;
2423
import com.linbit.linstor.api.model.ApiCallRcList;
@@ -52,7 +51,7 @@ public class LinstorUtil {
5251
public static final String CLUSTER_DEFAULT_MAX_IOPS = "clusterDefaultMaxIops";
5352

5453
public static DevelopersApi getLinstorAPI(String linstorUrl) {
55-
ApiClient client = Configuration.getDefaultApiClient();
54+
ApiClient client = new ApiClient();
5655
client.setBasePath(linstorUrl);
5756
return new DevelopersApi(client);
5857
}

0 commit comments

Comments
 (0)