[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-22 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r429326862 ## File path: services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java ## @@ -1347

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-22 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r429326862 ## File path: services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java ## @@ -1347

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-20 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427814248 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/StorageOrchestrator.java ## @@ -0,0 +1,629 @@ +// Lic

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-20 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427807563 ## File path: server/src/main/java/com/cloud/configuration/Config.java ## @@ -16,6 +16,15 @@ // under the License. package com.cloud.configurati

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-20 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427804902 ## File path: engine/storage/src/main/java/org/apache/cloudstack/storage/datastore/ObjectInDataStoreManagerImpl.java ## @@ -104,6 +104,14 @@ publ

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-19 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r427257039 ## File path: engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/DataMigrationUtility.java ## @@ -0,0 +1,262 @@ +// Li

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-18 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r426669651 ## File path: server/src/main/java/com/cloud/storage/secondary/SecondaryStorageVmManager.java ## @@ -31,6 +32,7 @@ public static final int D

[GitHub] [cloudstack] DaanHoogland commented on a change in pull request #4053: Secondary Storage Usage Improvements

2020-05-08 Thread GitBox
DaanHoogland commented on a change in pull request #4053: URL: https://github.com/apache/cloudstack/pull/4053#discussion_r422041219 ## File path: api/src/main/java/org/apache/cloudstack/api/ApiConstants.java ## @@ -777,6 +780,7 @@ public static final String EXITCODE = "exi