(cloudstack) branch dummy updated (d7c90f21d03 -> b224003a520)

2023-12-20 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch dummy in repository https://gitbox.apache.org/repos/asf/cloudstack.git discard d7c90f21d03 do not merge add 969e094419d server: improve stats collector logs to state what the collector does

Re: [PR] CleanUp Async Jobs after mgmt server maintenance [cloudstack]

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

Re: [I] Cloudstack virtual machines can't communicate with virtual routers when they are on different hosts [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache closed issue #8360: Cloudstack virtual machines can't communicate with virtual routers when they are on different hosts URL: https://github.com/apache/cloudstack/issues/8360 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] Cloudstack virtual machines can't communicate with virtual routers when they are on different hosts [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on issue #8360: URL: https://github.com/apache/cloudstack/issues/8360#issuecomment-1865809774 thanks for the update @herdiansah571 -- 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] Normalize loggers and upgrade log4j 1.2 to log4j 2.19 [cloudstack]

2023-12-20 Thread via GitHub
github-actions[bot] commented on PR #7131: URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1865809457 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache

(cloudstack) branch 4.18-dummy updated (a361982334e -> fa1c05ca49c)

2023-12-20 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch 4.18-dummy in repository https://gitbox.apache.org/repos/asf/cloudstack.git discard a361982334e don't merge add 969e094419d server: improve stats collector logs to state what the collector

(cloudstack) branch main updated (1411da1a22b -> 6d916cad348)

2023-12-20 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 1411da1a22b Add e2e tests for listing resources (#8281) add 969e094419d server: improve stats collector logs to

(cloudstack) 01/01: Merge remote-tracking branch 'origin/4.18'

2023-12-20 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 6d916cad348f5833a567c17f5c9dbccaf2135448 Merge: 1411da1a22b 969e094419d Author: Rohit Yadav AuthorDate: Thu Dec 21

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

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

(cloudstack) branch 4.18 updated: server: improve stats collector logs to state what the collector does (#8387)

2023-12-20 Thread shwstppr
This is an automated email from the ASF dual-hosted git repository. shwstppr pushed a commit to branch 4.18 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.18 by this push: new 969e094419d server: improve stats collector

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
shwstppr merged PR #8387: URL: https://github.com/apache/cloudstack/pull/8387 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1865771509 @harikrishna-patnala 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

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
harikrishna-patnala commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1865769600 @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

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865745815 @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 test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
vishesh92 commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865744848 @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] CleanUp Async Jobs after mgmt server maintenance [cloudstack]

2023-12-20 Thread via GitHub
harikrishna-patnala commented on code in PR #8394: URL: https://github.com/apache/cloudstack/pull/8394#discussion_r1433619783 ## framework/jobs/src/main/java/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java: ## @@ -35,12 +35,22 @@ import javax.inject.Inject;

Re: [PR] CleanUp Async Jobs after mgmt server maintenance [cloudstack]

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

Re: [PR] CleanUp Async Jobs after mgmt server maintenance [cloudstack]

2023-12-20 Thread via GitHub
harikrishna-patnala commented on PR #8394: URL: https://github.com/apache/cloudstack/pull/8394#issuecomment-1865646801 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865641034 @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] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
shwstppr commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865639667 @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] CleanUp Async Jobs after mgmt server maintenance [cloudstack]

2023-12-20 Thread via GitHub
codecov[bot] commented on PR #8394: URL: https://github.com/apache/cloudstack/pull/8394#issuecomment-1865616137 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8394?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

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

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8387: URL: https://github.com/apache/cloudstack/pull/8387#issuecomment-1865602079 [SF] Trillian test result (tid-8649) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 39924 seconds Marvin logs:

[PR] CleanUp Async Jobs after mgmt server maintenance [cloudstack]

2023-12-20 Thread via GitHub
kishankavala opened a new pull request, #8394: URL: https://github.com/apache/cloudstack/pull/8394 ### Description This PR fixes moves resources stuck in transition state during async job cleanup Problem: During maintenance of the management server, other servers in the

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
sureshanaparti commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865528461 @vishesh92 can you update the description (about the issue, fix) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
sureshanaparti commented on code in PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#discussion_r1433548007 ## test/integration/smoke/test_list_volumes.py: ## @@ -140,6 +140,26 @@ def setUpClass(cls): def tearDownClass(cls): super(TestListVolumes,

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1865516551 [SF] Trillian test result (tid-8648) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 40634 seconds Marvin logs:

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
codecov[bot] commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865505110 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8392?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

[I] Linstor primary storages are not created for disabled hosts [cloudstack]

2023-12-20 Thread via GitHub
rajujith opened a new issue, #8393: URL: https://github.com/apache/cloudstack/issues/8393 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ primary storage Linstor ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1 ~~~ #

Re: [PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865499296 @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 test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
vishesh92 commented on PR #8392: URL: https://github.com/apache/cloudstack/pull/8392#issuecomment-1865498408 @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

[PR] Fixup test_list_volumes.py failure [cloudstack]

2023-12-20 Thread via GitHub
vishesh92 opened a new pull request, #8392: URL: https://github.com/apache/cloudstack/pull/8392 ### Description This PR... ### Types of changes - [ ] Breaking change (fix or feature that would cause existing

Re: [I] Management server metrics only shows on 1 GUI at a time [cloudstack]

2023-12-20 Thread via GitHub
rajujith commented on issue #8235: URL: https://github.com/apache/cloudstack/issues/8235#issuecomment-1865489136 @Tbaugus44 like @DaanHoogland mentioned there could be some stale records for the same management server, try this: `select id,msid,name,state,last_update from mshost

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

2023-12-20 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1865480123 [SF] Trillian test result (tid-8645) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 58108 seconds Marvin logs:

Re: [PR] Option to create StorPool primary storage with a valid URL [cloudstack]

2023-12-20 Thread via GitHub
sureshanaparti commented on code in PR #8356: URL: https://github.com/apache/cloudstack/pull/8356#discussion_r1433486324 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -240,6 +246,13 @@ public

Re: [PR] Change password.policy.regex default to empty [cloudstack]

2023-12-20 Thread via GitHub
sureshanaparti commented on code in PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#discussion_r1433479556 ## server/src/main/java/com/cloud/user/PasswordPolicyImpl.java: ## @@ -188,12 +188,12 @@ protected void validateIfPasswordMatchesRegex(String password,

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
sureshanaparti commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1433478710 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,7 +517,13 @@ public static

Re: [I] Cloudstack virtual machines can't communicate with virtual routers when they are on different hosts [cloudstack]

2023-12-20 Thread via GitHub
herdiansah571 commented on issue #8360: URL: https://github.com/apache/cloudstack/issues/8360#issuecomment-1865454056 Hello @weizhouapache Currently Cloudstack virtual machines can communicate with virtual routers when they are on different hosts after configure VLAN Trunks on

Re: [PR] Option to create StorPool primary storage with a valid URL [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8356: URL: https://github.com/apache/cloudstack/pull/8356#issuecomment-1865422925 [SF] Trillian test result (tid-8647) Environment: kvm-alma9 (x2), Advanced Networking with Mgmt server a9 Total time taken: 52392 seconds Marvin logs:

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

2023-12-20 Thread via GitHub
blueorangutan commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1865417199 [SF] Trillian test result (tid-8642) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 55967 seconds Marvin logs:

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

2023-12-20 Thread via GitHub
blueorangutan commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1865318631 [SF] Trillian test result (tid-8641) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 46694 seconds Marvin logs:

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

2023-12-20 Thread via GitHub
blueorangutan commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1865306450 [SF] Trillian test result (tid-8644) Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7 Total time taken: 41648 seconds Marvin logs:

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

2023-12-20 Thread via GitHub
blueorangutan commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1865300504 [SF] Trillian test result (tid-8643) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 44910 seconds Marvin logs:

Re: [PR] Veeam: Support Veeam 11 and 12 [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8241: URL: https://github.com/apache/cloudstack/pull/8241#issuecomment-1865280063 @weizhouapache a [LL] Trillian-Jenkins test job (rocky8 mgmt + vmware-70u3) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To

Re: [PR] Veeam: Support Veeam 11 and 12 [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on PR #8241: URL: https://github.com/apache/cloudstack/pull/8241#issuecomment-1865278641 @blueorangutan LLtest rocky8 vmware-70u3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] Veeam: Support Veeam 11 and 12 [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8241: URL: https://github.com/apache/cloudstack/pull/8241#issuecomment-1865273902 [LL]Trillian test result (tid-6902) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 48743 seconds Marvin logs:

[I] Improvement on collecting/updating Secondary Storage total size/capacity [cloudstack]

2023-12-20 Thread via GitHub
andrijapanicsb opened a new issue, #8391: URL: https://github.com/apache/cloudstack/issues/8391 Improvement request: - the capacity of the Secondary Storage seem updated/obtained from time to time, by SSVMs and the value (capacity) is held in the memory of the mgmt server. In

[I] VMware: VMs can't be started when no secondary storage with enough capacity + wrong messages around secondary storage [cloudstack]

2023-12-20 Thread via GitHub
andrijapanicsb opened a new issue, #8390: URL: https://github.com/apache/cloudstack/issues/8390 An environment with 2 Secondary Storages - both over 90% full (global setting set's the treshold to 0.9) While ACS tries to start a VM (SSVM in this case) - it fails to start it with the

Re: [PR] server: fix debug message when expunge a vm [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8374: URL: https://github.com/apache/cloudstack/pull/8374#issuecomment-1865028605 [SF] Trillian test result (tid-8639) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 47391 seconds Marvin logs:

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8387: URL: https://github.com/apache/cloudstack/pull/8387#issuecomment-1864983753 @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

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on PR #8387: URL: https://github.com/apache/cloudstack/pull/8387#issuecomment-1864978125 @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] prevent nic removal on out of bounds router stop [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8371: URL: https://github.com/apache/cloudstack/pull/8371#discussion_r1433064693 ## server/src/main/java/com/cloud/network/guru/ControlNetworkGuru.java: ## @@ -166,18 +167,25 @@ public boolean release(NicProfile nic, VirtualMachineProfile

Re: [PR] prevent nic removal on out of bounds router stop [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on code in PR #8371: URL: https://github.com/apache/cloudstack/pull/8371#discussion_r1433034778 ## server/src/main/java/com/cloud/network/router/VirtualNetworkApplianceManager.java: ## @@ -36,82 +36,83 @@ */ public interface

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864911698 @weizhouapache 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

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864909880 @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: additional check to ensure hosts are left in up state [cloudstack]

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

Re: [I] Template reported as Ready/Installed only based on the template_stor_ref, ignoring template_zone_ref [cloudstack]

2023-12-20 Thread via GitHub
andrijapanicsb commented on issue #8388: URL: https://github.com/apache/cloudstack/issues/8388#issuecomment-1864856382 I faced the issue when triggering template auto-install for ADDITIONAL zone, while (by mistake) the template was not marked as cross_zone=1 (it was =0) Not sure if

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864819972 @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] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
shwstppr commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864813808 @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] server: improve stats collector logs to state what the collector does [cloudstack]

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

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
codecov[bot] commented on PR #8387: URL: https://github.com/apache/cloudstack/pull/8387#issuecomment-1864785118 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8387?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report Attention:

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

2023-12-20 Thread via GitHub
codecov[bot] commented on PR #8389: URL: https://github.com/apache/cloudstack/pull/8389#issuecomment-1864784680 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8389?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

Re: [PR] prevent nic removal on out of bounds router stop [cloudstack]

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

[PR] [DRAFT] Add support for Ceph RGW Object Store [cloudstack]

2023-12-20 Thread via GitHub
wido opened a new pull request, #8389: URL: https://github.com/apache/cloudstack/pull/8389 ### Description This PR adds support for the Ceph RGW as an Object Store Driver in CloudStack ### Types of changes - [ x ] New feature (non-breaking change which adds

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

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

Re: [I] Template reported as Ready/Installed only based on the template_stor_ref, ignoring template_zone_ref [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on issue #8388: URL: https://github.com/apache/cloudstack/issues/8388#issuecomment-1864691977 @andrijapanicsb did you face the issue in other scenario ? for example, add new zone ? I wonder if there is a way to reproduce the issue without manual db changes.

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

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

Re: [PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
rohityadavcloud commented on PR #8387: URL: https://github.com/apache/cloudstack/pull/8387#issuecomment-1864672568 @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

[I] Template reported as Ready/Installed only based on the template_stor_ref, ignoring template_zone_ref [cloudstack]

2023-12-20 Thread via GitHub
andrijapanicsb opened a new issue, #8388: URL: https://github.com/apache/cloudstack/issues/8388 ACS 4.18.1 As the title says: ![image](https://github.com/apache/cloudstack/assets/45762285/7c3e6a8b-ec16-4125-8420-6104a5aef442) The template_zone_ref for the given template

[PR] server: improve stats collector logs to state what the collector does [cloudstack]

2023-12-20 Thread via GitHub
rohityadavcloud opened a new pull request, #8387: URL: https://github.com/apache/cloudstack/pull/8387 This simply improves the log statement that prints debug statements during beginning of a stats collector run for hosts or VMs. ### Types of changes - [ ] Breaking change (fix

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432836221 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static

[I] Port forwarding rules gets the same public port when creating port forwarding rules with CloudStack's API [cloudstack]

2023-12-20 Thread via GitHub
saffronjam opened a new issue, #8386: URL: https://github.com/apache/cloudstack/issues/8386 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ API for Virtual Router Port forwarding rules ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1 ~~~ #

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

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

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1864619273 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
slavkap commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432822793 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static SpApiResponse

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432812725 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
slavkap commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432774853 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static SpApiResponse

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
codecov[bot] commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1864514086 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8385?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

Re: [PR] prevent nic removal on out of bounds router stop [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8371: URL: https://github.com/apache/cloudstack/pull/8371#discussion_r1432735875 ## engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java: ## @@ -2310,12 +2311,12 @@ public void

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1864502841 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] [Usage] Create network billing [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on PR #7236: URL: https://github.com/apache/cloudstack/pull/7236#issuecomment-1864494295 tested list usage types and list usagerecords for the new type. the output is good. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864488693 @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] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
shwstppr commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864486594 @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: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
shwstppr commented on code in PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#discussion_r1432723691 ## test/integration/smoke/test_vm_life_cycle.py: ## @@ -1011,8 +1011,36 @@ def setUpClass(cls): @classmethod def tearDownClass(cls): +

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
shwstppr commented on code in PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#discussion_r1432721985 ## test/integration/smoke/test_vm_life_cycle.py: ## @@ -1011,8 +1011,36 @@ def setUpClass(cls): @classmethod def tearDownClass(cls): +

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
shwstppr commented on code in PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#discussion_r1432721308 ## test/integration/smoke/test_vm_life_cycle.py: ## @@ -1011,8 +1011,36 @@ def setUpClass(cls): @classmethod def tearDownClass(cls): +

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on code in PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#discussion_r1432713732 ## test/integration/smoke/test_vm_life_cycle.py: ## @@ -1011,8 +1011,36 @@ def setUpClass(cls): @classmethod def tearDownClass(cls): +

Re: [PR] prevent nic removal on out of bounds router stop [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on code in PR #8371: URL: https://github.com/apache/cloudstack/pull/8371#discussion_r1432662763 ## engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java: ## @@ -2310,12 +2311,12 @@ public void

Re: [PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
boring-cyborg[bot] commented on PR #8385: URL: https://github.com/apache/cloudstack/pull/8385#issuecomment-1864451339 Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our

[PR] vpc: optimize createMonitorServiceCommand() execution [cloudstack]

2023-12-20 Thread via GitHub
phsm opened a new pull request, #8385: URL: https://github.com/apache/cloudstack/pull/8385 ### Description This PR moves an expensive getRouterHealthChecksConfig() upper the execution stack during VPC restart. So this way it is only executed once instead of tens or hundreds times

Re: [I] Backup virtual router of a redundant VPC stays in starting state after restart with clean up. [cloudstack]

2023-12-20 Thread via GitHub
weizhouapache commented on issue #8055: URL: https://github.com/apache/cloudstack/issues/8055#issuecomment-1864401554 > @weizhouapache Hi Wei, > > I've actually managed to implement the patch myself. > > The test looks promising: Without this patch applied, the 10-tier VPC took

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432652852 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static

Re: [I] Backup virtual router of a redundant VPC stays in starting state after restart with clean up. [cloudstack]

2023-12-20 Thread via GitHub
phsm commented on issue #8055: URL: https://github.com/apache/cloudstack/issues/8055#issuecomment-1864388934 @weizhouapache Hi Wei, I've actually managed to implement the patch myself. The test looks promising: Without this patch applied, the 10-tier VPC took 61 minutes

Re: [PR] test: additional check to ensure hosts are left in up state [cloudstack]

2023-12-20 Thread via GitHub
codecov[bot] commented on PR #8383: URL: https://github.com/apache/cloudstack/pull/8383#issuecomment-1864386737 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8383?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report All

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432649252 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static

Re: [PR] remove StorPool tags from detached volumes [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on code in PR #8377: URL: https://github.com/apache/cloudstack/pull/8377#discussion_r1432649252 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/util/StorPoolUtil.java: ## @@ -517,6 +517,12 @@ public static

Re: [PR] Option to create StorPool primary storage with a valid URL [cloudstack]

2023-12-20 Thread via GitHub
blueorangutan commented on PR #8356: URL: https://github.com/apache/cloudstack/pull/8356#issuecomment-1864384554 @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

Re: [PR] Option to create StorPool primary storage with a valid URL [cloudstack]

2023-12-20 Thread via GitHub
DaanHoogland commented on PR #8356: URL: https://github.com/apache/cloudstack/pull/8356#issuecomment-1864382564 @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

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

2023-12-20 Thread via GitHub
DaanHoogland commented on PR #7345: URL: https://github.com/apache/cloudstack/pull/7345#issuecomment-1864381200 @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

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

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

  1   2   >