Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1962239217 [SF] Trillian test result (tid-9322) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46824 seconds Marvin logs: https://github.com

Re: [PR] Improve logs in primary storage removal process [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1962238864 [SF] Trillian test result (tid-9321) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 51083 seconds Marvin logs: https://github.com

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1962216318 [SF] Trillian test result (tid-9319) Environment: kvm-alma9 (x2), Advanced Networking with Mgmt server a9 Total time taken: 45921 seconds Marvin logs: https://github.com/

Re: [PR] Fix resource count discrepancies [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8302: URL: https://github.com/apache/cloudstack/pull/8302#issuecomment-1962216134 [SF] Trillian test result (tid-9318) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 49483 seconds Marvin logs: https://github.com

Re: [PR] cks: fix list apis response count [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1962214173 @shwstppr 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 respo

Re: [PR] cks: fix list apis response count [cloudstack]

2024-02-23 Thread via GitHub
shwstppr commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1962214029 @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] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
kohrar commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1962073812 A couple questions, if you don't mind: 1. Would it make sense from a latency-perspective to only wait 1ms if there's actually no data coming in? If there's a big chunk of data com

Re: [PR] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

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

Re: [PR] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8689: URL: https://github.com/apache/cloudstack/pull/8689#issuecomment-1962047621 @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 being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

2024-02-23 Thread via GitHub
weizhouapache commented on PR #8689: URL: https://github.com/apache/cloudstack/pull/8689#issuecomment-1962045647 @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] Console Proxy VM: High CPU when using secure VNC / NIOConnection [cloudstack]

2024-02-23 Thread via GitHub
kohrar commented on issue #8692: URL: https://github.com/apache/cloudstack/issues/8692#issuecomment-1962027968 I've rebuilt the cloud-systemvm files and copied it into the systemVM. I think your change helps as the process is no longer pegging the CPU at 100%. Now it's down to ~25% for a si

Re: [PR] Fix `deleteUser` API to prevent deletion of the caller [cloudstack]

2024-02-23 Thread via GitHub
lucas-a-martins commented on PR #8691: URL: https://github.com/apache/cloudstack/pull/8691#issuecomment-1961947798 > clgtm, not sure if a unit test is in order for this, @lucas-a-martins ? @DaanHoogland thanks for your review and approval. I've added some unit tests in the latest comm

Re: [PR] cks: fix list apis response count [cloudstack]

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

Re: [PR] cks: fix list apis response count [cloudstack]

2024-02-23 Thread via GitHub
codecov[bot] commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1961900904 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8701?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] cks: fix list apis response count [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1961854441 @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] cks: fix list apis response count [cloudstack]

2024-02-23 Thread via GitHub
shwstppr commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1961852076 @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] cks: fix list apis response count [cloudstack]

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

Re: [PR] cks: fix list apis count [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1961810602 @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] cks: fix list apis count [cloudstack]

2024-02-23 Thread via GitHub
shwstppr commented on PR #8701: URL: https://github.com/apache/cloudstack/pull/8701#issuecomment-1961807852 @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

[PR] cks: fix list apis count [cloudstack]

2024-02-23 Thread via GitHub
shwstppr opened a new pull request, #8701: URL: https://github.com/apache/cloudstack/pull/8701 ### Description Fixes count value in listKubernetesClusters and listSupportedKubernetesVersions APIs response. Currently, it returns items count of the response ### Types of change

Re: [I] Console Proxy VM: High CPU when using secure VNC / NIOConnection [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on issue #8692: URL: https://github.com/apache/cloudstack/issues/8692#issuecomment-1961798839 I was thinking you should build yourself if you can. If you can't we have to think of something else. I can't give you access directly. -- This is an automated message from

Re: [I] VPC network tier DNS doesn't reflect VPC DNS settings [cloudstack]

2024-02-23 Thread via GitHub
alexandremattioli commented on issue #8598: URL: https://github.com/apache/cloudstack/issues/8598#issuecomment-1961714359 @Pearl1594 @nvazquez related to NSX as well -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [I] Console Proxy VM: High CPU when using secure VNC / NIOConnection [cloudstack]

2024-02-23 Thread via GitHub
kohrar commented on issue #8692: URL: https://github.com/apache/cloudstack/issues/8692#issuecomment-1961672520 @DaanHoogland Could you point me to how I can test out the fix in the Console Proxy VM? Is there a way to get access to the built files by blue orangutan in that particular PR? Th

Re: [PR] Fixed deleteVlanIpRange API description [cloudstack]

2024-02-23 Thread via GitHub
codecov[bot] commented on PR #8700: URL: https://github.com/apache/cloudstack/pull/8700#issuecomment-1961650574 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8700?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] Fix XenServer nested extra configuration [cloudstack]

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

Re: [PR] Fixed deleteVlanIpRange API description [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8700: URL: https://github.com/apache/cloudstack/pull/8700#issuecomment-1961639052 Awaiting build. GHA should be enough for this one. -- 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] Fix XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8698: URL: https://github.com/apache/cloudstack/pull/8698#issuecomment-1961628341 @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 XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8698: URL: https://github.com/apache/cloudstack/pull/8698#issuecomment-1961626367 @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] Fix XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8698: URL: https://github.com/apache/cloudstack/pull/8698#issuecomment-1961625022 > @DaanHoogland Thanks for the review. > > > CLGTM, but don't you think the replacement code should be called inside the `VirtualMachine.add<>()` methods, @gpordeus ? >

[PR] Fixed deleteVlanIpRange API description [cloudstack]

2024-02-23 Thread via GitHub
OlegChuev opened a new pull request, #8700: URL: https://github.com/apache/cloudstack/pull/8700 ### Description This PR updates `deleteVlanIpRange` API description ### Types of changes - [ ] Breaking change (fix or featu

Re: [PR] CKS: fix /opt/bin/deploy-cloudstack-secret in CKS control nodes [cloudstack]

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

Re: [PR] NSX: Cleanup PF rule when VM is expunged and fix UDP - LB rules [cloudstack]

2024-02-23 Thread via GitHub
codecov[bot] commented on PR #8699: URL: https://github.com/apache/cloudstack/pull/8699#issuecomment-1961606002 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8699?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] Fix XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
gpordeus commented on PR #8698: URL: https://github.com/apache/cloudstack/pull/8698#issuecomment-1961602112 @DaanHoogland Thanks for the review. > CLGTM, but don't you think the replacement code should be called inside the `VirtualMachine.add<>()` methods, @gpordeus ? Do you me

[PR] NSX: Cleanup PF rule when VM is expunged and fix UDP - LB rules [cloudstack]

2024-02-23 Thread via GitHub
Pearl1594 opened a new pull request, #8699: URL: https://github.com/apache/cloudstack/pull/8699 ### Description This PR... ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionali

(cloudstack) branch fr03-cleanup-PF-udp-lb updated: fix unit test

2024-02-23 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a commit to branch fr03-cleanup-PF-udp-lb in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/fr03-cleanup-PF-udp-lb by this push: new 5a74c3e

Re: [PR] Added JS Script to pick Favicon, Title from config file [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #7418: URL: https://github.com/apache/cloudstack/pull/7418#issuecomment-1961579693 tested, the favicon gets set for all pages (or at least those few I checked. The login has another title than the rest. This seems ok to me. @sbrueseke @rohityadavcloud @shwstppr

(cloudstack-www) branch staging-site updated: survey as redirect (#175)

2024-02-23 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch staging-site in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git The following commit(s) were added to refs/heads/staging-site by this push: new d090da946 survey as redirect (

Re: [PR] CKS: fix /opt/bin/deploy-cloudstack-secret in CKS control nodes [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8697: URL: https://github.com/apache/cloudstack/pull/8697#issuecomment-1961499848 @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] CKS: fix /opt/bin/deploy-cloudstack-secret in CKS control nodes [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8697: URL: https://github.com/apache/cloudstack/pull/8697#issuecomment-1961496357 @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] Fix XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
codecov[bot] commented on PR #8698: URL: https://github.com/apache/cloudstack/pull/8698#issuecomment-1961411502 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8698?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1961409387 @DaanHoogland 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 r

Re: [PR] CKS: fix /opt/bin/deploy-cloudstack-secret in CKS control nodes [cloudstack]

2024-02-23 Thread via GitHub
codecov[bot] commented on PR #8697: URL: https://github.com/apache/cloudstack/pull/8697#issuecomment-1961406280 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8697?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1961406027 @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 co

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1961405603 @JoaoJandre at several location more then one debug statement is called in a row. Should these be unified? -- This is an automated message from the Apache Git Service. To respon

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1961404549 @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 co

Re: [PR] Fix XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on code in PR #8698: URL: https://github.com/apache/cloudstack/pull/8698#discussion_r1500696327 ## plugins/hypervisors/xenserver/src/main/java/org/apache/cloudstack/hypervisor/xenserver/ExtraConfigurationUtility.java: ## @@ -71,26 +77,26 @@ private static

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

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

Re: [PR] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

2024-02-23 Thread via GitHub
gpordeus commented on PR #8689: URL: https://github.com/apache/cloudstack/pull/8689#issuecomment-1961364011 > @gpordeus , this sounds like a good use case for an integration test. Will you consider that? Sure, on it. -- This is an automated message from the Apache Git Service. To r

[PR] Fix XenServer nested extra configuration [cloudstack]

2024-02-23 Thread via GitHub
gpordeus opened a new pull request, #8698: URL: https://github.com/apache/cloudstack/pull/8698 ### Description This PR fixes a problem when trying to boot up a XenServer VM with a template through an ISO. Steps to reproduce: - Enable `enable.additional.vm.configuration` and

[PR] CKS: fix /opt/bin/deploy-cloudstack-secret in CKS control nodes [cloudstack]

2024-02-23 Thread via GitHub
weizhouapache opened a new pull request, #8697: URL: https://github.com/apache/cloudstack/pull/8697 ### Description This PR fixes the typos in deploy-cloudstack-secret script. ### Types of changes - [ ] Breaking chan

Re: [I] CloudStack API not Listed [cloudstack]

2024-02-23 Thread via GitHub
weizhouapache commented on issue #8695: URL: https://github.com/apache/cloudstack/issues/8695#issuecomment-1961324544 @th3cod3r in the control node, there is a script to do it ``` root@CKS-1292-2-control-18dcb0cd536:~# deploy-cloudstack-secret -h Usage: ./deploy-cloudstack-secret

Re: [PR] Improve logs in primary storage removal process [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1961295883 @DaanHoogland 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 r

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1961295819 @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] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#issuecomment-1961295078 @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] Improve logs in primary storage removal process [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1961292904 @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 co

Re: [PR] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8689: URL: https://github.com/apache/cloudstack/pull/8689#issuecomment-1961283712 @gpordeus , this sounds like a good use case for an integration test. Will you considder that? -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1961280972 @DaanHoogland a [SL] Trillian-Jenkins test job (alma9 mgmt + kvm-alma9) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respo

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1961278721 @blueorangutan test alma9 kvm-alma9 -- 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: [I] CloudStack API not Listed [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on issue #8695: URL: https://github.com/apache/cloudstack/issues/8695#issuecomment-1961277784 @th3cod3r , I am not clear on what you are looking for, but if it is the cloudstack API keys, you need to go into the user. ![image](https://github.com/apache/cloudstack/a

Re: [PR] Improve logs when adding components to avoid set [cloudstack]

2024-02-23 Thread via GitHub
JoaoJandre commented on code in PR #7214: URL: https://github.com/apache/cloudstack/pull/7214#discussion_r1500615169 ## server/src/main/java/com/cloud/deploy/DeploymentPlanningManagerImpl.java: ## @@ -298,24 +301,33 @@ protected void avoidOtherClustersForDeploymentIfMigrationDi

Re: [PR] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

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

Re: [PR] New API "checkVolume" to check and repair any leaks or issues reported by qemu-img check [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Added JS Script to pick Favicon, Title from config file [cloudstack]

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

Re: [PR] Fix resource count discrepancies [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8302: URL: https://github.com/apache/cloudstack/pull/8302#issuecomment-1961200332 @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] Fix resource count discrepancies [cloudstack]

2024-02-23 Thread via GitHub
vishesh92 commented on PR #8302: URL: https://github.com/apache/cloudstack/pull/8302#issuecomment-1961200017 @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] docs: fix spelling in code comments [cloudstack]

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

Re: [PR] feature: webhooks [cloudstack]

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

Re: [PR] console proxy client sleep more generic [cloudstack]

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

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1961123680 Packaging result [LL]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 6211 -- This is an auto

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
codecov[bot] commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1961030109 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8694?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

[I] CloudStack API not Listed [cloudstack]

2024-02-23 Thread via GitHub
th3cod3r opened a new issue, #8695: URL: https://github.com/apache/cloudstack/issues/8695 I onboard a K8s cluster on version 1.28.4 with Management and KVM on a single node using 4.18.1 cloudstack version. ISO was uploaded from https://download.cloudstack.org/cks/ After that I

Re: [PR] console proxy client sleep more generic [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

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

Re: [PR] Added JS Script to pick Favicon, Title from config file [cloudstack]

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

Re: [PR] Added JS Script to pick Favicon, Title from config file [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #7418: URL: https://github.com/apache/cloudstack/pull/7418#issuecomment-1960957264 @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] Added JS Script to pick Favicon, Title from config file [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #7418: URL: https://github.com/apache/cloudstack/pull/7418#issuecomment-1960956155 @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] New API "checkVolume" to check and repair any leaks or issues reported by qemu-img check [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8577: URL: https://github.com/apache/cloudstack/pull/8577#issuecomment-1960952770 @JoaoJandre is this ok for you now? @kiranchavala do you have any qa work for this planned? -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
weizhouapache commented on code in PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#discussion_r1500374118 ## services/console-proxy/server/src/main/java/com/cloud/consoleproxy/ConsoleProxyNoVncClient.java: ## @@ -139,6 +134,11 @@ public void run() {

Re: [PR] Fix resource count discrepancies [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8302: URL: https://github.com/apache/cloudstack/pull/8302#issuecomment-1960947997 @vishesh92 as you changed the base branch to main, i adjusted the milestone for this -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1960939292 @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] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1960937972 @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] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8694: URL: https://github.com/apache/cloudstack/pull/8694#issuecomment-1960935833 @weizhouapache @nvazquez , this is purely based on the description in #8692. Any advice on verification? -- This is an automated message from the Apache Git Service. To respond

[PR] console proxy client sleep more generic [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland opened a new pull request, #8694: URL: https://github.com/apache/cloudstack/pull/8694 ### Description This PR... Fixes: #8692 ### Types of changes - [ ] Breaking change (fix or feature that would cause existi

Re: [PR] Improve logs in primary storage removal process [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1960920586 @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] Improve logs in primary storage removal process [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1960919770 @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] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8689: URL: https://github.com/apache/cloudstack/pull/8689#issuecomment-1960897760 @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] Fix being able to expunge a VM through destroyVirtualMachine even when role rule does not allow [cloudstack]

2024-02-23 Thread via GitHub
blueorangutan commented on PR #8689: URL: https://github.com/apache/cloudstack/pull/8689#issuecomment-1960899608 @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] Linstor 4.19 fix script alllines [cloudstack]

2024-02-23 Thread via GitHub
rp- commented on PR #8670: URL: https://github.com/apache/cloudstack/pull/8670#issuecomment-1960893110 > So this must be included to allow for disaster recovery on large scale installations I read. Why is that? I use `Script` to read `drbdsetup` command output to check resource state

Re: [PR] Linstor 4.19 fix script alllines [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on PR #8670: URL: https://github.com/apache/cloudstack/pull/8670#issuecomment-1960887920 So this must be included to allow for disaster recovery on large scale installations I read. Why is that? -- This is an automated message from the Apache Git Service. To resp

Re: [PR] Linstor 4.19 fix script alllines [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland commented on code in PR #8670: URL: https://github.com/apache/cloudstack/pull/8670#discussion_r1500318740 ## utils/src/test/java/com/cloud/utils/ScriptTest.java: ## @@ -111,6 +111,20 @@ public String interpret(BufferedReader reader) throws IOException { As

Re: [PR] removing the usage of volumeFreeze StorPool API call [cloudstack]

2024-02-23 Thread via GitHub
slavkap commented on code in PR #8575: URL: https://github.com/apache/cloudstack/pull/8575#discussion_r1500316315 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -632,22 +590,17 @@ public

Re: [PR] removing the usage of volumeFreeze StorPool API call [cloudstack]

2024-02-23 Thread via GitHub
slavkap commented on code in PR #8575: URL: https://github.com/apache/cloudstack/pull/8575#discussion_r1500315507 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -485,55 +484,14 @@ public

(cloudstack) branch 4.19 updated: UI: remove duplicated section for backup (#8678)

2024-02-23 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.19 by this push: new ed579bfeca5 UI: remove duplicated section for back

Re: [PR] Linstor 4.19 fix script alllines [cloudstack]

2024-02-23 Thread via GitHub
rp- commented on code in PR #8670: URL: https://github.com/apache/cloudstack/pull/8670#discussion_r1500313991 ## utils/src/test/java/com/cloud/utils/ScriptTest.java: ## @@ -111,6 +111,20 @@ public String interpret(BufferedReader reader) throws IOException { Assert.asse

Re: [PR] UI: remove duplicated section for backup [cloudstack]

2024-02-23 Thread via GitHub
DaanHoogland merged PR #8678: URL: https://github.com/apache/cloudstack/pull/8678 -- 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: commits-unsubscr...@cl