ravening commented on a change in pull request #4078:
URL: https://github.com/apache/cloudstack/pull/4078#discussion_r525988804



##########
File path: 
services/secondary-storage/controller/src/main/java/org/apache/cloudstack/secondarystorage/SecondaryStorageManagerImpl.java
##########
@@ -744,9 +743,6 @@ private void allocCapacity(long dataCenterId, 
SecondaryStorageVm.Role role) {
                 if (_allocLock.lock(ACQUIRE_GLOBAL_LOCK_TIMEOUT_FOR_SYNC)) {
                     try {
                         secStorageVm = startNew(dataCenterId, role);
-                        for (UploadVO upload : _uploadDao.listAll()) {

Review comment:
       @rhtyd @DaanHoogland I tested by uploading volumes from local as well as 
from url but `upload` tables were still empty. So looks like uploaded 
volume/template entries are not stored in this table.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to