Re: [PR] Restrict the migration of volumes attached to VMs in Starting state [cloudstack]

2025-01-03 Thread via GitHub
JoaoJandre commented on PR #9725: URL: https://github.com/apache/cloudstack/pull/9725#issuecomment-2569344644 @DaanHoogland considering it's an easy change, I don't see why introduce this and change it later. Since @FelipeM525 is no longer working on this PR: @bernardodemarco could you

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

2025-01-03 Thread via GitHub
hsato03 commented on code in PR #6589: URL: https://github.com/apache/cloudstack/pull/6589#discussion_r1902177532 ## server/src/main/java/org/apache/cloudstack/backup/BackupManagerImpl.java: ## @@ -836,7 +837,8 @@ public boolean restoreBackupVolumeAndAttachToVM(final String bac

Re: [I] Cloustack agent in KVM zone not connect, It stuck in alert when cloustack-managerment restart. [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #10154: URL: https://github.com/apache/cloudstack/issues/10154#issuecomment-2568857270 > This is full logs for logid:f8b676f6. Hi @leduyquy , it seems cut short as it does not include (go up to) the exception trace itself. btw, I am trying to rep

(cloudstack) branch healthcheck-4.19 updated (200912abe25 -> 5611139c239)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch healthcheck-4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git discard 200912abe25 DO NOT MERGE add d8c321d57ae check tags while fetching storage pool for importing vm (#9

(cloudstack) branch main updated: Add pre-commit hook `fix-byte-order-marker` (#10147)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new 0f3ea8ceb2e Add pre-commit hook `fix-byte-order-ma

Re: [PR] pre-commit check `.cfg` files with the trailing whitespace hook [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland merged PR #10151: URL: https://github.com/apache/cloudstack/pull/10151 -- 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: [PR] Add pre-commit hook `fix-byte-order-marker` [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland merged PR #10147: URL: https://github.com/apache/cloudstack/pull/10147 -- 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...@

(cloudstack) branch main updated: pre-commit check `.cfg` files with the trailing whitespace hook (#10151)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/main by this push: new 2b070880c10 pre-commit check `.cfg` files with the

Re: [PR] pre-commit fix mixed line endings in XML files [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland merged PR #10148: URL: https://github.com/apache/cloudstack/pull/10148 -- 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: [PR] [HEALTH] 4.19 LTS Health Check, please don't merge this! [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2570003855 [SF] Trillian test result (tid-12041) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 46400 seconds Marvin logs: https://github.com/

Re: [PR] CKS Enhancements [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9102: URL: https://github.com/apache/cloudstack/pull/9102#issuecomment-2570004416 [SF] Trillian test result (tid-12031) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 109796 seconds Marvin logs: https://github.com

Re: [PR] pre-commit add hook `check-shebang-scripts-are-executable` for Shell [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #10150: URL: https://github.com/apache/cloudstack/pull/10150#issuecomment-2569751748 [SF] Trillian test result (tid-12030) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 110094 seconds Marvin logs: https://github.c

Re: [PR] check tags while fetching storage pool for importing vm [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland merged PR #9764: URL: https://github.com/apache/cloudstack/pull/9764 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cl

Re: [PR] pre-commit upgrade codespell and fix spelling [cloudstack]

2025-01-03 Thread via GitHub
vishesh92 commented on code in PR #10144: URL: https://github.com/apache/cloudstack/pull/10144#discussion_r1901553450 ## .github/linters/codespell.txt: ## @@ -35,6 +35,7 @@ aqcuire aqcuired aquire aquiring +assertin Review Comment: @jbampton got it. But can you remoe `fal

(cloudstack) branch 4.19 updated: check tags while fetching storage pool for importing vm (#9764)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.19 by this push: new d8c321d57ae check tags while fetching storage pool

Re: [I] Incorrectly tagged storage pool selected while importing VM leading to failure [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland closed issue #9670: Incorrectly tagged storage pool selected while importing VM leading to failure URL: https://github.com/apache/cloudstack/issues/9670 -- 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: [I] Incorrectly tagged storage pool selected while importing VM leading to failure [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #9670: URL: https://github.com/apache/cloudstack/issues/9670#issuecomment-2568825711 fixed in #9764 -- 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] pre-commit fix mixed line endings in XML files [cloudstack]

2025-01-03 Thread via GitHub
vishesh92 commented on PR #10148: URL: https://github.com/apache/cloudstack/pull/10148#issuecomment-2568834315 > @vishesh92 the error seems unrelated. agree? I agree -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

Re: [PR] Fix ordering of secondary storages with the algorithm `firstfitleastconsumed` [cloudstack]

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

Re: [PR] Fix ordering of secondary storages with the algorithm `firstfitleastconsumed` [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #8557: URL: https://github.com/apache/cloudstack/pull/8557#issuecomment-2568840940 @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

(cloudstack) 01/01: Merge release branch 4.19 to 4.20

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 79316450d19df68bb042106477e09edbce12704b Merge: 3d0b4fca847 d8c321d57ae Author: Daan Hoogland AuthorDate: Fri Jan 3 09:4

(cloudstack) branch 4.20 updated: log name change after merge forward

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn 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 cfafcaeb01b log name change after merge forward cf

(cloudstack) branch main updated (5bae1188ff6 -> 30b2588c062)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 5bae1188ff6 pre-commit fix mixed line endings in XML files (#10148) add 3d0b4fca847 UI: Add cluster arch type to

(cloudstack) 01/01: Merge release branch 4.20 to main

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 30b2588c062898b57da564d0a2d9e57e8774b0a6 Merge: 5bae1188ff6 cfafcaeb01b Author: Daan Hoogland AuthorDate: Fri Jan 3 10:1

Re: [I] Cloustack agent in KVM zone not connect, It stuck in alert when cloustack-managerment restart. [cloudstack]

2025-01-03 Thread via GitHub
leduyquy commented on issue #10154: URL: https://github.com/apache/cloudstack/issues/10154#issuecomment-2568883776 Hi @DaanHoogland , sorry my bad. Maybe it's cut off by my browser's character limit I have attached full logs in the attached file. Please help me check it. [logs.txt](ht

Re: [I] Cloustack agent in KVM zone not connect, It stuck in alert when cloustack-managerment restart. [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #10154: URL: https://github.com/apache/cloudstack/issues/10154#issuecomment-2568913456 @leduyquy the log looks good but it is not for the same ID ``` 2025-01-03 15:46:17,798 ERROR [c.c.a.m.AgentManagerImpl] (AgentConnectTaskPool-17819:ctx-02ff844d) (

(cloudstack) branch 4.20 updated (3d0b4fca847 -> 79316450d19)

2025-01-03 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 3d0b4fca847 UI: Add cluster arch type to the zone creation wizard (#10080) add d8c321d57ae check tags while fetc

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

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2568927827 @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 for Management Server Maintenance Mode [cloudstack]

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

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

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #10006: URL: https://github.com/apache/cloudstack/pull/10006#issuecomment-2568928765 @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] [HEALTH] 4.20 Health Check, please don't merge this! [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #10006: URL: https://github.com/apache/cloudstack/pull/10006#issuecomment-2568929677 @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

(cloudstack) branch healthcheck-4.20 updated (245cbdc50bb -> f2343b62ff7)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch healthcheck-4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git discard 245cbdc50bb dummy commit , please don't merge add cbac6cc05bd Add ownership selection to VPC form (#1

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

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2568929729 @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] 4.19 LTS Health Check, please don't merge this! [cloudstack]

2025-01-03 Thread via GitHub
sonarqubecloud[bot] commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2568941150 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://s

(cloudstack) branch healthcheck-main updated (a31574890fe -> e3fbe41efbc)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch healthcheck-main in repository https://gitbox.apache.org/repos/asf/cloudstack.git omit a31574890fe 4.20/main Health Check, please don't merge this! add ee893d12027 Clean up Python code; Py

Re: [I] Cloustack agent in KVM zone not connect, It stuck in alert when cloustack-managerment restart. [cloudstack]

2025-01-03 Thread via GitHub
leduyquy commented on issue #10154: URL: https://github.com/apache/cloudstack/issues/10154#issuecomment-2568932215 Hi @DaanHoogland. I have attached the information you need in the attached files. Besides. I deployed cloudstack with non-internet environment so it may encounter errors

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

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

Re: [PR] deal with NPE during host reconnect [cloudstack]

2025-01-03 Thread via GitHub
rohityadavcloud commented on PR #10158: URL: https://github.com/apache/cloudstack/pull/10158#issuecomment-2568952326 @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 specif

Re: [PR] deal with NPE during host reconnect [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #10158: URL: https://github.com/apache/cloudstack/pull/10158#issuecomment-2568954493 @rohityadavcloud [SL] I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test [mgmt os] [hypervisor] [keepEnv] [qemuEv] [basicZone|se

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

2025-01-03 Thread via GitHub
blueorangutan commented on PR #10149: URL: https://github.com/apache/cloudstack/pull/10149#issuecomment-2568986860 @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] [HEALTH] 4.21/main Health Check, please don't merge this [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #10149: URL: https://github.com/apache/cloudstack/pull/10149#issuecomment-2568985969 @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] CKS Enhancements [cloudstack]

2025-01-03 Thread via GitHub
nvazquez commented on PR #9102: URL: https://github.com/apache/cloudstack/pull/9102#issuecomment-2570036583 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific com

Re: [PR] CKS Enhancements [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9102: URL: https://github.com/apache/cloudstack/pull/9102#issuecomment-2570037047 @nvazquez 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.

(cloudstack-documentation) branch 4.19 updated: vxlan prefix must be given to prevent interpretation as vlan (#465)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/4.19 by this push: new ba41bd9 vxlan prefix must be given

[I] `cloudstack-documentation`: `.rst` files possible spelling mistakes [cloudstack]

2025-01-03 Thread via GitHub
jbampton opened a new issue, #10159: URL: https://github.com/apache/cloudstack/issues/10159 ``` cloudstack-documentation$ find . -name '*.rst' -exec codespell {} \; ./source/plugins/ipv6.rst:285: suported ==> supported ./source/plugins/ipv6.rst:287: paramter ==> parameter ./source

Re: [I] `cloudstack-documentation`: `.rst` files possible spelling mistakes [cloudstack]

2025-01-03 Thread via GitHub
boring-cyborg[bot] commented on issue #10159: URL: https://github.com/apache/cloudstack/issues/10159#issuecomment-2569243983 Thanks for opening your first issue here! Be sure to follow the issue template! -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] Restrict the migration of volumes attached to VMs in Starting state [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #9725: URL: https://github.com/apache/cloudstack/pull/9725#issuecomment-2569224407 thanks @bernardodemarco @JoaoJandre , do you agree that this PR is good to go, even when more improvements are possible? -- This is an automated message from the Apache

(cloudstack-go) branch fix-vm-usage-history updated (b471df6 -> 818910e)

2025-01-03 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch fix-vm-usage-history in repository https://gitbox.apache.org/repos/asf/cloudstack-go.git from b471df6 Fix VM Usage histroy response add bfc6c0c Add test data for register userkeys

Re: [PR] Adding Count Resource Domain [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #7274: URL: https://github.com/apache/cloudstack/pull/7274#issuecomment-2569550562 @CharlesQueiroz we are nearing 4.19.2 . Do you plan to work on this? cc @benj-n -- This is an automated message from the Apache Git Service. To respond to the message, please

[I] ACL Rule works intermittently, requires reboot or cleanup [cloudstack]

2025-01-03 Thread via GitHub
btzq opened a new issue, #10160: URL: https://github.com/apache/cloudstack/issues/10160 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ VR ~~~ # CLOUDSTACK VERSION ~~~ 4.19.0 ~~~ # CONFIGURATION #

Re: [PR] Restrict the migration of volumes attached to VMs in Starting state [cloudstack]

2025-01-03 Thread via GitHub
bernardodemarco commented on PR #9725: URL: https://github.com/apache/cloudstack/pull/9725#issuecomment-2569607448 > @DaanHoogland considering it's an easy change, I don't see why introduce this and change it later. Since @FelipeM525 is no longer working on this PR: @bernardodemarco could y

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

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

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

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

Re: [PR] Improve logging to include more identifiable information [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9873: URL: https://github.com/apache/cloudstack/pull/9873#issuecomment-2569045160 @kiranchavala 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

(cloudstack-documentation) branch main updated: added steps to expose service a vmware based cks cluster (#466)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git The following commit(s) were added to refs/heads/main by this push: new 09fce67 added steps to expose servi

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

2025-01-03 Thread via GitHub
blueorangutan commented on PR #10149: URL: https://github.com/apache/cloudstack/pull/10149#issuecomment-2569145352 @DaanHoogland a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smok

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

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2569145201 @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] [HEALTH] 4.19 LTS Health Check, please don't merge this! [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2569147640 @DaanHoogland a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke

Re: [I] Cloustack agent in KVM zone not connect, It stuck in alert when cloustack-managerment restart. [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #10154: URL: https://github.com/apache/cloudstack/issues/10154#issuecomment-2569151174 @leduyquy I have not found more info in your logs that could help and I tried to replicate your scenario, but no error occured. Can you think of anything else that might ha

Re: [I] `cloudstack-documentation`: `.rst` files possible spelling mistakes [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #10159: URL: https://github.com/apache/cloudstack/issues/10159#issuecomment-2569271943 good effort @jbampton , I think most of those must be easy fixes. /me looking at ``` ./source/plugins/vxlan.rst:301: nd ==> and, 2nd ./source/plugins/vxlan.rst:304:

Re: [I] `cloudstack-documentation`: `.rst` files possible spelling mistakes [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #10159: URL: https://github.com/apache/cloudstack/issues/10159#issuecomment-2569278279 > good effort @jbampton , I think most of those must be easy fixes. /me looking at > > ``` > ./source/plugins/vxlan.rst:301: nd ==> and, 2nd > ./source/plugins/

Re: [PR] Reserve memory for host [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #6809: URL: https://github.com/apache/cloudstack/pull/6809#issuecomment-2569448062 @benj-n are you still working on this? -- 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] Reserve memory for host [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on code in PR #6809: URL: https://github.com/apache/cloudstack/pull/6809#discussion_r1901921667 ## engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java: ## @@ -124,6 +131,8 @@ import com.cloud.utils.nio.Task; import com.cloud.

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

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #10149: URL: https://github.com/apache/cloudstack/pull/10149#issuecomment-2569143248 @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 sp

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

2025-01-03 Thread via GitHub
sonarqubecloud[bot] commented on PR #10149: URL: https://github.com/apache/cloudstack/pull/10149#issuecomment-2569002384 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https:/

Re: [I] Prometheus integration: requiring allowed ips is too restrictive [cloudstack]

2025-01-03 Thread via GitHub
NuxRo commented on issue #2801: URL: https://github.com/apache/cloudstack/issues/2801#issuecomment-2569103804 @araujobsd Indeed, some more flexibility and support for cidr annotation would be great, but the issue can be easily worked around with a local proxy (haproxy, nginx, whatever) and

Re: [PR] Reserve memory for host [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #6809: URL: https://github.com/apache/cloudstack/pull/6809#issuecomment-2569429586 Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 11984 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

Re: [PR] Improve logging to include more identifiable information [cloudstack]

2025-01-03 Thread via GitHub
kiranchavala commented on PR #9873: URL: https://github.com/apache/cloudstack/pull/9873#issuecomment-2569043081 @blueorangutan test 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 to go to the spe

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

2025-01-03 Thread via GitHub
sonarqubecloud[bot] commented on PR #10006: URL: https://github.com/apache/cloudstack/pull/10006#issuecomment-2569002456 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https:/

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

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

Re: [PR] CKS Enhancements [cloudstack]

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

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

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2570024864 [SF] Trillian test result (tid-12042) Environment: kvm-ubuntu22 (x2), Advanced Networking with Mgmt server u22 Total time taken: 50763 seconds Marvin logs: https://github

(cloudstack) 01/01: Revert "pre-commit: add hook `check-yaml` (#9133)"

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch revert-9133-pre-commit-check-yaml in repository https://gitbox.apache.org/repos/asf/cloudstack.git commit 61328bcd54a59cba8cffc566f5e0b0c368068df0 Author: dahn AuthorDate: Sat Jan 4 08:21:37 2025 +

(cloudstack) branch revert-9133-pre-commit-check-yaml created (now 61328bcd54a)

2025-01-03 Thread dahn
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch revert-9133-pre-commit-check-yaml in repository https://gitbox.apache.org/repos/asf/cloudstack.git at 61328bcd54a Revert "pre-commit: add hook `check-yaml` (#9133)" This branch includes the

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

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2570386008 [SF] Trillian test result (tid-12044) Environment: xcpng82 (x2), Advanced Networking with Mgmt server ol9 Total time taken: 64616 seconds Marvin logs: https://github.com/

Re: [I] ACL Rule works intermittently, requires reboot or cleanup [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on issue #10160: URL: https://github.com/apache/cloudstack/issues/10160#issuecomment-2570529118 @btzq sounds bad, but as you say hit or miss. This will be also true for anybody fixing it, so please add as much details as you can. errors from logs on the router host an

Re: [PR] Revert "pre-commit: add hook `check-yaml`" [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #10161: URL: https://github.com/apache/cloudstack/pull/10161#issuecomment-2570542483 @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] pre-commit: add hook `check-yaml` [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2570487755 Sorry @nvazquez , I was a bit eager on this one, let's revert. cc @vishesh92 @jbampton @nvazquez , do you see a possibility to make this check work (by options on the check or

[PR] Revert "pre-commit: add hook `check-yaml`" [cloudstack]

2025-01-03 Thread via GitHub
DaanHoogland opened a new pull request, #10161: URL: https://github.com/apache/cloudstack/pull/10161 Reverts apache/cloudstack#9133 -- 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] Revert "pre-commit: add hook `check-yaml`" [cloudstack]

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

Re: [PR] Revert "pre-commit: add hook `check-yaml`" [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #10161: URL: https://github.com/apache/cloudstack/pull/10161#issuecomment-2570550207 @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] Improve logging to include more identifiable information [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9873: URL: https://github.com/apache/cloudstack/pull/9873#issuecomment-2570014731 [SF] Trillian test result (tid-12036) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 53744 seconds Marvin logs: https://github.com/

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

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9315: URL: https://github.com/apache/cloudstack/pull/9315#issuecomment-2570020365 [SF] Trillian test result (tid-12043) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 49719 seconds Marvin logs: https://github.

Re: [PR] CKS Enhancements [cloudstack]

2025-01-03 Thread via GitHub
blueorangutan commented on PR #9102: URL: https://github.com/apache/cloudstack/pull/9102#issuecomment-2570027677 [SF] Trillian test result (tid-12032) Environment: kvm-ubuntu22 (x2), Advanced Networking with Mgmt server u22 Total time taken: 114599 seconds Marvin logs: https://githu

Re: [PR] pre-commit: add hook `check-yaml` [cloudstack]

2025-01-03 Thread via GitHub
nvazquez commented on PR #9133: URL: https://github.com/apache/cloudstack/pull/9133#issuecomment-2570042812 Hi guys @jbampton @DaanHoogland @vishesh92 I'm not able to get CKS clusters into Running state after these changes (at least after merging main branch on my PR #9102): ![image](ht