(cloudstack) branch 9962-ssvm-unable-to-find-valid-certification-path-to-requested-target updated (969205abd35 -> 0bfaef3fd1f)

2024-12-05 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch 9962-ssvm-unable-to-find-valid-certification-path-to-requested-target in repository https://gitbox.apache.org/repos/asf/cloudstack.git discard 969205abd35 Remove SNI option that is correct as def

Re: [PR] Remove SNI option in _run.sh, as it is correct as default. [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10028: URL: https://github.com/apache/cloudstack/pull/10028#issuecomment-2519619546 > [SF] Trillian test result (tid-11840) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 52094 seconds Marvin logs: https://github.com/blueo

[PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti opened a new pull request, #10045: URL: https://github.com/apache/cloudstack/pull/10045 ### Description This PR allows config drive deletion of migrated VM, on host maintenance (from it's last source location - host cache/primary/secondary storage). Fixes below i

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

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

Re: [PR] UI improvements [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #9773: URL: https://github.com/apache/cloudstack/pull/9773#issuecomment-2519636943 @andrijapanicsb @rohityadavcloud , are your concerns dealt with, here? this seems ready for merge otherwise. -- This is an automated message from the Apache Git Service. To resp

Re: [PR] Remove SNI option in _run.sh, as it is correct as default. [cloudstack]

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

Re: [PR] Bug fix : add check befor fencing [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on code in PR #10039: URL: https://github.com/apache/cloudstack/pull/10039#discussion_r1870919549 ## plugins/hypervisors/kvm/src/main/java/org/apache/cloudstack/kvm/ha/KVMHAProvider.java: ## @@ -84,11 +85,18 @@ public boolean recover(Host r) throws HARecov

Re: [PR] Bug fix : add check befor fencing [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10039: URL: https://github.com/apache/cloudstack/pull/10039#issuecomment-2519644078 sorry to bug you with this, the linter says no. (just some trailing whitespace.) -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Bug fix : add check befor fencing [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10039: URL: https://github.com/apache/cloudstack/pull/10039#issuecomment-2519658977 @slavkap @weizhouapache any sugestions for testing? Also, I marked this for 4.21 @leo79901 , but it seems to me it can go onto 4.19. I think you'd like this fix sooner rat

Re: [PR] UI: Fix `Create` dropdown menu not changing language dynamically [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland merged PR #10023: URL: https://github.com/apache/cloudstack/pull/10023 -- 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...@

Re: [I] Hide VPN users' passwords in virtual router [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache commented on issue #10024: URL: https://github.com/apache/cloudstack/issues/10024#issuecomment-2519742283 Not sure if strongswan supports encrypted password for vpn users -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] Storage issues on VMware 8.0U1 (8.0.1.0) [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache closed issue #7572: Storage issues on VMware 8.0U1 (8.0.1.0) URL: https://github.com/apache/cloudstack/issues/7572 -- 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.

[PR] upgrade: consider multiple hypervisors and secondary storages [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache opened a new pull request, #10046: URL: https://github.com/apache/cloudstack/pull/10046 This fixes an issue during upgrade in edge case. Steps to reproduce the issue - deploy an env with ACS 4.18 - the env has multiple hypervisors: KVM and VMware - the env has mu

Re: [I] Storage issues on VMware 8.0U1 (8.0.1.0) [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache commented on issue #7572: URL: https://github.com/apache/cloudstack/issues/7572#issuecomment-2519773692 this has been fixed by #9625 tested ok on vmware 8.0.1/8.0.2/8.0.3 #9591 -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Add parameter to not create additional users on `cloudstack-setup-databases` [cloudstack]

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

Re: [PR] Fix NPE issues during host rolling maintenance, due to host tags and custom constrained/unconstrained service offering [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on code in PR #9844: URL: https://github.com/apache/cloudstack/pull/9844#discussion_r1871603847 ## server/src/main/java/com/cloud/resource/RollingMaintenanceManagerImpl.java: ## @@ -666,11 +682,36 @@ private Pair performCapacityChecksBeforeHostInMaintenan

Re: [PR] api: fix access for listSystemVmUsageHistory [cloudstack]

2024-12-05 Thread via GitHub
scottsignal commented on PR #10032: URL: https://github.com/apache/cloudstack/pull/10032#issuecomment-2520973902 @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 com

Re: [I] LInk Account to LDAP Improvements [cloudstack]

2024-12-05 Thread via GitHub
scottsignal commented on issue #10030: URL: https://github.com/apache/cloudstack/issues/10030#issuecomment-2520952021 > So remaining are two improvement requests > > ad 2; this to be put behind a feature flag: As an operator I want to have the ability to disable the manual creation of

Re: [PR] api: fix access for listSystemVmUsageHistory [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10032: URL: https://github.com/apache/cloudstack/pull/10032#issuecomment-2520975650 @scottsignal 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 respon

Re: [PR] api: fix access for listSystemVmUsageHistory [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10032: URL: https://github.com/apache/cloudstack/pull/10032#issuecomment-2521031742 UI build: :heavy_check_mark: Live QA URL: https://qa.cloudstack.cloud/simulator/pr/10032 (QA-JID-496) -- This is an automated message from the Apache Git Service. To respon

Re: [I] MariaDB schema upgrade fails for CloudStack 4.20.0.0 fresh installation on EL9 [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland closed issue #10048: MariaDB schema upgrade fails for CloudStack 4.20.0.0 fresh installation on EL9 URL: https://github.com/apache/cloudstack/issues/10048 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] Fix assign instance translation issue [cloudstack]

2024-12-05 Thread via GitHub
BryanMLima commented on PR #10052: URL: https://github.com/apache/cloudstack/pull/10052#issuecomment-2520444266 @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] UI: Fix assign instance translation issue [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10052: URL: https://github.com/apache/cloudstack/pull/10052#issuecomment-2520491862 UI build: :heavy_check_mark: Live QA URL: https://qa.cloudstack.cloud/simulator/pr/10052 (QA-JID-495) -- This is an automated message from the Apache Git Service. To respon

Re: [PR] Support for Management Server Maintenance [cloudstack]

2024-12-05 Thread via GitHub
vladimirpetrov commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2520688357 @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] Support for Management Server Maintenance [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2520691545 @vladimirpetrov 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: [I] Fix com.cloud.server.ManagementServerImpl#listStoragePoolsForSystemMigrationOfVolume [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on issue #10021: URL: https://github.com/apache/cloudstack/issues/10021#issuecomment-2520599165 @JoaoJandre, - are you fixing this? - is this serious? - is this easy to fix? i.e. - is this going to be feasible for 4.19.2? -- This is an automated messag

(cloudstack-cloudmonkey) 01/01: List free IPs when performing associate IPAddress operation

2024-12-05 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a commit to branch list-free-ips-associate-ips in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git commit 8df6018686359fa8af7106991bd364e99e3b906f Author: Pearl Dsilva AuthorDate: Thu D

(cloudstack-cloudmonkey) branch list-free-ips-associate-ips created (now 8df6018)

2024-12-05 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch list-free-ips-associate-ips in repository https://gitbox.apache.org/repos/asf/cloudstack-cloudmonkey.git at 8df6018 List free IPs when performing associate IPAddress operation This br

Re: [PR] Add parameter to not create additional users on `cloudstack-setup-databases` [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #9969: URL: https://github.com/apache/cloudstack/pull/9969#issuecomment-2520416387 @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] Add parameter to not create additional users on `cloudstack-setup-databases` [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9969: URL: https://github.com/apache/cloudstack/pull/9969#issuecomment-2520420516 @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 assign instance translation issue [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10052: URL: https://github.com/apache/cloudstack/pull/10052#issuecomment-2520445902 @BryanMLima 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] Certificate and VM hostname validation improvements [cloudstack]

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

Re: [PR] Bug fix : add check befor fencing [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10039: URL: https://github.com/apache/cloudstack/pull/10039#issuecomment-2520465753 > @DaanHoogland Thanks for your help! Yes, I really want to use this in our environment which is 4.19.1.2. But i found that 'log' in 4.19 is 'LOG', and in 4.20 it is 'logger'. D

Re: [PR] Certificate and VM hostname validation improvements [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on code in PR #10051: URL: https://github.com/apache/cloudstack/pull/10051#discussion_r1871483426 ## utils/src/main/java/com/cloud/utils/net/NetUtils.java: ## @@ -99,6 +99,10 @@ public class NetUtils { public final static int IPV6_EUI64_11TH_BYTE = -1

Re: [PR] Add parameter to not create additional users on `cloudstack-setup-databases` [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9969: URL: https://github.com/apache/cloudstack/pull/9969#issuecomment-2520439045 [SF] Trillian Build Failed (tid-11858) -- 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 for Management Server Maintenance [cloudstack]

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

[PR] Fix assign instance translation issue [cloudstack]

2024-12-05 Thread via GitHub
BryanMLima opened a new pull request, #10052: URL: https://github.com/apache/cloudstack/pull/10052 ### Description This PR addresses an issue when trying to assign an instance to another account/project when using ACS in another language besides English. ### Types of changes

Re: [PR] Add access modifier to `VolumeVO` [cloudstack]

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

Re: [PR] UI: Fix assign instance translation issue [cloudstack]

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

Re: [PR] Add access modifier to `VolumeVO` [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #9394: URL: https://github.com/apache/cloudstack/pull/9394#issuecomment-2520456380 @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] upgrade: consider multiple hypervisors and secondary storages [cloudstack]

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

Re: [PR] Add access modifier to `VolumeVO` [cloudstack]

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

[I] console not working after system vm has been recreated on a different cluster [cloudstack]

2024-12-05 Thread via GitHub
sbrueseke opened a new issue, #10050: URL: https://github.com/apache/cloudstack/issues/10050 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ Console Proxy ~~~ # CLOUDSTACK VERSION ~~~ 4.19.1.2 ~~~ # CONFIGURATIO

Re: [PR] Introducing granular command timeouts global setting [cloudstack]

2024-12-05 Thread via GitHub
shwstppr commented on code in PR #9659: URL: https://github.com/apache/cloudstack/pull/9659#discussion_r1871289972 ## engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java: ## @@ -424,15 +428,78 @@ private void setEmptyAnswers(final Commands commands,

Re: [PR] [WIP] Rename ambiguous global setting: enable.kvm.host.auto.enable.disable [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on code in PR #10042: URL: https://github.com/apache/cloudstack/pull/10042#discussion_r1870869785 ## engine/schema/src/main/resources/META-INF/db/schema-41910to42000.sql: ## @@ -425,3 +425,5 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hyper

Re: [PR] server, plugin: enhance storage stats for IOPS [cloudstack]

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

Re: [PR] set ulimit for server according to redhat spec [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache commented on PR #10040: URL: https://github.com/apache/cloudstack/pull/10040#issuecomment-2519560474 > @weizhouapache I think the best value is installation dependant. If it works on debian-likes, there is nothing to do there. This was reported by a user on centos7 and does no

Re: [PR] server, plugin: enhance storage stats for IOPS [cloudstack]

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

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

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9817: URL: https://github.com/apache/cloudstack/pull/9817#issuecomment-2519514027 @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] [HEALTH] 21/main Health Check, please don't merge this! [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #9817: URL: https://github.com/apache/cloudstack/pull/9817#issuecomment-2519512192 @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] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

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

Re: [PR] Bug fix : add check befor fencing [cloudstack]

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

Re: [PR] Support for Management Server Maintenance [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2520255785 @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] Support for Management Server Maintenance [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on PR #9854: URL: https://github.com/apache/cloudstack/pull/9854#issuecomment-2520250026 @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] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

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

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1871049690 ## engine/schema/src/main/java/com/cloud/storage/dao/BucketDaoImpl.java: ## @@ -79,4 +98,21 @@ public List searchByIds(Long[] ids) { sc.setParamete

Re: [PR] SAML2: add cookie with HttpOnly too #10013 [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10047: URL: https://github.com/apache/cloudstack/pull/10047#issuecomment-2519856955 @kiranchavala , maybe we can look at testing this together? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1871077159 ## server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java: ## @@ -502,13 +577,61 @@ public boolean createBackup(final Long vmId) {

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10045: URL: https://github.com/apache/cloudstack/pull/10045#issuecomment-2519861992 @blueorangutan test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] Remove SNI option in _run.sh, as it is correct as default. [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10028: URL: https://github.com/apache/cloudstack/pull/10028#issuecomment-2519860073 @blueorangutan test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] KVM/s390x Support: Add support for KVM on s390x architecture [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache commented on code in PR #10038: URL: https://github.com/apache/cloudstack/pull/10038#discussion_r1871154421 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java: ## @@ -247,8 +247,8 @@ public class LibvirtComputin

Re: [PR] KVM/s390x Support: Add support for KVM on s390x architecture [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache commented on PR #10038: URL: https://github.com/apache/cloudstack/pull/10038#issuecomment-2519960567 @niyamsw `"s390x".equals(System.getProperty("os.arch")` is used in several lines, we can replace them. it checks if the host is s390x, is it same as `isGuestS390x` ?

Re: [PR] Add parameter to not create additional users on `cloudstack-setup-databases` [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9969: URL: https://github.com/apache/cloudstack/pull/9969#issuecomment-2519981352 [SF] Trillian Build Failed (tid-11852) -- 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] SAML2: add cookie with HttpOnly too #10013 [cloudstack]

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

Re: [PR] Remove SNI option in _run.sh, as it is correct as default. [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #10028: URL: https://github.com/apache/cloudstack/pull/10028#issuecomment-2519620384 @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] SAML2: add cookie with HttpOnly too #10013 [cloudstack]

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

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1871052364 ## engine/schema/src/main/java/org/apache/cloudstack/backup/dao/BackupDaoImpl.java: ## @@ -142,6 +168,31 @@ public List syncBackups(Long zoneId, Long vmId,

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1871048325 ## engine/schema/src/main/java/com/cloud/storage/dao/BucketDaoImpl.java: ## @@ -79,4 +98,21 @@ public List searchByIds(Long[] ids) { sc.setParamete

(cloudstack) branch dependabot/maven/org.owasp.esapi-esapi-2.6.0.0 deleted (was e3a9e91c094)

2024-12-05 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/maven/org.owasp.esapi-esapi-2.6.0.0 in repository https://gitbox.apache.org/repos/asf/cloudstack.git was e3a9e91c094 Bump org.owasp.esapi:esapi from 2.1.0.1 to 2.6.0.0 The r

Re: [PR] upgrade: consider multiple hypervisors and secondary storages [cloudstack]

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

Re: [PR] upgrade: consider multiple hypervisors and secondary storages [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10046: URL: https://github.com/apache/cloudstack/pull/10046#issuecomment-2519842343 @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 progr

Re: [PR] Add access modifier to `VolumeVO` [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on PR #9394: URL: https://github.com/apache/cloudstack/pull/9394#issuecomment-2519919771 @FelipeM525 @JoaoJandre , I did not investigate deeply but systemVMs do not start in this env: ``` 2024-12-05 10:28:41,870 ERROR [c.c.v.VmWorkJobDispatcher] (Work-Job-Execut

Re: [PR] Bump org.owasp.esapi:esapi from 2.1.0.1 to 2.6.0.0 [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland closed pull request #9980: Bump org.owasp.esapi:esapi from 2.1.0.1 to 2.6.0.0 URL: https://github.com/apache/cloudstack/pull/9980 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the spe

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1871060030 ## server/src/main/java/com/cloud/configuration/Config.java: ## @@ -1415,6 +1447,8 @@ public enum Config { DefaultMaxDomainPublicIPs("Domain Defaults",

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1871061131 ## server/src/main/java/com/cloud/configuration/Config.java: ## @@ -1423,6 +1457,9 @@ public enum Config { DefaultMaxDomainPrimaryStorage("Domain Defau

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

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

Re: [PR] Remove SNI option in _run.sh, as it is correct as default. [cloudstack]

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

[I] NAS BACKUP PLUGIN ERROR [cloudstack]

2024-12-05 Thread via GitHub
fermosan opened a new issue, #10049: URL: https://github.com/apache/cloudstack/issues/10049 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ NAS BACKUP PLUGIN ~~~ # CLOUDSTACK VERSION ~~~ 4.20 ~~~ # CONFIGURATION

Re: [PR] SAML2: add cookie with HttpOnly too #10013 [cloudstack]

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

Re: [PR] Support for Management Server Maintenance [cloudstack]

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

Re: [PR] Create API to list Quota credits [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9590: URL: https://github.com/apache/cloudstack/pull/9590#issuecomment-2521161307 @winterhazel 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] Create API to list Quota credits [cloudstack]

2024-12-05 Thread via GitHub
winterhazel commented on PR #9590: URL: https://github.com/apache/cloudstack/pull/9590#issuecomment-2521158527 @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 improvements [cloudstack]

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

Re: [PR] Add access modifier to `VolumeVO` [cloudstack]

2024-12-05 Thread via GitHub
weizhouapache merged PR #9394: URL: https://github.com/apache/cloudstack/pull/9394 -- 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...@c

(cloudstack) branch 4.20 updated: engine/schema: Add access modifier to `VolumeVO` (#9394)

2024-12-05 Thread weizhou
This is an automated email from the ASF dual-hosted git repository. weizhou pushed a commit to branch 4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.20 by this push: new 810c4108c7b engine/schema: Add access modifier

Re: [PR] UI improvements [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #9773: URL: https://github.com/apache/cloudstack/pull/9773#issuecomment-2522309419 @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 access modifier to `VolumeVO` [cloudstack]

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

Re: [PR] Fix NPE issues during host rolling maintenance, due to host tags and custom constrained/unconstrained service offering [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #9844: URL: https://github.com/apache/cloudstack/pull/9844#discussion_r1872703053 ## server/src/main/java/com/cloud/resource/RollingMaintenanceManagerImpl.java: ## @@ -666,11 +682,36 @@ private Pair performCapacityChecksBeforeHostInMainten

Re: [PR] server, plugin: enhance storage stats for IOPS [cloudstack]

2024-12-05 Thread via GitHub
shwstppr commented on PR #10034: URL: https://github.com/apache/cloudstack/pull/10034#issuecomment-2522264359 @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] server, plugin: enhance storage stats for IOPS [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10034: URL: https://github.com/apache/cloudstack/pull/10034#issuecomment-2522265895 @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] Bug fix : add check befor fencing [cloudstack]

2024-12-05 Thread via GitHub
leo79901 commented on PR #10039: URL: https://github.com/apache/cloudstack/pull/10039#issuecomment-2522008955 Thanks really for your help. I have run the command. It's my great honor to contribute to the project. I will do more within my capacity in the future. -- This is an aut

Re: [I] LInk Account to LDAP Improvements [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on issue #10030: URL: https://github.com/apache/cloudstack/issues/10030#issuecomment-2519527773 So remaining are two improvement requests ad 2; this to be put behind a feature flag: As an operator I want to have the ability to disable the manual creation of nati

Re: [PR] server, plugin: enhance storage stats for IOPS [cloudstack]

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

Re: [I] Ceph RGW Object Storage couldn't be added (Invalid credentials or URL) [cloudstack]

2024-12-05 Thread via GitHub
hanisirfan commented on issue #10044: URL: https://github.com/apache/cloudstack/issues/10044#issuecomment-2519534947 Just created a new RGW user using the Dashboard, but this time it's not under a tenant. Still the same error 😞 -- This is an automated message from the Apache Git Service.

Re: [I] listSystemVmsUsageHistory is available to Domain Admin by Default [cloudstack]

2024-12-05 Thread via GitHub
DaanHoogland commented on issue #10031: URL: https://github.com/apache/cloudstack/issues/10031#issuecomment-2519623030 @scottsignal , @shwstppr made a fix in #10032 . Can you test this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

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

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

Re: [PR] Allow config drive deletion of migrated VM, on host maintenance [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10045: URL: https://github.com/apache/cloudstack/pull/10045#issuecomment-2519637469 @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 prog

Re: [PR] server, plugin: enhance storage stats for IOPS [cloudstack]

2024-12-05 Thread via GitHub
blueorangutan commented on PR #10034: URL: https://github.com/apache/cloudstack/pull/10034#issuecomment-2519716604 [SF] Trillian Build Failed (tid-11850) -- 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

(cloudstack) branch 4.20 updated (0eb94a13962 -> 4c801e24f44)

2024-12-05 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch 4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 0eb94a13962 move upgrade code to the right file (#10036) add 4c801e24f44 Fix Create dropdown menu not changing la

Re: [PR] Add Resource Limits to Backups and Object Storage [cloudstack]

2024-12-05 Thread via GitHub
sureshanaparti commented on code in PR #10017: URL: https://github.com/apache/cloudstack/pull/10017#discussion_r1870981463 ## api/src/main/java/org/apache/cloudstack/api/command/user/bucket/UpdateBucketCmd.java: ## @@ -56,7 +56,7 @@ public class UpdateBucketCmd extends BaseCmd {

  1   2   >