Re: [PR] Allow locking and unlocking users via UI [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland commented on PR #8337: URL: https://github.com/apache/cloudstack/pull/8337#issuecomment-1862275877 code looks good @winterhazel . @JoaoJandre , did you test the new functionality exclusively or poke around for regressions as well? -- This is an automated message from the A

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1862242710 @weizhouapache a [LL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests -- This

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1862241671 @blueorangutan LLtest 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 s

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1862241572 smoke tests look ok. the failures might be caused by the busy trillian envs. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#issuecomment-1862240846 @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] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#issuecomment-1862239679 @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 c

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

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

Re: [PR] Fix reorder/list pools when cluster details are not set, while deploying vm / attaching volume [cloudstack]

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

Re: [PR] Fix reorder/list pools when cluster details are not set, while deploying vm / attaching volume [cloudstack]

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

Re: [PR] Fix reorder/list pools when cluster details are not set, while deploying vm / attaching volume [cloudstack]

2023-12-18 Thread via GitHub
sureshanaparti commented on PR #8373: URL: https://github.com/apache/cloudstack/pull/8373#issuecomment-1862186344 @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

[PR] Fix reorder/list pools when cluster details are not set, while deploying vm / attaching volume [cloudstack]

2023-12-18 Thread via GitHub
sureshanaparti opened a new pull request, #8373: URL: https://github.com/apache/cloudstack/pull/8373 ### Description This PR fixes reorder/list pools when cluster details are not set, while deploying vm / attaching volume. Problem: Attach volume to a VM fails, on infra with

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#issuecomment-1862158430 @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] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
vishesh92 commented on PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#issuecomment-1862157683 @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] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

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

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

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

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

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

Re: [PR] Add e2e tests for listing resources [cloudstack]

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

Re: [PR] Add e2e tests for listing resources [cloudstack]

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

Re: [PR] move nic removal from shutdown to expunge [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8371: URL: https://github.com/apache/cloudstack/pull/8371#issuecomment-1861831009 This code snippet seems not related to private ip release. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

(cloudstack-kubernetes-provider) branch dependabot/go_modules/golang.org/x/crypto-0.17.0 created (now f2aa932a)

2023-12-18 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/golang.org/x/crypto-0.17.0 in repository https://gitbox.apache.org/repos/asf/cloudstack-kubernetes-provider.git at f2aa932a Bump golang.org/x/crypto from 0.1.0 t

Re: [PR] Logging improvements on migration in the VmwareResource [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8300: URL: https://github.com/apache/cloudstack/pull/8300#issuecomment-1861819836 [SF] Trillian test result (tid-8618) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server a9 Total time taken: 47084 seconds Marvin logs: https://github.co

Re: [PR] move nic removal from shutdown to expunge [cloudstack]

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

Re: [I] Better handling for outband power operations on VR [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland commented on issue #8284: URL: https://github.com/apache/cloudstack/issues/8284#issuecomment-1861704415 The idea is that you don't run all at once and saying people don´t use it like it was designed is as much speculating as saying that they do. -- This is an automated messag

Re: [PR] Add e2e tests for listing resources [cloudstack]

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

Re: [PR] move nic removal from shutdown to expunge [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8371: URL: https://github.com/apache/cloudstack/pull/8371#issuecomment-1861695779 @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] move nic removal from shutdown to expunge [cloudstack]

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

Re: [PR] Allow locking and unlocking users via UI [cloudstack]

2023-12-18 Thread via GitHub
winterhazel commented on code in PR #8337: URL: https://github.com/apache/cloudstack/pull/8337#discussion_r1430513885 ## ui/src/config/section/user.js: ## @@ -96,6 +96,19 @@ export default { record.state === 'enabled' } }, +{ + api: 'lockUser', +

Re: [I] Better handling for outband power operations on VR [cloudstack]

2023-12-18 Thread via GitHub
andrijapanicsb commented on issue #8284: URL: https://github.com/apache/cloudstack/issues/8284#issuecomment-1861158226 (in other words you are speculating that some people might use it, while in reality we had known users hit the wall with the bug) (i.e. let's not speculate?) -- This i

Re: [I] Better handling for outband power operations on VR [cloudstack]

2023-12-18 Thread via GitHub
andrijapanicsb commented on issue #8284: URL: https://github.com/apache/cloudstack/issues/8284#issuecomment-1861156918 By removing NIC IPs of the VR, you give the false expectation that they have enough IPs - if they need to start VRs that are stopped (but IPs are now being used for other V

Re: [PR] Allow locking and unlocking users via UI [cloudstack]

2023-12-18 Thread via GitHub
winterhazel commented on code in PR #8337: URL: https://github.com/apache/cloudstack/pull/8337#discussion_r1430463427 ## ui/src/config/section/user.js: ## @@ -96,6 +96,19 @@ export default { record.state === 'enabled' } }, +{ + api: 'lockUser', +

Re: [PR] move nic removal from shutdown to expunge [cloudstack]

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

Re: [PR] NSX integration [cloudstack]

2023-12-18 Thread via GitHub
sonarcloud[bot] commented on PR #7919: URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1861063871 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://sonar

(cloudstack) branch nsx-integration updated (7cabc66f1c7 -> 2b896a3a218)

2023-12-18 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch nsx-integration in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 7cabc66f1c7 Merge branch 'main' of https://github.com/apache/cloudstack into nsx-integration ad

Re: [PR] NSX integration [cloudstack]

2023-12-18 Thread via GitHub
sonarcloud[bot] commented on PR #7919: URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1860879233 ## [![Quality Gate Failed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-failed-20px.png 'Quality Gate Failed')](https://sonar

Re: [PR] [Quota] Add API to list preset variables [cloudstack]

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

[PR] [Quota] Add API to list preset variables [cloudstack]

2023-12-18 Thread via GitHub
BryanMLima opened a new pull request, #8372: URL: https://github.com/apache/cloudstack/pull/8372 ### Description This PR extends the functionality introduced in PR #5909, allowing users to list the preset variables via API, facilitating the creation of the activation rules while crea

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

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

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

[PR] move nic removal from shutdown to expunge [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland opened a new pull request, #8371: URL: https://github.com/apache/cloudstack/pull/8371 ### Description This PR... does an attempt at tries Fixes: #8284 ### Types of changes - [ ] Breaking change (fix or fea

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

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

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

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

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on code in PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#discussion_r1430289668 ## plugins/integrations/kubernetes-service/src/main/java/com/cloud/kubernetes/cluster/KubernetesClusterManagerImpl.java: ## @@ -1068,30 +1068,35 @@ public Kube

(cloudstack) branch nsx-integration updated (7288ac458fc -> 7cabc66f1c7)

2023-12-18 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch nsx-integration in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 7288ac458fc NSX: Add unit tests to increase coverage (#8355) add de095ba70d2 server: fix url che

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland commented on code in PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#discussion_r1430230650 ## plugins/integrations/kubernetes-service/src/main/java/com/cloud/kubernetes/cluster/KubernetesClusterManagerImpl.java: ## @@ -1068,30 +1068,35 @@ public Kuber

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

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

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

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#issuecomment-1860610344 @JoaoJandre can checks be skipped if source is User.Source.LDAP ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

(cloudstack) branch nsx-integration updated (4457c62ad32 -> 7288ac458fc)

2023-12-18 Thread pearl11594
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch nsx-integration in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 4457c62ad32 [NSX] Address SonarCloud Bugs (#8341) add 7288ac458fc NSX: Add unit tests to increas

Re: [PR] NSX: Add unit tests to increase coverage [cloudstack]

2023-12-18 Thread via GitHub
Pearl1594 merged PR #8355: URL: https://github.com/apache/cloudstack/pull/8355 -- 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...@cloud

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

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

2023-12-18 Thread via GitHub
shwstppr commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1860531590 @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] [HEALTH] 4.19/main Health Check, please don't merge this! [cloudstack]

2023-12-18 Thread via GitHub
sonarcloud[bot] commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1860515304 ## [![Quality Gate Passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/qg-passed-20px.png 'Quality Gate Passed')](https://sonar

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

2023-12-18 Thread via GitHub
JoaoJandre commented on PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#issuecomment-1860496605 > is it wise to allow for empty passwords? I may be misinterpreting this code, but I would not want this in a default install. Or is this only applicable for LDAP logins? As

Re: [PR] systemvm: vncport evaluation [cloudstack]

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

(cloudstack) branch dummy updated (06be550b6c2 -> 40efb5b0032)

2023-12-18 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 06be550b6c2 do not merge add 16d45f731d7 Save the account which deliberately removed a public IP from quarantin

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

2023-12-18 Thread via GitHub
DaanHoogland commented on PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#issuecomment-1860380779 is it wise to allow for empty passwords? I may be misinterpreting this code, but I would not want this in a default install. Or is this only applicable for LDAP logins? BTW

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

2023-12-18 Thread via GitHub
shwstppr commented on PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#issuecomment-1860366231 @JoaoJandre would be good to fix in 4.18 branch but it seems it needs a DB change in the upgrade path. Not sure if it is desired at this point to have a 4181to4182 upgrade path --

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

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

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

2023-12-18 Thread via GitHub
JoaoJandre commented on PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#issuecomment-1860355742 I'm not sure if this is significant enough for 4.19, if not, I can target 4.18.2. -- This is an automated message from the Apache Git Service. To respond to the message, please lo

Re: [PR] Password policies [cloudstack]

2023-12-18 Thread via GitHub
JoaoJandre commented on PR #6567: URL: https://github.com/apache/cloudstack/pull/6567#issuecomment-1860353728 > This looks like a regression of the pr https://lists.apache.org/thread/tqtcwsfb0knmvqct1hlow4ty2nc2w2j2 > > cc @JoaoJandre @DaanHoogland @GutoVeronezi @weizhouapache

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

2023-12-18 Thread via GitHub
JoaoJandre opened a new pull request, #8370: URL: https://github.com/apache/cloudstack/pull/8370 ### Description This PR changes the `password.policy.regex` default value to empty. With an empty value for the configuration, it is skipped during the password policy check, only when th

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

2023-12-18 Thread via GitHub
blueorangutan commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1860325285 @shwstppr [SL] I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test [mgmt os] [hypervisor] [keepEnv] [qemuEv] Mgmt OS options: ['c

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

2023-12-18 Thread via GitHub
shwstppr commented on PR #7344: URL: https://github.com/apache/cloudstack/pull/7344#issuecomment-1860323891 @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 commen

(cloudstack-csbench) branch main updated: Add description for parameters in config file (#2)

2023-12-18 Thread vladi
This is an automated email from the ASF dual-hosted git repository. vladi pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/cloudstack-csbench.git The following commit(s) were added to refs/heads/main by this push: new 07cb8c5 Add description for parameters i

Re: [PR] systemvm: vncport evaluation [cloudstack]

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

Re: [PR] vncport evaluation [cloudstack]

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

Re: [PR] vncport evaluation [cloudstack]

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

[PR] vncport evaluation [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland opened a new pull request, #8369: URL: https://github.com/apache/cloudstack/pull/8369 ### Description This PR... Fixes: #8303 ### Types of changes - [ ] Breaking change (fix or feature that would cause exist

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

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

Re: [I] Creation of large files with vmdk format in secondary storage-CloudStack 4.17.2.0 [cloudstack]

2023-12-18 Thread via GitHub
rajujith commented on issue #8348: URL: https://github.com/apache/cloudstack/issues/8348#issuecomment-1860045138 @yeganehmehr volumes in the secondary storage information can be found in table "volume_store_ref" -- This is an automated message from the Apache Git Service. To respond to t

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

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

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on code in PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#discussion_r1429839640 ## server/src/main/java/com/cloud/network/security/SecurityGroupManagerImpl.java: ## @@ -1410,7 +1407,7 @@ public SecurityGroupVO getDefaultSecurityGroup(long

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

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

Re: [PR] ITCO watchdog added [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8282: URL: https://github.com/apache/cloudstack/pull/8282#issuecomment-1859967662 > > @DaanHoogland maybe we could add new action `inject-nmi` as well. > > seems like a new PR, would you suggest a API/UI enhancement for that? (creating #8368) no,

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
vishesh92 commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1859964748 > > > @vishesh92 similar failures in VMware test results. Anything related? > > > > > > No. It doesn't have anything related to my changes. > > Considering the curren

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1859962083 @vishesh92 a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests -- This is a

Re: [PR] Add e2e tests for listing resources [cloudstack]

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

Re: [PR] CKS: create Security Groups for CKS clusters of each account [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland commented on code in PR #8316: URL: https://github.com/apache/cloudstack/pull/8316#discussion_r1429792084 ## server/src/main/java/com/cloud/network/security/SecurityGroupManagerImpl.java: ## @@ -1410,7 +1407,7 @@ public SecurityGroupVO getDefaultSecurityGroup(long

Re: [PR] ITCO watchdog added [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland commented on PR #8282: URL: https://github.com/apache/cloudstack/pull/8282#issuecomment-1859885924 > @DaanHoogland maybe we could add new action `inject-nmi` as well. seems like a new PR, would you suggest a API/UI enhancement for that? (creating #8368) -- This is an a

[I] KVM: add new action `inject-nmi` [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland opened a new issue, #8368: URL: https://github.com/apache/cloudstack/issues/8368 # ISSUE TYPE * Enhancement Request # COMPONENT NAME ~~~ KVM ~~~ # CLOUDSTACK VERSION ~~~ ~~~ # CONFIGURATION

Re: [PR] Add e2e tests for listing resources [cloudstack]

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

Re: [PR] Logging improvements on migration in the VmwareResource [cloudstack]

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

Re: [PR] Logging improvements on migration in the VmwareResource [cloudstack]

2023-12-18 Thread via GitHub
DaanHoogland commented on PR #8300: URL: https://github.com/apache/cloudstack/pull/8300#issuecomment-1859858604 @blueorangutan test alma9 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 URL above to go

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
weizhouapache commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1859798879 > > @vishesh92 similar failures in VMware test results. Anything related? > > No. It doesn't have anything related to my changes. Considering the current smoke tests

[I] VSAN Storage Policy [cloudstack]

2023-12-18 Thread via GitHub
ahmadamirahmadi007 opened a new issue, #8367: URL: https://github.com/apache/cloudstack/issues/8367 # ISSUE TYPE * Bug Report # COMPONENT NAME Categorize the issue, UI # CLOUDSTACK VERSION 4.17.2 # CONFIGURATION Vmware Hypervisor

Re: [PR] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
blueorangutan commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1859756809 @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] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
vishesh92 commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1859751668 @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] Add e2e tests for listing resources [cloudstack]

2023-12-18 Thread via GitHub
vishesh92 commented on PR #8281: URL: https://github.com/apache/cloudstack/pull/8281#issuecomment-1859741014 @shwstppr I checked a few failures and most of them seem related to the environment itself or some error while cleaning up resources. -- This is an automated message from the Apach

Re: [PR] Quota email configuration [cloudstack]

2023-12-18 Thread via GitHub
github-actions[bot] commented on PR #8307: URL: https://github.com/apache/cloudstack/pull/8307#issuecomment-1859726092 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] Quota tariff order [cloudstack]

2023-12-18 Thread via GitHub
github-actions[bot] commented on PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#issuecomment-1859726118 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] api,server,ui: tagged resource limits [cloudstack]

2023-12-18 Thread via GitHub
github-actions[bot] commented on PR #8362: URL: https://github.com/apache/cloudstack/pull/8362#issuecomment-1859726157 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] NSX integration [cloudstack]

2023-12-18 Thread via GitHub
github-actions[bot] commented on PR #7919: URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1859726052 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] [Usage] Create network billing [cloudstack]

2023-12-18 Thread via GitHub
github-actions[bot] commented on PR #7236: URL: https://github.com/apache/cloudstack/pull/7236#issuecomment-1859726020 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] [Usage] Create VPC billing [cloudstack]

2023-12-18 Thread via GitHub
github-actions[bot] commented on PR #7235: URL: https://github.com/apache/cloudstack/pull/7235#issuecomment-1859725984 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