GabrielBrascher commented on a change in pull request #2500: Restrict the
number of managed clustered file systems per compute cluster
URL: https://github.com/apache/cloudstack/pull/2500#discussion_r215785244
##########
File path:
engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java
##########
@@ -55,41 +54,30 @@
public abstract class AbstractStoragePoolAllocator extends AdapterBase
implements StoragePoolAllocator {
private static final Logger s_logger =
Logger.getLogger(AbstractStoragePoolAllocator.class);
- @Inject
- StorageManager storageMgr;
- protected @Inject
- PrimaryDataStoreDao _storagePoolDao;
- @Inject
- VolumeDao _volumeDao;
- @Inject
- ConfigurationDao _configDao;
- @Inject
- ClusterDao _clusterDao;
- protected @Inject
- DataStoreManager dataStoreMgr;
- protected BigDecimal _storageOverprovisioningFactor = new BigDecimal(1);
- long _extraBytesPerVolume = 0;
- Random _rand;
- boolean _dontMatter;
- protected String _allocationAlgorithm = "random";
- @Inject
- DiskOfferingDao _diskOfferingDao;
- @Inject
- CapacityDao _capacityDao;
+
+ protected BigDecimal storageOverprovisioningFactor = new BigDecimal(1);
+ protected String allocationAlgorithm = "random";
+ protected @Inject DataStoreManager dataStoreMgr;
+ @Inject PrimaryDataStoreDao storagePoolDao;
+ @Inject VolumeDao volumeDao;
+ @Inject ConfigurationDao configDao;
+ long extraBytesPerVolume = 0;
+ @Inject private CapacityDao capacityDao;
+ @Inject private ClusterDao clusterDao;
+ @Inject private StorageManager storageMgr;
+ @Inject private StorageUtil storageUtil;
@Override
public boolean configure(String name, Map<String, Object> params) throws
ConfigurationException {
super.configure(name, params);
- if(_configDao != null) {
- Map<String, String> configs = _configDao.getConfiguration(null,
params);
+ if(configDao != null) {
Review comment:
Thanks for keeping [Java variables naming
conventions](https://docs.oracle.com/javase/tutorial/java/nutsandbolts/variables.html)
:+1:
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
