Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2169162965 @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 progres

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

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

(cloudstack) branch main updated: server: fix mgmt server 503 error by renaming kubernetesClusterHelper to kubernetesServiceHelper

2024-06-14 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou 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 aab13619491 server: fix mgmt server 503 error b

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache closed pull request #8602: [HEALTH] 4.20/main Health Check, please don't merge this! URL: https://github.com/apache/cloudstack/pull/8602 -- 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] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2169146467 @vishesh92 a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To resp

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

2024-06-14 Thread via GitHub
vishesh92 commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2169146122 @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 comme

Re: [PR] [HEALTH] 4.19 Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8601: URL: https://github.com/apache/cloudstack/pull/8601#issuecomment-2169069359 [SF] Trillian test result (tid-10450) Environment: kvm-ubuntu24 (x2), Advanced Networking with Mgmt server u24 Total time taken: 48479 seconds Marvin logs: https://github

Re: [PR] User friendly name of Downloaded Templates Volumes and ISOs [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9252: URL: https://github.com/apache/cloudstack/pull/9252#issuecomment-2168821597 [SF] Trillian test result (tid-10445) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 42388 seconds Marvin logs: https://github.co

Re: [PR] Add logs to CPVM connection process [cloudstack]

2024-06-14 Thread via GitHub
bernardodemarco commented on PR #8924: URL: https://github.com/apache/cloudstack/pull/8924#issuecomment-2168787249 > clgtm, can we now remove `com.cloud.consoleproxy.util.Logger` from the code base @bernardodemarco ? Just to deal with the technical debt. (we can always add a new proxy once

(cloudstack) branch main updated: fix build error with LinstorVMSnapshotStrategy

2024-06-14 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou 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 3a5c4a0cba5 fix build error with LinstorVMSnaps

Re: [PR] DRS: Ensure the destination host is part of the same cluster [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9245: URL: https://github.com/apache/cloudstack/pull/9245#issuecomment-2168737109 [SF] Trillian test result (tid-10443) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 48859 seconds Marvin logs: https://github.co

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

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-216870 [SF] Trillian test result (tid-10442) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 47866 seconds Marvin logs: https://github.co

Re: [PR] Add action button to enable/disable Oauth provider [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9242: URL: https://github.com/apache/cloudstack/pull/9242#issuecomment-2168705601 [SF] Trillian test result (tid-10444) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 43513 seconds Marvin logs: https://github.co

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

2024-06-14 Thread via GitHub
hsato03 commented on code in PR #8475: URL: https://github.com/apache/cloudstack/pull/8475#discussion_r1640296771 ## server/src/main/java/com/cloud/server/ManagementServerImpl.java: ## @@ -5068,31 +5068,66 @@ public Pair, Integer> listHypervisorCapab return new Pair,

Re: [PR] Normalizing volume consolidation in live migration on KVM [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on PR #8911: URL: https://github.com/apache/cloudstack/pull/8911#issuecomment-2168619354 > > > @weizhouapache , I don't think I got your point, could you explain it? > > > > @JoaoJandre As you described in the description `Currently, when a volume that was crea

Re: [PR] Fix assignVirtualMachine template checkAccess [cloudstack]

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

Re: [PR] Refactor type and range validation in configuration update process [cloudstack]

2024-06-14 Thread via GitHub
codecov[bot] commented on PR #9107: URL: https://github.com/apache/cloudstack/pull/9107#issuecomment-2168595317 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9107?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2168593190 [SF] Trillian Build Failed (tid-10453) -- 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 g

Re: [PR] Normalizing volume consolidation in live migration on KVM [cloudstack]

2024-06-14 Thread via GitHub
JoaoJandre commented on PR #8911: URL: https://github.com/apache/cloudstack/pull/8911#issuecomment-2168571595 > > @weizhouapache , I don't think I got your point, could you explain it? > > @JoaoJandre As you described in the description `Currently, when a volume that was created with

Re: [PR] Normalizing volume consolidation in live migration on KVM [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on PR #8911: URL: https://github.com/apache/cloudstack/pull/8911#issuecomment-2168564907 > @weizhouapache , I don't think I got your point, could you explain it? @JoaoJandre As you described in the description `Currently, when a volume that was created w

Re: [PR] Refactor type and range validation in configuration update process [cloudstack]

2024-06-14 Thread via GitHub
hsato03 commented on code in PR #9107: URL: https://github.com/apache/cloudstack/pull/9107#discussion_r1640203662 ## server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java: ## @@ -682,9 +682,8 @@ public boolean stop() { @DB public String updateConfi

Re: [PR] Normalizing volume consolidation in live migration on KVM [cloudstack]

2024-06-14 Thread via GitHub
JoaoJandre commented on PR #8911: URL: https://github.com/apache/cloudstack/pull/8911#issuecomment-2168533009 @weizhouapache , I don't think I got your point, could you explain it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Fix assignVirtualMachine template checkAccess [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8886: URL: https://github.com/apache/cloudstack/pull/8886#issuecomment-2168511845 @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 progres

Re: [PR] Fix assignVirtualMachine template checkAccess [cloudstack]

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

Re: [PR] Fix assignVirtualMachine template checkAccess [cloudstack]

2024-06-14 Thread via GitHub
hsato03 commented on PR #8886: URL: https://github.com/apache/cloudstack/pull/8886#issuecomment-2168478541 LGTM. Reproduced the tests and verified that `share.public.templates.with.other.domains` now works when assigning public template instances between different domains and that th

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

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

Re: [PR] [Veeam] disable jobs but keep backups [cloudstack]

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

Re: [I] DNS service not working for System VMs on a fresh new install Cloudstack [cloudstack]

2024-06-14 Thread via GitHub
bradsmin commented on issue #7473: URL: https://github.com/apache/cloudstack/issues/7473#issuecomment-2168338740 Yes, internal dns defined ( provided google dns ) at the time of configuration. Also tried to disable ufw firewall at KVM host. But issue still exists. -- This is an autom

Re: [PR] Update extraconfig for platform param in xen/xcpng [cloudstack]

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

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2168262991 @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 progres

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

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

Re: [PR] [Veeam] disable jobs but keep backups [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #6589: URL: https://github.com/apache/cloudstack/pull/6589#issuecomment-2168253109 @BryanMLima 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] [Veeam] disable jobs but keep backups [cloudstack]

2024-06-14 Thread via GitHub
BryanMLima commented on PR #6589: URL: https://github.com/apache/cloudstack/pull/6589#issuecomment-2168249840 @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] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#issuecomment-2168207157 @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 progre

Re: [PR] Ensure affinity groups are honored when VMs are deployed in parallel [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on PR #9201: URL: https://github.com/apache/cloudstack/pull/9201#issuecomment-2168182709 > @vishesh92 I had a look of the process of vm deployment, would it make sense to lock the method below ? > > https://github.com/apache/cloudstack/blob/cb9b3134f7fff972b63

Re: [I] Unable to live migrate VMware VMs with snapshot [cloudstack]

2024-06-14 Thread via GitHub
Pearl1594 commented on issue #9061: URL: https://github.com/apache/cloudstack/issues/9061#issuecomment-2168183656 @shwstppr I had tested that - yes reverting to another snapshot worked I believe -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-14 Thread via GitHub
sureshanaparti commented on PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#issuecomment-2168206243 @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] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-14 Thread via GitHub
sureshanaparti commented on code in PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#discussion_r1639947136 ## plugins/hypervisors/xenserver/src/main/java/org/apache/cloudstack/hypervisor/xenserver/ExtraConfigurationUtility.java: ## @@ -169,12 +178,28 @@ private sta

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2168201893 [SF] Trillian Build Failed (tid-10452) -- 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 g

Re: [I] Unable to live migrate VMware VMs with snapshot [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on issue #9061: URL: https://github.com/apache/cloudstack/issues/9061#issuecomment-2168191062 > @shwstppr I had tested that - yes reverting to another snapshot worked I believe so we can remove the restriction for vmware ? -- This is an automated message fro

Re: [PR] make server threads configurable with server.properties file [cloudstack]

2024-06-14 Thread via GitHub
codecov[bot] commented on PR #9260: URL: https://github.com/apache/cloudstack/pull/9260#issuecomment-2168154064 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9260?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [I] Unable to live migrate VMware VMs with snapshot [cloudstack]

2024-06-14 Thread via GitHub
alexandremattioli commented on issue #9061: URL: https://github.com/apache/cloudstack/issues/9061#issuecomment-2168126529 @Pearl1594 thanks for checking this. @shwstppr I think it should work fine. I've deployed an env, took a snapshot of a VM, then migrated it with storage directly in vC

Re: [I] Unable to live migrate VMware VMs with snapshot [cloudstack]

2024-06-14 Thread via GitHub
alexandremattioli commented on issue #9061: URL: https://github.com/apache/cloudstack/issues/9061#issuecomment-2168126843 @DaanHoogland I'd say yes, 4.20. -- 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 t

Re: [PR] [Quota] Improve Quota balance calculation flow [cloudstack]

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

Re: [PR] Support migration of VM imported from a remote host [cloudstack]

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

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2168078562 [SF] Trillian Build Failed (tid-10451) -- 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 g

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

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

Re: [PR] agent: reconnect after waiting 5 seconds [cloudstack]

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

Re: [I] Documentation: Missing information regarding deploying UEFI 'enabled' hosts [cloudstack]

2024-06-14 Thread via GitHub
rohityadavcloud closed issue #7126: Documentation: Missing information regarding deploying UEFI 'enabled' hosts URL: https://github.com/apache/cloudstack/issues/7126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] Support migration of VM imported from a remote host [cloudstack]

2024-06-14 Thread via GitHub
codecov[bot] commented on PR #9259: URL: https://github.com/apache/cloudstack/pull/9259#issuecomment-2168054397 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9259?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [PR] Support migration of VM imported from a remote host [cloudstack]

2024-06-14 Thread via GitHub
Pearl1594 commented on PR #9259: URL: https://github.com/apache/cloudstack/pull/9259#issuecomment-2168025903 live migration @sureshanaparti -- 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 sp

Re: [PR] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9166: URL: https://github.com/apache/cloudstack/pull/9166#issuecomment-2168025207 UI build: :heavy_check_mark: Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9166 (QA-JID-382) -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] make server threads configurable with server.properties file [cloudstack]

2024-06-14 Thread via GitHub
ahmedali6 closed pull request #9165: make server threads configurable with server.properties file URL: https://github.com/apache/cloudstack/pull/9165 -- 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] Support migration of VM imported from a remote host [cloudstack]

2024-06-14 Thread via GitHub
sureshanaparti commented on PR #9259: URL: https://github.com/apache/cloudstack/pull/9259#issuecomment-2168017826 @Pearl1594 Is this live migration of the volume, or offline? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Ensure affinity groups are honored when VMs are deployed in parallel [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on PR #9201: URL: https://github.com/apache/cloudstack/pull/9201#issuecomment-2168028191 @vishesh92 I had a look of the process of vm deployment, would it make sense to lock the method below ? https://github.com/apache/cloudstack/blob/cb9b3134f7fff972b63d8

[PR] make server threads configurable with server.properties file [cloudstack]

2024-06-14 Thread via GitHub
ahmedali6 opened a new pull request, #9260: URL: https://github.com/apache/cloudstack/pull/9260 ### Description Fixes: https://github.com/apache/cloudstack/issues/9161 ### Types of changes - Enhancement (improves an existing

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2168013987 [SF] Trillian Build Failed (tid-10449) -- 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 g

Re: [PR] Support migration of VM imported from a remote host [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on PR #9259: URL: https://github.com/apache/cloudstack/pull/9259#issuecomment-2168012125 @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] Support migration of VM imported from a remote host [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9259: URL: https://github.com/apache/cloudstack/pull/9259#issuecomment-2168015281 @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 progress

Re: [PR] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9166: URL: https://github.com/apache/cloudstack/pull/9166#issuecomment-2167990603 @DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on PR #9166: URL: https://github.com/apache/cloudstack/pull/9166#issuecomment-2167989203 @blueorangutan ui -- 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 comm

Re: [PR] [Quota] Improve Quota balance calculation flow [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-2167990630 @BryanMLima 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] Allow deletion of external managed cks nodes [cloudstack]

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

Re: [PR] [Quota] Improve Quota balance calculation flow [cloudstack]

2024-06-14 Thread via GitHub
BryanMLima commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-2167987711 @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] [HEALTH] 4.19 Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8601: URL: https://github.com/apache/cloudstack/pull/8601#issuecomment-2167980194 @weizhouapache a [SL] Trillian-Jenkins test job (ubuntu24 mgmt + kvm-ubuntu24) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. T

Re: [PR] [Quota] Improve Quota balance calculation flow [cloudstack]

2024-06-14 Thread via GitHub
BryanMLima commented on code in PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#discussion_r1639785664 ## usage/src/main/java/com/cloud/usage/UsageManagerImpl.java: ## @@ -405,9 +405,14 @@ protected void runInContextInternal() { cal.add(Calendar.MI

Re: [PR] ui: fix user create featured template from volume [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on PR #9253: URL: https://github.com/apache/cloudstack/pull/9253#issuecomment-2167979570 @shwstppr , I tried in qa but see no featured field: with change it looks good but without (qa without PR uri) it looks the same. Any extra condition I am missing maybe?

Re: [PR] [HEALTH] 4.19 Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on PR #8601: URL: https://github.com/apache/cloudstack/pull/8601#issuecomment-2167978010 @blueorangutan test ubuntu24 kvm-ubuntu24 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] Support migration of VM imported from a remote host [cloudstack]

2024-06-14 Thread via GitHub
Pearl1594 opened a new pull request, #9259: URL: https://github.com/apache/cloudstack/pull/9259 ### Description This PR fixes: https://github.com/apache/cloudstack/issues/8827 Importing a VM from a remote host uses a dummy VM template which is not present on the secondary store, le

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2167966026 @blueorangutan test ubuntu24 kvm-ubuntu24 -- 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 t

Re: [PR] [HEALTH] 4.20/main Health Check, please don't merge this! [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8602: URL: https://github.com/apache/cloudstack/pull/8602#issuecomment-2167968748 @weizhouapache a [SL] Trillian-Jenkins test job (ubuntu24 mgmt + kvm-ubuntu24) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. T

(cloudstack) 01/01: Support migration of VM imported from a remote host

2024-06-14 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a commit to branch 8827-fix-migration-of-imported-vm-kvm in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 2a96d9f98962e6945548d4b92c20a5d43436883f Author: Pearl Dsilva AuthorDate: Fri Jun

(cloudstack) branch 8827-fix-migration-of-imported-vm-kvm created (now 2a96d9f9896)

2024-06-14 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch 8827-fix-migration-of-imported-vm-kvm in repository https://gitbox.apache.org/repos/asf/cloudstack.git at 2a96d9f9896 Support migration of VM imported from a remote host This branch in

Re: [PR] Add certificate validation to check headers [cloudstack]

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

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2167954127 @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 progres

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

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

Re: [I] [UI] Error on navigating to Network -> VNF appliances page [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on issue #9257: URL: https://github.com/apache/cloudstack/issues/9257#issuecomment-2167951582 yes, duplicate of #8994, closing -- 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

Re: [I] [UI] Error on navigating to Network -> VNF appliances page [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland closed issue #9257: [UI] Error on navigating to Network -> VNF appliances page URL: https://github.com/apache/cloudstack/issues/9257 -- 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] [Quota] Add API to list preset variables [cloudstack]

2024-06-14 Thread via GitHub
BryanMLima commented on PR #8372: URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2167950272 > @BryanMLima shouldn't we need to run smoketests on the PR, or do we think they were covered by GHA/simulator smoketests? @rohityadavcloud, the smoke tests were already run i

Re: [I] [UI] Error on navigating to Network -> VNF appliances page [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache commented on issue #9257: URL: https://github.com/apache/cloudstack/issues/9257#issuecomment-2167946598 @sureshanaparti @DaanHoogland this should be addressed by #8994 -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] agent: reconnect after waiting 5 seconds [cloudstack]

2024-06-14 Thread via GitHub
codecov[bot] commented on PR #9258: URL: https://github.com/apache/cloudstack/pull/9258#issuecomment-2167936692 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9258?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments

Re: [PR] User data content size validation, and related code improvements [cloudstack]

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

Re: [PR] agent: reconnect after waiting 5 seconds [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9258: URL: https://github.com/apache/cloudstack/pull/9258#issuecomment-2167921160 @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 progres

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

2024-06-14 Thread via GitHub
vishesh92 commented on code in PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#discussion_r1639746356 ## core/src/main/java/com/cloud/agent/api/ManageServiceCommand.java: ## @@ -0,0 +1,49 @@ +// +// Licensed to the Apache Software Foundation (ASF) under one +// or

Re: [PR] agent: reconnect after waiting 5 seconds [cloudstack]

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

Re: [PR] agent: reconnect after waiting 5 seconds [cloudstack]

2024-06-14 Thread via GitHub
github-actions[bot] commented on PR #9258: URL: https://github.com/apache/cloudstack/pull/9258#issuecomment-2167910977 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 Gi

Re: [PR] Add certificate validation to check headers [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on code in PR #9255: URL: https://github.com/apache/cloudstack/pull/9255#discussion_r1639740153 ## framework/security/src/main/java/org/apache/cloudstack/framework/security/keystore/KeystoreManagerImpl.java: ## @@ -47,24 +48,38 @@ public class KeystoreMana

[PR] agent: reconnect after waiting 5 seconds [cloudstack]

2024-06-14 Thread via GitHub
weizhouapache opened a new pull request, #9258: URL: https://github.com/apache/cloudstack/pull/9258 ### Description This PR fixes #8517 see the steps to reproduce the issue in the description of #8517 ### Types of

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

2024-06-14 Thread via GitHub
vishesh92 commented on code in PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#discussion_r1639739261 ## server/src/main/java/com/cloud/network/element/VpcVirtualRouterElement.java: ## @@ -285,30 +318,7 @@ public boolean prepare(final Network network, final NicPro

[I] [UI] Error on navigating to Network -> VNF appliances page [cloudstack]

2024-06-14 Thread via GitHub
sureshanaparti opened a new issue, #9257: URL: https://github.com/apache/cloudstack/issues/9257 [UI] Error below on navigating to Network -> VNF appliances page `Unable to execute API command listtemplates due to missing parameter templatefilter` # ISSUE TYPE

Re: [PR] ui: fix user create featured template from volume [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9253: URL: https://github.com/apache/cloudstack/pull/9253#issuecomment-2167892437 UI build: :heavy_check_mark: Live QA URL: https://qa.cloudstack.cloud/simulator/pr/9253 (QA-JID-381) -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] Prevent network disruption on adding a VPC tier for redundant VRs [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on code in PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#discussion_r1639724171 ## core/src/main/java/com/cloud/agent/api/ManageServiceCommand.java: ## @@ -0,0 +1,49 @@ +// +// Licensed to the Apache Software Foundation (ASF) under one +//

Re: [PR] Allow deletion of external managed cks nodes [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9183: URL: https://github.com/apache/cloudstack/pull/9183#issuecomment-2167870950 @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 progress

Re: [PR] Allow deletion of external managed cks nodes [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on PR #9183: URL: https://github.com/apache/cloudstack/pull/9183#issuecomment-2167869242 @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] ui: fix user create featured template from volume [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9253: URL: https://github.com/apache/cloudstack/pull/9253#issuecomment-2167867845 @DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] ui: fix user create featured template from volume [cloudstack]

2024-06-14 Thread via GitHub
DaanHoogland commented on PR #9253: URL: https://github.com/apache/cloudstack/pull/9253#issuecomment-2167866997 @blueorangutan ui -- 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 comm

Re: [PR] veeam: fix issues with PreSetup and DVS and Solidfire [cloudstack]

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

Re: [PR] Add certificate validation to check headers [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #9255: URL: https://github.com/apache/cloudstack/pull/9255#issuecomment-216784 @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 p

Re: [PR] Add certificate validation to check headers [cloudstack]

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

Re: [PR] User data content size validation, and related code improvements [cloudstack]

2024-06-14 Thread via GitHub
blueorangutan commented on PR #8418: URL: https://github.com/apache/cloudstack/pull/8418#issuecomment-2167820541 @kiranchavala 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

  1   2   >