Skip to content

Commit 5f38aaa

Browse files
committed
fix build
1 parent 0454e8c commit 5f38aaa

File tree

7 files changed

+9
-14
lines changed

7 files changed

+9
-14
lines changed

api/src/main/java/com/cloud/user/ResourceLimitService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@ public interface ResourceLimitService {
254254
void updateTaggedResourceLimitsAndCountsForAccounts(List<AccountResponse> responses, String tag);
255255
void updateTaggedResourceLimitsAndCountsForDomains(List<DomainResponse> responses, String tag);
256256
void checkVolumeResourceLimit(Account owner, Boolean display, Long size, DiskOffering diskOffering, List<Reserver> reservations) throws ResourceAllocationException;
257-
List<String> getResourceLimitStorageTagsForResourceCountOperation(Boolean display, DiskOffering diskOffering);
257+
List<String> getResourceLimitStorageTagsForResourceCountOperation(Boolean display, DiskOffering diskOffering, Boolean enforceResourceLimitOnDisplayFalse);
258258
void checkVolumeResourceLimitForDiskOfferingChange(Account owner, Boolean display, Long currentSize, Long newSize,
259259
DiskOffering currentOffering, DiskOffering newOffering, List<Reserver> reservations) throws ResourceAllocationException;
260260

server/src/main/java/com/cloud/resourcelimit/ResourceLimitManagerImpl.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1789,8 +1789,8 @@ public void checkVolumeResourceLimitForDiskOfferingChange(Account owner, Boolean
17891789
return;
17901790
}
17911791

1792-
List<String> currentTags = getResourceLimitStorageTagsForResourceCountOperation(true, currentOffering);
1793-
List<String> tagsAfterUpdate = getResourceLimitStorageTagsForResourceCountOperation(true, newOffering);
1792+
List<String> currentTags = getResourceLimitStorageTagsForResourceCountOperation(true, currentOffering, null);
1793+
List<String> tagsAfterUpdate = getResourceLimitStorageTagsForResourceCountOperation(true, newOffering, null);
17941794
if (currentTags.isEmpty() && tagsAfterUpdate.isEmpty()) {
17951795
return;
17961796
}
@@ -2118,8 +2118,8 @@ private void checkVmResourceLimitsForServiceOfferingAndTemplateChange(Account ow
21182118
Long currentMemory, Long newMemory, ServiceOffering currentOffering, ServiceOffering newOffering,
21192119
VirtualMachineTemplate currentTemplate, VirtualMachineTemplate newTemplate, List<Reserver> reservations
21202120
) throws ResourceAllocationException {
2121-
List<String> currentTags = getResourceLimitHostTagsForResourceCountOperation(true, currentOffering, currentTemplate);
2122-
List<String> tagsAfterUpdate = getResourceLimitHostTagsForResourceCountOperation(true, newOffering, newTemplate);
2121+
List<String> currentTags = getResourceLimitHostTagsForResourceCountOperation(true, currentOffering, currentTemplate, null);
2122+
List<String> tagsAfterUpdate = getResourceLimitHostTagsForResourceCountOperation(true, newOffering, newTemplate, null);
21232123
if (currentTags.isEmpty() && tagsAfterUpdate.isEmpty()) {
21242124
return;
21252125
}

server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -964,7 +964,7 @@ public VolumeVO allocVolume(CreateVolumeCmd cmd) throws ResourceAllocationExcept
964964

965965
Storage.ProvisioningType provisioningType = diskOffering.getProvisioningType();
966966

967-
List<String> tags = _resourceLimitMgr.getResourceLimitStorageTagsForResourceCountOperation(displayVolume, diskOffering);
967+
List<String> tags = _resourceLimitMgr.getResourceLimitStorageTagsForResourceCountOperation(displayVolume, diskOffering, null);
968968
if (tags.size() == 1 && tags.get(0) == null) {
969969
tags = new ArrayList<>();
970970
}
@@ -2809,7 +2809,7 @@ public Volume attachVolumeToVM(Long vmId, Long volumeId, Long deviceId, Boolean
28092809
}
28102810

28112811
Account owner = _accountDao.findById(volumeToAttach.getAccountId());
2812-
List<String> resourceLimitStorageTags = _resourceLimitMgr.getResourceLimitStorageTagsForResourceCountOperation(true, diskOffering);
2812+
List<String> resourceLimitStorageTags = _resourceLimitMgr.getResourceLimitStorageTagsForResourceCountOperation(true, diskOffering, null);
28132813
Long requiredPrimaryStorageSpace = getRequiredPrimaryStorageSizeForVolumeAttach(resourceLimitStorageTags, volumeToAttach);
28142814

28152815
try (CheckedReservation primaryStorageReservation = new CheckedReservation(owner, ResourceType.primary_storage, resourceLimitStorageTags, requiredPrimaryStorageSpace, reservationDao, _resourceLimitMgr)) {

server/src/main/java/com/cloud/vm/UserVmManager.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@
4040
import com.cloud.template.VirtualMachineTemplate;
4141
import com.cloud.uservm.UserVm;
4242
import com.cloud.utils.Pair;
43-
import org.jetbrains.annotations.NotNull;
4443

4544
import static com.cloud.user.ResourceLimitService.ResourceLimitHostTags;
4645

server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -183,10 +183,6 @@
183183
import com.cloud.vm.dao.VMInstanceDao;
184184
import com.cloud.vm.dao.VMInstanceDetailsDao;
185185
import com.google.gson.Gson;
186-
import com.google.gson.GsonBuilder;
187-
import com.google.gson.reflect.TypeToken;
188-
import org.apache.commons.lang3.builder.ReflectionToStringBuilder;
189-
import org.apache.commons.lang3.builder.ToStringStyle;
190186

191187
public class BackupManagerImpl extends ManagerBase implements BackupManager {
192188

server/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2773,7 +2773,7 @@ private UserVm importKvmVirtualMachineFromDisk(final ImportSource importSource,
27732773
DiskOfferingVO diskOffering = diskOfferingDao.findById(serviceOffering.getDiskOfferingId());
27742774

27752775
List<Reserver> reservations = new ArrayList<>();
2776-
List<String> resourceLimitStorageTags = resourceLimitService.getResourceLimitStorageTagsForResourceCountOperation(true, diskOffering);
2776+
List<String> resourceLimitStorageTags = resourceLimitService.getResourceLimitStorageTagsForResourceCountOperation(true, diskOffering, null);
27772777
try {
27782778
CheckedReservation volumeReservation = new CheckedReservation(owner, Resource.ResourceType.volume, resourceLimitStorageTags,
27792779
CollectionUtils.isNotEmpty(resourceLimitStorageTags) ? 1L : 0L, reservationDao, resourceLimitService);

server/src/test/java/com/cloud/vpc/MockResourceLimitManagerImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ public void checkVolumeResourceLimit(Account owner, Boolean display, Long size,
279279
}
280280

281281
@Override
282-
public List<String> getResourceLimitStorageTagsForResourceCountOperation(Boolean display, DiskOffering diskOffering) {
282+
public List<String> getResourceLimitStorageTagsForResourceCountOperation(Boolean display, DiskOffering diskOffering, Boolean enforceResourceLimitOnDisplayFalse) {
283283
return null;
284284
}
285285

0 commit comments

Comments
 (0)