Download system template while adding S3 image store is working.

minchen07 · minchen07 · commit f965e2d89201 · 2013-05-09T21:49:12.000-07:00
diff --git a/core/src/com/cloud/storage/resource/LocalNfsSecondaryStorageResource.java b/core/src/com/cloud/storage/resource/LocalNfsSecondaryStorageResource.java
@@ -10,6 +10,7 @@
 import java.net.URISyntaxException;
 import java.net.URL;
 import java.util.List;
+import java.util.concurrent.Executors;
 
 import org.apache.cloudstack.storage.command.DownloadSystemTemplateCommand;
 import org.springframework.stereotype.Component;
@@ -34,6 +35,7 @@ public class LocalNfsSecondaryStorageResource extends
 
     public LocalNfsSecondaryStorageResource(){
         this._dlMgr = new DownloadManagerImpl();
+        ((DownloadManagerImpl)_dlMgr).setThreadPool(Executors.newFixedThreadPool(10));
     }
 
     @Override
diff --git a/core/src/com/cloud/storage/template/S3TemplateDownloader.java b/core/src/com/cloud/storage/template/S3TemplateDownloader.java
@@ -279,9 +279,9 @@ public long download(boolean resume, DownloadCompleteCallback callback) {
                         @Override
                         public void progressChanged(
                                 ProgressEvent progressEvent) {
-                            s_logger.info(progressEvent.getBytesTransfered()
-                                    + " number of byte transferd "
-                                    + new Date());
+                           // s_logger.debug(progressEvent.getBytesTransfered()
+                           //         + " number of byte transferd "
+                           //         + new Date());
                             totalBytes += progressEvent.getBytesTransfered();
                             if (progressEvent.getEventCode() == ProgressEvent.COMPLETED_EVENT_CODE) {
                                 s_logger.info("download completed");
diff --git a/engine/storage/image/src/org/apache/cloudstack/storage/image/TemplateServiceImpl.java b/engine/storage/image/src/org/apache/cloudstack/storage/image/TemplateServiceImpl.java
@@ -477,7 +477,9 @@ protected Void createTemplateCallback(AsyncCallbackDispatcher<TemplateServiceImp
             return null;
         }
 
-        parentCallback.complete(result);
+        if (parentCallback != null){
+            parentCallback.complete(result);
+        }
         return null;
     }
 
diff --git a/setup/db/db/schema-410to420.sql b/setup/db/db/schema-410to420.sql
@@ -105,6 +105,7 @@ CREATE VIEW `cloud`.`image_store_view` AS
         image_store.protocol,
         image_store.url,
         image_store.scope,
+        image_store.role,
         data_center.id data_center_id,
         data_center.uuid data_center_uuid,
         data_center.name data_center_name,
@@ -115,8 +116,8 @@ CREATE VIEW `cloud`.`image_store_view` AS
             left join
         `cloud`.`data_center` ON image_store.data_center_id = data_center.id
             left join
-        `cloud`.`image_store_details` ON image_store_details.store_id = image_store.id
-    where image_store.role = 'Image';
+        `cloud`.`image_store_details` ON image_store_details.store_id = image_store.id;
+
             
 -- here we have to allow null for store_id to accomodate baremetal case to search for ready templates since template state is only stored in this table
 -- FK also commented out due to this            
@@ -732,3 +733,6 @@ CREATE VIEW `cloud`.`volume_view` AS
         `cloud`.`async_job` ON async_job.instance_id = volumes.id
             and async_job.instance_type = 'Volume'
             and async_job.job_status = 0;     
+            
+ update  `cloud`.`vm_template` set state='Allocated' where state is NULL;
+ update  `cloud`.`vm_template` set update_count=0 where update_count is NULL;
diff --git a/tools/devcloud/devcloud_s3.cfg b/tools/devcloud/devcloud_s3.cfg
@@ -91,7 +91,7 @@
                         },
                         {
                             "key" : "secretkey",
-                            "value" : "OYAZXCAFUC1DAFOXNJWI"
+                            "value" : "YHpRtzNDwl12DtrQmwRvdpnf2xK2AeVM30rXxQ=="
                         },
                         {
                             "key" : "endpoint",

Original file line number	Diff line number	Diff line change
`@@ -477,7 +477,9 @@ protected Void createTemplateCallback(AsyncCallbackDispatcher<TemplateServiceImp`
`477`	`477`	`return null;`
`478`	`478`	`}`
`479`	`479`
`480`		`- parentCallback.complete(result);`
	`480`	`+ if (parentCallback != null){`
	`481`	`+ parentCallback.complete(result);`
	`482`	`+ }`
`481`	`483`	`return null;`
`482`	`484`	`}`
`483`	`485`
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@`
`91`	`91`	`},`
`92`	`92`	`{`
`93`	`93`	`"key" : "secretkey",`
`94`		`- "value" : "OYAZXCAFUC1DAFOXNJWI"`
	`94`	`+ "value" : "YHpRtzNDwl12DtrQmwRvdpnf2xK2AeVM30rXxQ=="`
`95`	`95`	`},`
`96`	`96`	`{`
`97`	`97`	`"key" : "endpoint",`