Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-12 Thread via GitHub
github-actions[bot] commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2875036559 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-12 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2874822413 [SF] Trillian test result (tid-13274) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 56331 seconds Marvin logs: https://github.co

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-12 Thread via GitHub
github-actions[bot] commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2873091485 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-12 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2871805991 @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-12 Thread via GitHub
DaanHoogland commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2871802749 @blueorangutan test -- 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

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-12 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2871186940 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13338 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-11 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2870960850 @rohityadavcloud a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make pro

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-11 Thread via GitHub
rohityadavcloud commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2870957552 @blueorangutan package -- 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 spe

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-05-11 Thread via GitHub
rohityadavcloud closed pull request #10696: Introduce volume allocation algorithm global configuration URL: https://github.com/apache/cloudstack/pull/10696 -- 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

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-25 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2830243078 > LGTM. Tested with multiple shared NFS primary storage. If the logging for the firstfitleastconsumed can print the available capacity as well in the logs it will be useful. >

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-22 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2823147549 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13137 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-22 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2823019435 @sudo87 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-22 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2823017786 @blueorangutan package -- 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 com

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-22 Thread via GitHub
github-actions[bot] commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2820674075 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-17 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2811778150 @sudo87 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-17 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2814226933 [SF] Trillian test result (tid-13028) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 56146 seconds Marvin logs: https://github.co

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-17 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2812065082 @sudo87 a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-17 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2812060023 @blueorangutan test -- 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 commen

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-17 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2812010455 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13089 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-16 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2811776171 @blueorangutan package -- 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 com

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-16 Thread via GitHub
weizhouapache commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2810163435 "random" can be replaced by VolumeAllocationAlgorithm.getDefaultValue() -- This is an automated message from the Apache Git Service. To respond to the message

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-16 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2809289459 > code lgtm > > However, I think we need to think of the upgrade for example, now the vm/volume allocation algorithm is set to `firstfit`, when uprgade to a new version with th

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-15 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2807253806 [SF] Trillian test result (tid-12988) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 54138 seconds Marvin logs: https://github.co

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-15 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2803688965 @sudo87 a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-15 Thread via GitHub
weizhouapache commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2805004962 code lgtm However, I think we need to think of the upgrade for example, now the vm/volume allocation algorithm is set to `firstfit`, when uprgade to a new version with

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2803687730 @blueorangutan test -- 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 commen

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2041685566 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -209,7 +204,7 @@ public List reorderPools(List poo

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2801507426 @sudo87 a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2801747060 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13046 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
sudo87 commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2801503033 @blueorangutan package -- 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 com

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2041974395 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -209,7 +204,7 @@ public List reorderPools(List poo

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2041685566 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -209,7 +204,7 @@ public List reorderPools(List poo

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
DaanHoogland commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2041667109 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -209,7 +204,7 @@ public List reorderPools(Lis

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-14 Thread via GitHub
weizhouapache commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2041778685 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -209,7 +204,7 @@ public List reorderPools(Li

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039462659 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanner {

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
weizhouapache commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039365227 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanne

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2796566406 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13020 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039241348 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanner {

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039297583 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanner {

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039241348 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanner {

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039241348 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanner {

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
sudo87 commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039241348 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanner {

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
weizhouapache commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039239311 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends DeploymentPlanne

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
harikrishna-patnala commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2039223750 ## api/src/main/java/com/cloud/deploy/DeploymentClusterPlanner.java: ## @@ -62,7 +62,7 @@ public interface DeploymentClusterPlanner extends Deployment

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2796393089 @DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progre

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
DaanHoogland commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2796389142 @blueorangutan package -- 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 specif

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
DaanHoogland commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2038955053 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -16,53 +16,51 @@ // under the License. pack

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-11 Thread via GitHub
DaanHoogland commented on code in PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#discussion_r2038954734 ## engine/storage/src/main/java/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java: ## @@ -16,53 +16,51 @@ // under the License. pack

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-10 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2795983072 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13019 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-10 Thread via GitHub
blueorangutan commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2795876357 @rajujith a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-10 Thread via GitHub
rajujith commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2795875425 @blueorangutan package -- 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 c

Re: [PR] Introduce volume allocation algorithm global configuration [cloudstack]

2025-04-10 Thread via GitHub
codecov[bot] commented on PR #10696: URL: https://github.com/apache/cloudstack/pull/10696#issuecomment-2795870907 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10696?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comme