mirror of
https://github.com/apache/cloudstack.git
synced 2025-10-26 08:42:29 +01:00
CLOUDSTACK-5853 cannot deploy vm with differing service storage tag and data disk storage tag
Changes: - Reverting Marcus's fix since this issue has already fixed by https://issues.apache.org/jira/browse/CLOUDSTACK-5426
This commit is contained in:
parent
7f54894045
commit
968ca060ee
@ -176,11 +176,6 @@ public interface DeploymentPlanner extends Adapter {
|
||||
_poolIds.add(poolId);
|
||||
}
|
||||
|
||||
public void removePool(long poolId) {
|
||||
if (_poolIds != null) {
|
||||
_poolIds.remove(poolId);
|
||||
}
|
||||
}
|
||||
|
||||
public void addDataCenter(long dataCenterId) {
|
||||
if (_dcIds == null) {
|
||||
|
||||
@ -86,12 +86,6 @@ public class ClusterScopeStoragePoolAllocator extends AbstractStoragePoolAllocat
|
||||
avoid.addPool(pool.getId());
|
||||
}
|
||||
|
||||
// make sure our matching pool was not in avoid set
|
||||
for (StoragePoolVO pool : pools) {
|
||||
s_logger.debug("Removing pool " + pool + " from avoid set, must have been inserted when searching for another disk's tag");
|
||||
avoid.removePool(pool.getId());
|
||||
}
|
||||
|
||||
if (pools.size() == 0) {
|
||||
if (s_logger.isDebugEnabled()) {
|
||||
s_logger.debug("No storage pools available for " + ServiceOffering.StorageType.shared.toString() + " volume allocation, returning");
|
||||
|
||||
@ -80,11 +80,6 @@ public class ZoneWideStoragePoolAllocator extends AbstractStoragePoolAllocator {
|
||||
avoid.addPool(pool.getId());
|
||||
}
|
||||
|
||||
// make sure our matching pool was not in avoid set
|
||||
for (StoragePoolVO pool : storagePoolsByHypervisor) {
|
||||
s_logger.debug("Removing pool " + pool + " from avoid set, must have been inserted when searching for another disk's tag");
|
||||
avoid.removePool(pool.getId());
|
||||
}
|
||||
|
||||
for (StoragePoolVO storage : storagePools) {
|
||||
if (suitablePools.size() == returnUpTo) {
|
||||
|
||||
@ -120,7 +120,6 @@ import com.cloud.utils.db.TransactionCallback;
|
||||
import com.cloud.utils.db.TransactionStatus;
|
||||
import com.cloud.utils.exception.CloudRuntimeException;
|
||||
import com.cloud.utils.fsm.StateListener;
|
||||
import com.cloud.utils.fsm.StateMachine2;
|
||||
import com.cloud.vm.DiskProfile;
|
||||
import com.cloud.vm.VMInstanceVO;
|
||||
import com.cloud.vm.VirtualMachine;
|
||||
|
||||
@ -86,9 +86,6 @@ import org.apache.cloudstack.storage.command.DeleteCommand;
|
||||
import org.apache.cloudstack.storage.command.DettachCommand;
|
||||
import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
|
||||
import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
|
||||
import org.apache.cloudstack.storage.datastore.db.TemplateDataStoreDao;
|
||||
import org.apache.cloudstack.storage.datastore.db.TemplateDataStoreVO;
|
||||
|
||||
import com.cloud.agent.AgentManager;
|
||||
import com.cloud.agent.api.Answer;
|
||||
import com.cloud.agent.api.Command;
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user