Re: [PR] VMware: Improve error messaging / logs when starting non-user VMs, and secondary storage not available or doesn't have enough capacity [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on code in PR #9207: URL: https://github.com/apache/cloudstack/pull/9207#discussion_r1635971625 ## plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java: ## @@ -599,13 +598,13 @@ public List>

Re: [PR] engine-schema: fix options for host.allocators.order config [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #8733: URL: https://github.com/apache/cloudstack/pull/8733#issuecomment-2162312414 @shwstppr 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] engine-schema: fix options for host.allocators.order config [cloudstack]

2024-06-12 Thread via GitHub
shwstppr commented on PR #8733: URL: https://github.com/apache/cloudstack/pull/8733#issuecomment-2162310414 @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

Re: [PR] api,server: allow updating hypervisor capabilities with hypervisor and version [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #8475: URL: https://github.com/apache/cloudstack/pull/8475#issuecomment-2162309238 @shwstppr 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] prevent duplicate ip table rules in SSVM [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#issuecomment-2162305739 @weizhouapache 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

Re: [PR] api,server: allow updating hypervisor capabilities with hypervisor and version [cloudstack]

2024-06-12 Thread via GitHub
shwstppr commented on PR #8475: URL: https://github.com/apache/cloudstack/pull/8475#issuecomment-2162305798 @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

Re: [I] Event state is incorrect when vpc network is created [cloudstack]

2024-06-12 Thread via GitHub
kiranchavala closed issue #8496: Event state is incorrect when vpc network is created URL: https://github.com/apache/cloudstack/issues/8496 -- 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

Re: [I] Event state is incorrect when vpc network is created [cloudstack]

2024-06-12 Thread via GitHub
kiranchavala commented on issue #8496: URL: https://github.com/apache/cloudstack/issues/8496#issuecomment-2162303597 Closing the issue as the PR is merged -- 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

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-06-12 Thread via GitHub
weizhouapache commented on PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#issuecomment-2162301703 @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

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-06-12 Thread via GitHub
weizhouapache commented on code in PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#discussion_r1635956238 ## services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/resource/IpTablesHelper.java: ## @@ -0,0 +1,66 @@ +// +// Licensed to the

Re: [PR] vpc,event: fix events for createVpc [cloudstack]

2024-06-12 Thread via GitHub
kiranchavala merged PR #9055: URL: https://github.com/apache/cloudstack/pull/9055 -- 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. To unsubscribe, e-mail:

Re: [PR] Fixup startVM on simulator [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland closed pull request #9199: Fixup startVM on simulator URL: https://github.com/apache/cloudstack/pull/9199 -- 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. To

(cloudstack) branch 4.19 updated (edf7394919f -> ca597200156)

2024-06-12 Thread kiranchavala
This is an automated email from the ASF dual-hosted git repository. kiranchavala pushed a change to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from edf7394919f server: search template name and display text if keyword is passed (#9215) add ca597200156

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on code in PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#discussion_r1635954603 ## services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/resource/IpTablesHelper.java: ## @@ -0,0 +1,66 @@ +// +// Licensed to the

Re: [PR] List volumes by service offering id [cloudstack]

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

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on code in PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#discussion_r1635943766 ## services/secondary-storage/server/src/main/java/org/apache/cloudstack/storage/resource/IpTablesHelper.java: ## @@ -0,0 +1,66 @@ +// +// Licensed to the

Re: [PR] prevent duplicate ip table rules in SSVM [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on code in PR #8530: URL: https://github.com/apache/cloudstack/pull/8530#discussion_r1635937702 ## utils/src/main/java/com/cloud/utils/script/Script.java: ## @@ -242,13 +242,20 @@ public String execute(OutputInterpreter interpreter) {

Re: [PR] linstor: Support VM-Instance Disk snapshots [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on PR #8796: URL: https://github.com/apache/cloudstack/pull/8796#issuecomment-2162277497 > @DaanHoogland Can I get this added to the 4.19.1 milestone, this PR is now sitting around for a few months now... You are free to do that yourself (as a committer) but I

(cloudstack) branch main updated: [UI] Add option to specify account/project while deploying VMs and creating networks (#8919)

2024-06-12 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new 3fe2b46dd0b [UI] Add option to specify

Re: [PR] [UI] Add option to specify account/project while deploying VMs and creating networks [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland merged PR #8919: URL: https://github.com/apache/cloudstack/pull/8919 -- 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. To unsubscribe, e-mail:

Re: [I] cloud-init not running after boot [cloudstack]

2024-06-12 Thread via GitHub
MartinEmrich commented on issue #8129: URL: https://github.com/apache/cloudstack/issues/8129#issuecomment-2162263937 Ok. IDK if the documented change would work, too, bud the `ds-identify.cfg` file works fine for me. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-2162259625 @harikrishna-patnala 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

Re: [PR] engine-storage: Set SecretConsumerDetail for VM live migration with storage on shared NFS [cloudstack]

2024-06-12 Thread via GitHub
abh1sar commented on PR #9222: URL: https://github.com/apache/cloudstack/pull/9222#issuecomment-2162258634 @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

Re: [PR] engine-storage: Set SecretConsumerDetail for VM live migration with storage on shared NFS [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #9222: URL: https://github.com/apache/cloudstack/pull/9222#issuecomment-2162259568 @abh1sar 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] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-12 Thread via GitHub
harikrishna-patnala commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-2162257510 @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

Re: [PR] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-12 Thread via GitHub
harikrishna-patnala commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-2162254011 @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

Re: [I] Delete Template from specific secondary storage [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland closed issue #9228: Delete Template from specific secondary storage URL: https://github.com/apache/cloudstack/issues/9228 -- 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: [I] Delete Template from specific secondary storage [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on issue #9228: URL: https://github.com/apache/cloudstack/issues/9228#issuecomment-2162250982 @nxsbi , you are right. This would be a good addition. As a workaround you can look at template_store_ref. It does not have a removed field but for your use case it should,

Re: [PR] list by isEncrypted [cloudstack]

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

Re: [I] Allow migration of encrypted volumes between 2 data store clusters [cloudstack]

2024-06-12 Thread via GitHub
kiranchavala closed issue #9192: Allow migration of encrypted volumes between 2 data store clusters URL: https://github.com/apache/cloudstack/issues/9192 -- 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: [I] Allow migration of encrypted volumes between 2 data store clusters [cloudstack]

2024-06-12 Thread via GitHub
kiranchavala commented on issue #9192: URL: https://github.com/apache/cloudstack/issues/9192#issuecomment-2162243167 Thanks, @sureshanaparti Issue is not observed with 4.18.1 Closing the issue -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-12 Thread via GitHub
harikrishna-patnala commented on code in PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#discussion_r1635907580 ## server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java: ## @@ -2235,6 +2236,12 @@ private void setNewIopsLimits(VolumeVO volume,

Re: [PR] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-12 Thread via GitHub
harikrishna-patnala commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-2162241741 @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

Re: [PR] Timeout config to copy the disks of remote KVM instance while importing the instance from an external host [cloudstack]

2024-06-12 Thread via GitHub
sureshanaparti commented on code in PR #9213: URL: https://github.com/apache/cloudstack/pull/9213#discussion_r1635887296 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java: ## @@ -5336,18 +5336,18 @@ public void

Re: [PR] Timeout config to copy the disks of remote KVM instance while importing the instance from an external host [cloudstack]

2024-06-12 Thread via GitHub
sureshanaparti commented on code in PR #9213: URL: https://github.com/apache/cloudstack/pull/9213#discussion_r1635886690 ## api/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManager.java: ## @@ -30,6 +30,15 @@ public interface UnmanagedVMsManager extends VmImportService,

Re: [PR] engine-storage: Set SecretConsumerDetail for VM live migration with storage on shared NFS [cloudstack]

2024-06-12 Thread via GitHub
abh1sar commented on PR #9222: URL: https://github.com/apache/cloudstack/pull/9222#issuecomment-2162208826 Removed a condition which was not allowing live migrate for powerflex as @harikrishna-patnala suggested on issue https://github.com/apache/cloudstack/issues/8255 -- This is an

Re: [PR] storage: fix private templates are not copied to new image store [cloudstack]

2024-06-12 Thread via GitHub
weizhouapache commented on code in PR #9206: URL: https://github.com/apache/cloudstack/pull/9206#discussion_r1635877732 ## engine/storage/image/src/main/java/org/apache/cloudstack/storage/image/TemplateServiceImpl.java: ## @@ -532,11 +532,6 @@ public void

Re: [PR] Timeout config to copy the disks of remote KVM instance while importing the instance from an external host [cloudstack]

2024-06-12 Thread via GitHub
vishesh92 commented on code in PR #9213: URL: https://github.com/apache/cloudstack/pull/9213#discussion_r1635875776 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java: ## @@ -5336,18 +5336,18 @@ public void

Re: [PR] Timeout config to copy the disks of remote KVM instance while importing the instance from an external host [cloudstack]

2024-06-12 Thread via GitHub
vishesh92 commented on code in PR #9213: URL: https://github.com/apache/cloudstack/pull/9213#discussion_r1635875475 ## api/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManager.java: ## @@ -30,6 +30,15 @@ public interface UnmanagedVMsManager extends VmImportService,

Re: [PR] List volumes by service offering id [cloudstack]

2024-06-12 Thread via GitHub
blueorangutan commented on PR #9211: URL: https://github.com/apache/cloudstack/pull/9211#issuecomment-2162192749 @sureshanaparti 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

Re: [PR] List volumes by service offering id [cloudstack]

2024-06-12 Thread via GitHub
sureshanaparti commented on PR #9211: URL: https://github.com/apache/cloudstack/pull/9211#issuecomment-2162191425 @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

Re: [I] cloud-init not running after boot [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland commented on issue #8129: URL: https://github.com/apache/cloudstack/issues/8129#issuecomment-2162179075 @MartinEmrich , please see @weizhouapache 's comment above. I am closing the issue, but you are of course free to re-open or start a new one. In case implementation is

Re: [I] cloud-init not running after boot [cloudstack]

2024-06-12 Thread via GitHub
DaanHoogland closed issue #8129: cloud-init not running after boot URL: https://github.com/apache/cloudstack/issues/8129 -- 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. To

Re: [PR] Upload volume format check with the supported hypervisor, fail if supported hypervisor not found on zone [cloudstack]

2024-06-12 Thread via GitHub
sureshanaparti commented on code in PR #9224: URL: https://github.com/apache/cloudstack/pull/9224#discussion_r1635862036 ## server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java: ## @@ -564,7 +567,7 @@ private boolean validateVolume(Account caller, long ownerId,

Re: [PR] Upload volume format check with the supported hypervisor, fail if supported hypervisor not found on zone [cloudstack]

2024-06-12 Thread via GitHub
sureshanaparti commented on code in PR #9224: URL: https://github.com/apache/cloudstack/pull/9224#discussion_r1635862036 ## server/src/main/java/com/cloud/storage/VolumeApiServiceImpl.java: ## @@ -564,7 +567,7 @@ private boolean validateVolume(Account caller, long ownerId,

<    1   2   3   4