(cloudstack-www) branch security-limitation updated (afe78dca -> bb2b9845)

2024-07-23 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch security-limitation in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git from afe78dca security update add bb2b9845 Update src/pages/security.md No new revisions were adde

(cloudstack-www) branch security-limitation created (now afe78dca)

2024-07-23 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch security-limitation in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git at afe78dca security update This branch includes the following new commits: new afe78dca securi

(cloudstack-www) 01/01: security update

2024-07-23 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch security-limitation in repository https://gitbox.apache.org/repos/asf/cloudstack-www.git commit afe78dcab73ee84973c63b680f9fb3ab909e5942 Author: Daan Hoogland AuthorDate: Wed Jul 24 08:43:54 2024 +

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud commented on code in PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#discussion_r1689173861 ## tools/appliance/systemvmtemplate/template-base_x86_64-target_aarch64.json: ## @@ -0,0 +1,91 @@ +{ + "_license": "Apache License 2.0", + "builders": [ +

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud commented on code in PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#discussion_r1689172919 ## tools/appliance/systemvmtemplate/template-base_x86_64-target_aarch64.json: ## @@ -0,0 +1,91 @@ +{ + "_license": "Apache License 2.0", + "builders": [ +

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
StepBee commented on code in PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#discussion_r1689165052 ## tools/appliance/systemvmtemplate/template-base_x86_64-target_aarch64.json: ## @@ -0,0 +1,91 @@ +{ + "_license": "Apache License 2.0", + "builders": [ +{ +

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
StepBee commented on code in PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#discussion_r1689165052 ## tools/appliance/systemvmtemplate/template-base_x86_64-target_aarch64.json: ## @@ -0,0 +1,91 @@ +{ + "_license": "Apache License 2.0", + "builders": [ +{ +

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
StepBee commented on code in PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#discussion_r1689165052 ## tools/appliance/systemvmtemplate/template-base_x86_64-target_aarch64.json: ## @@ -0,0 +1,91 @@ +{ + "_license": "Apache License 2.0", + "builders": [ +{ +

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud commented on PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#issuecomment-2246941288 Thanks for replying @StepBee I've posted a question above. I'm trying to have the Jenkins job used to x86 systemvmtemplate build also build for aarch64, where I'm getting this

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
StepBee commented on PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#issuecomment-2246935292 @rohityadavcloud Thanks for picking this one up! Really great to see building arm64 systemvms is making it's way into cloudstack. Please let me know in case i can support some

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud commented on code in PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#discussion_r1689159278 ## tools/appliance/systemvmtemplate/template-base_x86_64-target_aarch64.json: ## @@ -0,0 +1,91 @@ +{ + "_license": "Apache License 2.0", + "builders": [ +

Re: [I] Reordering ACL rules inside the ACL list - does not apply the rule order inside the VR [cloudstack]

2024-07-23 Thread via GitHub
s-novakovic commented on issue #9398: URL: https://github.com/apache/cloudstack/issues/9398#issuecomment-2246928286 I've managed to rest this on the RC2/final 4.19.1.0 version - and the issue is still there. It seems that only the DB records get updated (the order of rules for the ACL list)

Re: [PR] test: improve purge expunged resources b/g task testcase [cloudstack]

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

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
andrijapanicsb commented on PR #9437: URL: https://github.com/apache/cloudstack/pull/9437#issuecomment-2246909203 Nice! -- 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 u

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

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

Re: [PR] test: improve purge expunged resources b/g task testcase [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9426: URL: https://github.com/apache/cloudstack/pull/9426#issuecomment-2246900208 [SF] Trillian test result (tid-10952) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 54988 seconds Marvin logs: https://github.c

Re: [PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud closed pull request #9437: Add systemvmtemplate arm64 support URL: https://github.com/apache/cloudstack/pull/9437 -- 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.

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#issuecomment-2246862685 [SF] Trillian test result (tid-10944) Environment: vmware-80 (x2), Advanced Networking with Mgmt server 7 Total time taken: 73848 seconds Marvin logs: https://github.com/

[PR] Add systemvmtemplate arm64 support [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud opened a new pull request, #9437: URL: https://github.com/apache/cloudstack/pull/9437 This add support for building arm64 systemvmtemplate, which is basically branch of https://github.com/StepBee/cloudstack/tree/add_systemvmtemplate_arm64_support and work by @StepBee

Re: [PR] Support user resource name / displaytext with emoji, unicode chars, and some sql exception msg improvements [cloudstack]

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

Re: [PR] Fixup CKS UI for external managed clusters [cloudstack]

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

Re: [PR] Fix link to removed volumes being shown in info card and list view [cloudstack]

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

Re: [PR] fix removeSecondaryStorageSelector response for docs [cloudstack]

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

Re: [PR] [4.20] VR: fix issue if userdata is binary data [cloudstack]

2024-07-23 Thread via GitHub
weizhouapache commented on PR #8739: URL: https://github.com/apache/cloudstack/pull/8739#issuecomment-2246414808 > just noticed I didn't report back here; tests ok: > ![image](https://github.com/user-attachments/assets/3fd39690-e83d-46e8-a8d0-6f411082ab3f) > used blApache to make sure

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

2024-07-23 Thread via GitHub
hsato03 commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2246360621 > @hsato03 seeing `2024-07-23T18:43:54,232 INFO [c.c.a.ApiServlet] (qtp253011924-22:[ctx-35b7f332, ctx-24c715d9]) (logid:0c5ca90f) (userId=2 accountId=2 sessionId=node048rydzt ycoujeea

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

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

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

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

Re: [PR] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2246151347 @vishesh92 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] API: Fix missing keys in listZonesMetrics response [cloudstack]

2024-07-23 Thread via GitHub
vishesh92 commented on PR #9419: URL: https://github.com/apache/cloudstack/pull/9419#issuecomment-2246150176 @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 co

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

2024-07-23 Thread via GitHub
blueorangutan commented on PR #8743: URL: https://github.com/apache/cloudstack/pull/8743#issuecomment-2246023180 @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 `deleteAccount` API to prevent deletion of the caller [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #8743: URL: https://github.com/apache/cloudstack/pull/8743#issuecomment-2246015368 @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-07-23 Thread via GitHub
DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2246004483 @hsato03 seeing `2024-07-23T18:43:54,232 INFO [c.c.a.ApiServlet] (qtp253011924-22:[ctx-35b7f332, ctx-24c715d9]) (logid:0c5ca90f) (userId=2 accountId=2 sessionId=node048rydzt y

Re: [PR] [4.20] VR: fix issue if userdata is binary data [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #8739: URL: https://github.com/apache/cloudstack/pull/8739#issuecomment-2245962000 just noticed I didn't report back here; tests ok: ![image](https://github.com/user-attachments/assets/3fd39690-e83d-46e8-a8d0-6f411082ab3f) used blApache to make sure nothing

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
sureshanaparti commented on code in PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#discussion_r1688466265 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -4650,16 +4650,25 @@ private void updateVMDiskController(UserVmVO vm, Map customParam

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
sureshanaparti commented on code in PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#discussion_r1688466265 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -4650,16 +4650,25 @@ private void updateVMDiskController(UserVmVO vm, Map customParam

Re: [I] Snapshot usage reported incorrectly when number of Daily Snapshot less than the snapshot.delta.max [cloudstack]

2024-07-23 Thread via GitHub
nxsbi commented on issue #9416: URL: https://github.com/apache/cloudstack/issues/9416#issuecomment-2245853426 Hello, Yes the storage.cleanup.enabled = True You are correct in that the snapshot is marked as Destroyed, but snapshot_store_ref still shows it as Ready. But keep in mind tha

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
winterhazel commented on code in PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#discussion_r1688382930 ## server/src/main/java/com/cloud/vm/UserVmManagerImpl.java: ## @@ -4650,16 +4650,25 @@ private void updateVMDiskController(UserVmVO vm, Map customParam

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

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

Re: [PR] ui: vm metrics note about behaviour across hypervisors [cloudstack]

2024-07-23 Thread via GitHub
nvazquez commented on PR #9399: URL: https://github.com/apache/cloudstack/pull/9399#issuecomment-2245398995 @shwstppr no problem, LGTM -- 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] test: improve purge expunged resources b/g task testcase [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9426: URL: https://github.com/apache/cloudstack/pull/9426#issuecomment-2245252148 [SF] Trillian Build Failed (tid-10951) -- 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] test: improve purge expunged resources b/g task testcase [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9426: URL: https://github.com/apache/cloudstack/pull/9426#issuecomment-2245247451 @DaanHoogland a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests -- This i

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

2024-07-23 Thread via GitHub
blueorangutan commented on PR #8743: URL: https://github.com/apache/cloudstack/pull/8743#issuecomment-2245247616 @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] test: improve purge expunged resources b/g task testcase [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #9426: URL: https://github.com/apache/cloudstack/pull/9426#issuecomment-2245245422 @blueorangutan test matrix -- 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] Fix `deleteAccount` API to prevent deletion of the caller [cloudstack]

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

(cloudstack-kubernetes-provider) branch support-acl updated (98fbfc55 -> 4aed034c)

2024-07-23 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch support-acl in repository https://gitbox.apache.org/repos/asf/cloudstack-kubernetes-provider.git discard 98fbfc55 skip adding network ACLs if network tier uses default ACL add 4aed034c

Re: [PR] test: improve purge expunged resources b/g task testcase [cloudstack]

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

Re: [PR] Fixup CKS UI for external managed clusters [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9434: URL: https://github.com/apache/cloudstack/pull/9434#issuecomment-2245188968 @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] Support user resource name / displaytext with emoji, unicode chars, and some sql exception msg improvements [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on code in PR #9340: URL: https://github.com/apache/cloudstack/pull/9340#discussion_r1687636615 ## engine/schema/src/main/resources/META-INF/db/schema-41910to42000.sql: ## @@ -151,6 +151,55 @@ WHERE name IN ("quota.usage.smtp.useStartTLS", "quota.usag

Re: [PR] Fixup CKS UI for external managed clusters [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #9434: URL: https://github.com/apache/cloudstack/pull/9434#issuecomment-2245183817 @blueorangutan test alma9 kvm-alma9 keepEnv -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

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

2024-07-23 Thread via GitHub
bernardodemarco commented on code in PR #8743: URL: https://github.com/apache/cloudstack/pull/8743#discussion_r1687972014 ## server/src/test/java/com/cloud/user/AccountManagerImplTest.java: ## @@ -198,6 +198,38 @@ public void deleteUserAccountCleanup() { Mockito.verify(

Re: [PR] Fix link to removed volumes being shown in info card and list view [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #8833: URL: https://github.com/apache/cloudstack/pull/8833#issuecomment-2245113378 [SF] Trillian Build Failed (tid-10948) -- 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] Fix link to removed volumes being shown in info card and list view [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #8833: URL: https://github.com/apache/cloudstack/pull/8833#issuecomment-2245048521 @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] Fix link to removed volumes being shown in info card and list view [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #8833: URL: https://github.com/apache/cloudstack/pull/8833#issuecomment-2245043918 @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] Support user resource name / displaytext with emoji, unicode chars, and some sql exception msg improvements [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9340: URL: https://github.com/apache/cloudstack/pull/9340#issuecomment-2245032505 @rohityadavcloud 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. T

Re: [PR] Support user resource name / displaytext with emoji, unicode chars, and some sql exception msg improvements [cloudstack]

2024-07-23 Thread via GitHub
rohityadavcloud commented on PR #9340: URL: https://github.com/apache/cloudstack/pull/9340#issuecomment-2245030208 @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] test: improve purge expunged resources b/g task testcase [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #9426: URL: https://github.com/apache/cloudstack/pull/9426#issuecomment-2245020148 @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] test: improve purge expunged resources b/g task testcase [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9426: URL: https://github.com/apache/cloudstack/pull/9426#issuecomment-2245022291 @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] Fixup CKS UI for external managed clusters [cloudstack]

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

Re: [PR] Enhance the `listAffinityGroups` API by adding the dedicated resources related to an affinity group [cloudstack]

2024-07-23 Thread via GitHub
bernardodemarco commented on PR #9188: URL: https://github.com/apache/cloudstack/pull/9188#issuecomment-2244984069 > ok @bernardodemarco , this means that listing will show the dedicated resources, even if the caller has no access to them, right? (that was what I was asking) Actu

[I] Some java classes use "java.security.InvalidParameterException" which should be "com.cloud.exception.InvalidParameterValueException" [cloudstack]

2024-07-23 Thread via GitHub
weizhouapache opened a new issue, #9436: URL: https://github.com/apache/cloudstack/issues/9436 just for cleanup # ISSUE TYPE * Bug Report * Improvement Request * Enhancement Request * Feature Idea * Documentation Report * Other # COMPONEN

Re: [PR] Fixup CKS UI for external managed clusters [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9434: URL: https://github.com/apache/cloudstack/pull/9434#issuecomment-2244863282 @vishesh92 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] Fixup CKS UI for external managed clusters [cloudstack]

2024-07-23 Thread via GitHub
vishesh92 commented on PR #9434: URL: https://github.com/apache/cloudstack/pull/9434#issuecomment-2244856018 @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 co

Re: [PR] Fix root disk resize (don't allow) when service offering has root disk size, only allow through service offering change [cloudstack]

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

Re: [PR] Fix link to removed volumes being shown in info card and list view [cloudstack]

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

Re: [PR] NSX: add back removed code for NSX [cloudstack]

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

Re: [PR] Fixup CKS UI for external managed clusters [cloudstack]

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

[PR] NSX: add back removed code for NSX [cloudstack]

2024-07-23 Thread via GitHub
weizhouapache opened a new pull request, #9435: URL: https://github.com/apache/cloudstack/pull/9435 ### Description The code snippet has been removed in the merge forward of PR #9240 in commit 90fe1d5fdcc2909130b5de6803b57feeff194f54 This PR adds the code back.

[PR] Fixup CKS UI for external managed clusters [cloudstack]

2024-07-23 Thread via GitHub
vishesh92 opened a new pull request, #9434: URL: https://github.com/apache/cloudstack/pull/9434 ### Description This PR fixes #9291 #9293 ### Types of changes - [ ] Breaking change (fix or feature that would cause existi

Re: [PR] Fix link to removed volumes being shown in info card and list view [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #8833: URL: https://github.com/apache/cloudstack/pull/8833#issuecomment-2244581886 @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 link to removed volumes being shown in info card and list view [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #8833: URL: https://github.com/apache/cloudstack/pull/8833#issuecomment-2244578381 @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 user resource name / displaytext with emoji, unicode chars, and some sql exception msg improvements [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on code in PR #9340: URL: https://github.com/apache/cloudstack/pull/9340#discussion_r1687636615 ## engine/schema/src/main/resources/META-INF/db/schema-41910to42000.sql: ## @@ -151,6 +151,55 @@ WHERE name IN ("quota.usage.smtp.useStartTLS", "quota.usag

Re: [PR] fix removeSecondaryStorageSelector response for docs [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9264: URL: https://github.com/apache/cloudstack/pull/9264#issuecomment-2244557077 @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] AS Numbers Management [cloudstack]

2024-07-23 Thread via GitHub
weizhouapache commented on code in PR #9410: URL: https://github.com/apache/cloudstack/pull/9410#discussion_r1687585454 ## engine/schema/src/main/resources/META-INF/db/schema-41910to42000.sql: ## @@ -154,3 +154,47 @@ WHERE -- Quota inject tariff result into subsequent ones C

Re: [PR] fix removeSecondaryStorageSelector response for docs [cloudstack]

2024-07-23 Thread via GitHub
DaanHoogland commented on PR #9264: URL: https://github.com/apache/cloudstack/pull/9264#issuecomment-2244553874 @blueorangutan test alma9 kvm-alma9 keepEnv -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

Re: [PR] Enable Security groups for Shared networks in Advanced zone without security groups [cloudstack]

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

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#issuecomment-2244501492 @sureshanaparti a [SL] Trillian-Jenkins test job (centos7 mgmt + vmware-80) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To r

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
sureshanaparti commented on PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#issuecomment-2244499160 @blueorangutan test centos7 vmware-80 -- 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] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
blueorangutan commented on PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#issuecomment-2244497835 @sureshanaparti [SL] I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test [mgmt os] [hypervisor] [keepEnv] [qemuEv] [basicZone|secur

Re: [PR] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

2024-07-23 Thread via GitHub
sureshanaparti commented on PR #9433: URL: https://github.com/apache/cloudstack/pull/9433#issuecomment-2244493840 @blueorangutan help -- 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] [VMware] Update data disk controller same as the root disk controller type when it is not set in the VM detail [cloudstack]

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

Re: [I] Snapshot usage reported incorrectly when number of Daily Snapshot less than the snapshot.delta.max [cloudstack]

2024-07-23 Thread via GitHub
sureshanaparti commented on issue #9416: URL: https://github.com/apache/cloudstack/issues/9416#issuecomment-2244450538 It seems the snapshots in the chain are marked as destroyed, but not removed. These will be removed by the storage garbage collector. So, maybe these are not counted in usa

Re: [PR] AS Numbers Management [cloudstack]

2024-07-23 Thread via GitHub
weizhouapache commented on code in PR #9410: URL: https://github.com/apache/cloudstack/pull/9410#discussion_r1687132627 ## server/src/main/java/com/cloud/network/vpc/VpcManagerImpl.java: ## @@ -266,6 +273,12 @@ public class VpcManagerImpl extends ManagerBase implements VpcManag

Re: [PR] Add support for Ceph RGW Object Store [cloudstack]

2024-07-23 Thread via GitHub
wido commented on code in PR #8389: URL: https://github.com/apache/cloudstack/pull/8389#discussion_r1687527655 ## utils/src/test/java/com/cloud/utils/UriUtilsTest.java: ## @@ -273,4 +273,16 @@ public void testIsUrlForCompressedFile() { Assert.assertTrue(UriUtils.isUrlF