Re: [PR] server: remove username required condition for listVmsForImport [cloudstack]

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

Re: [PR] Improvements on information about disks size and offering in VM deployment wizard's InfoCard [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on PR #9249: URL: https://github.com/apache/cloudstack/pull/9249#issuecomment-2167362275 @winterhazel for pure UI changes, pl use BO ui commands; the packaging commands add 3x load on backend CI systems and may be skipped for UI only PRs. Thanks. -- This is an a

Re: [PR] Add action button to enable/disable Oauth provider [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9242: URL: https://github.com/apache/cloudstack/pull/9242#issuecomment-2167362204 Packaging result [SF]: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 9941 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] vmware: do not tear down vm disks if deploy-as-is vm has vm snapshots [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9243: URL: https://github.com/apache/cloudstack/pull/9243#issuecomment-2167362014 [SF] Trillian test result (tid-10436) Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8 Total time taken: 54822 seconds Marvin logs: https://github.com

Re: [PR] Improvements on information about disks size and offering in VM deployment wizard's InfoCard [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9249: URL: https://github.com/apache/cloudstack/pull/9249#issuecomment-2167361168 @rohityadavcloud a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To resp

Re: [PR] Improvements on information about disks size and offering in VM deployment wizard's InfoCard [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on PR #9249: URL: https://github.com/apache/cloudstack/pull/9249#issuecomment-2167360004 @blueorangutan ui -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

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

2024-06-13 Thread via GitHub
rohityadavcloud commented on PR #8372: URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2167357934 @BryanMLima shouldn't we need to run smoketests on the PR, or do we think they were covered by GHA/simulator smoketests? -- This is an automated message from the Apache Git S

Re: [I] ISO.CREATE event is not present for Kubernetes ISO registration [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on issue #8043: URL: https://github.com/apache/cloudstack/issues/8043#issuecomment-2167350693 Fixed by https://github.com/apache/cloudstack/pull/9070 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

(cloudstack) branch 4.19 updated: cks: fix events (#9070)

2024-06-13 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit 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 ce9b2c52f32 cks: fix events (#9070) ce9b2c52f32 i

Re: [PR] cks: fix events [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud merged PR #9070: URL: https://github.com/apache/cloudstack/pull/9070 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...

Re: [I] ISO.CREATE event is not present for Kubernetes ISO registration [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud closed issue #8043: ISO.CREATE event is not present for Kubernetes ISO registration URL: https://github.com/apache/cloudstack/issues/8043 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

Re: [I] Minor: listVmsForImport API fix [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud closed issue #8414: Minor: listVmsForImport API fix URL: https://github.com/apache/cloudstack/issues/8414 -- 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 uns

(cloudstack-documentation) branch 4.19 updated: import-vm: remove username, password request params for vm import (#401)

2024-06-13 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit 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 6e81764 import-vm: remove username

Re: [PR] server: remove username required condition for listVmsForImport [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on PR #9124: URL: https://github.com/apache/cloudstack/pull/9124#issuecomment-2167336373 Merged based on GHA smoketests and build tests. -- 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] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-13 Thread via GitHub
shwstppr commented on code in PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#discussion_r1639345612 ## plugins/hypervisors/xenserver/src/main/java/org/apache/cloudstack/hypervisor/xenserver/ExtraConfigurationUtility.java: ## @@ -169,12 +178,28 @@ private static vo

(cloudstack) branch 4.19 updated: server: remove username required condition for listVmsForImport (#9124)

2024-06-13 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit 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 df5c5465594 server: remove username required cond

Re: [PR] server: remove username required condition for listVmsForImport [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud merged PR #9124: URL: https://github.com/apache/cloudstack/pull/9124 -- 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] protect against missing service offering [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9235: URL: https://github.com/apache/cloudstack/pull/9235#issuecomment-2167334830 @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: [I] Web UI: incorrect sorting of colums with numerical values [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on issue #8663: URL: https://github.com/apache/cloudstack/issues/8663#issuecomment-2167333801 Fixed by https://github.com/apache/cloudstack/pull/9247 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [I] Web UI: incorrect sorting of colums with numerical values [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud closed issue #8663: Web UI: incorrect sorting of colums with numerical values URL: https://github.com/apache/cloudstack/issues/8663 -- 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 th

(cloudstack) branch 4.19 updated (2ca0857bd59 -> 6ce2a58f9bc)

2024-06-13 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit pushed a change to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 2ca0857bd59 api: listVM API improvement followup, change returning of stats detail (#9177) add 6ce2a58f9bc ui:

Re: [PR] ui: improve sort utility for metrics [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud merged PR #9247: URL: https://github.com/apache/cloudstack/pull/9247 -- 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] DRS: Ensure the destination host is part of the same cluster [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9245: URL: https://github.com/apache/cloudstack/pull/9245#issuecomment-2167332284 @rohityadavcloud a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. T

Re: [PR] protect against missing service offering [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #9235: URL: https://github.com/apache/cloudstack/pull/9235#issuecomment-2167331952 @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] DRS: Ensure the destination host is part of the same cluster [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on PR #9245: URL: https://github.com/apache/cloudstack/pull/9245#issuecomment-2167330738 @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] [Quota] Improve Quota balance calculation flow [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-2167323795 @BryanMLima , if @GutoVeronezi 's concerns are met, we can merge. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-2167322511 @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 Servic

Re: [PR] Add volume encryption checks during the disk offering change [cloudstack]

2024-06-13 Thread via GitHub
harikrishna-patnala commented on PR #9209: URL: https://github.com/apache/cloudstack/pull/9209#issuecomment-2167320313 @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 spec

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2167316852 sounds good @dominar250 , but still a separate ip range for the host and for the SVMs (and VMs in general) is best. -- This is an automated message from the Apache Git Serv

Re: [PR] allow users to add description to private gateway and static routes [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #9135: URL: https://github.com/apache/cloudstack/pull/9135#issuecomment-2167308425 > @DaanHoogland will look into the lint errors in sql file and double check if description is needed in network, but VPCGateway description is definitely needed, thanks! Ye

Re: [I] Templates owned by Project Account not allowed to share templates [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on issue #7350: URL: https://github.com/apache/cloudstack/issues/7350#issuecomment-2167298069 @kiranchavala @rohityadavcloud @damonb123 , My take on this is that if the user has access to the template thay can adjust the sharing rights as well. I do not have clear yet

Re: [PR] Add action button to enable/disable Oauth provider [cloudstack]

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

Re: [PR] test: fix test/integration/smoke/test_register_userdata.py which caused networks not to be deleted [cloudstack]

2024-06-13 Thread via GitHub
weizhouapache commented on PR #9244: URL: https://github.com/apache/cloudstack/pull/9244#issuecomment-2167291840 this is ready for review @DaanHoogland @sureshanaparti the rest failures are caused by live migration on xcpng82, see #https://github.com/apache/cloudstack/issues/8586

Re: [PR] Add action button to enable/disable Oauth provider [cloudstack]

2024-06-13 Thread via GitHub
abh1sar commented on PR #9242: URL: https://github.com/apache/cloudstack/pull/9242#issuecomment-2167290930 @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] remove Project Template Permissions inhibition [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9196: URL: https://github.com/apache/cloudstack/pull/9196#issuecomment-2167290570 @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] remove Project Template Permissions inhibition [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #9196: URL: https://github.com/apache/cloudstack/pull/9196#issuecomment-2167289044 @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: fix test/integration/smoke/test_register_userdata.py which caused networks not to be deleted [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9244: URL: https://github.com/apache/cloudstack/pull/9244#issuecomment-2167286686 [SF] Trillian test result (tid-10431) Environment: xcpng82 (x2), Advanced Networking with Mgmt server r8 Total time taken: 70781 seconds Marvin logs: https://github.com/b

Re: [PR] Add action button to enable/disable Oauth provider [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9242: URL: https://github.com/apache/cloudstack/pull/9242#issuecomment-2167260658 @kiranchavala 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] server: remove username required condition for listVmsForImport [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9124: URL: https://github.com/apache/cloudstack/pull/9124#issuecomment-2167260600 @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] server: remove username required condition for listVmsForImport [cloudstack]

2024-06-13 Thread via GitHub
shwstppr commented on PR #9124: URL: https://github.com/apache/cloudstack/pull/9124#issuecomment-2167259557 @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] server: remove username required condition for listVmsForImport [cloudstack]

2024-06-13 Thread via GitHub
shwstppr commented on code in PR #9124: URL: https://github.com/apache/cloudstack/pull/9124#discussion_r1639297424 ## server/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManagerImpl.java: ## @@ -2042,10 +2042,6 @@ private UserVmResponse importKvmInstance(ImportVmCmd cmd)

Re: [PR] Add configuration to limit the number of rows deleted from vm_stats [cloudstack]

2024-06-13 Thread via GitHub
github-actions[bot] commented on PR #8740: URL: https://github.com/apache/cloudstack/pull/8740#issuecomment-2167259236 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] Add action button to enable/disable Oauth provider [cloudstack]

2024-06-13 Thread via GitHub
kiranchavala commented on PR #9242: URL: https://github.com/apache/cloudstack/pull/9242#issuecomment-2167258788 @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: [I] GUI - search instance always give the first VM name and GUI is very slow [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on issue #7910: URL: https://github.com/apache/cloudstack/issues/7910#issuecomment-2167244910 Additionally fixed in https://github.com/apache/cloudstack/pull/9177 To make the listing faster, the new global setting will need to be set to false. -- This is an a

Re: [I] When deploying node to Kubernetes it times out [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud commented on issue #8975: URL: https://github.com/apache/cloudstack/issues/8975#issuecomment-2167244666 Additionally fixed in https://github.com/apache/cloudstack/pull/9177 To make the listing faster, the new global setting will need to be set to false. -- This is an a

(cloudstack) branch 4.19 updated: api: listVM API improvement followup, change returning of stats detail (#9177)

2024-06-13 Thread rohit
This is an automated email from the ASF dual-hosted git repository. rohit 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 2ca0857bd59 api: listVM API improvement followup,

Re: [PR] api: listVM API improvement followup, change returning of stats detail [cloudstack]

2024-06-13 Thread via GitHub
rohityadavcloud merged PR #9177: URL: https://github.com/apache/cloudstack/pull/9177 -- 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] Update extraconfig for platform param in xen/xcpng [cloudstack]

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

Re: [PR] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#issuecomment-2167187979 @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] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-13 Thread via GitHub
sureshanaparti commented on PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#issuecomment-2167186873 @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] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-13 Thread via GitHub
sureshanaparti commented on PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#issuecomment-2167182865 @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] api: listVM API improvement followup, change returning of stats detail [cloudstack]

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

Re: [PR] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

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

[PR] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

2024-06-13 Thread via GitHub
vishesh92 opened a new pull request, #9251: URL: https://github.com/apache/cloudstack/pull/9251 ### Description This PR fixes https://github.com/apache/cloudstack/issues/8108 As of now, on adding a new network tier to a VPC, results in a small network disruption because of keep

Re: [PR] protect against missing service offering [cloudstack]

2024-06-13 Thread via GitHub
BryanMLima commented on code in PR #9235: URL: https://github.com/apache/cloudstack/pull/9235#discussion_r1638767609 ## server/src/main/java/com/cloud/api/query/dao/VolumeJoinDaoImpl.java: ## @@ -283,6 +283,19 @@ public VolumeResponse newVolumeResponse(ResponseView view, Volume

Re: [PR] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2166712630 @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] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

2024-06-13 Thread via GitHub
vishesh92 commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2166710169 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

Re: [PR] Quota tariff order [cloudstack]

2024-06-13 Thread via GitHub
github-actions[bot] commented on PR #8347: URL: https://github.com/apache/cloudstack/pull/8347#issuecomment-2166673372 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] Decrypt account and domain configurations when needed [cloudstack]

2024-06-13 Thread via GitHub
BryanMLima commented on PR #9088: URL: https://github.com/apache/cloudstack/pull/9088#issuecomment-2166627943 > LGTM, couldn't test it locally but I'm happy with your testing @BryanMLima, would it be possible to document the changes in a doc-pr? https://github.com/apache/cloudstack-document

Re: [PR] engine-orchestration: expunge destroyed system vm volume [cloudstack]

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

(cloudstack) branch main updated: [Quota] Add API to list preset variables (#8372)

2024-06-13 Thread bryan
This is an automated email from the ASF dual-hosted git repository. bryan 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 1cf1786ebb8 [Quota] Add API to list preset variab

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

2024-06-13 Thread via GitHub
BryanMLima merged PR #8372: URL: https://github.com/apache/cloudstack/pull/8372 -- 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...@clou

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

2024-06-13 Thread via GitHub
BryanMLima commented on PR #8372: URL: https://github.com/apache/cloudstack/pull/8372#issuecomment-2166648210 Merging this one based on reviews and manual tests. -- 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] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

2024-06-13 Thread via GitHub
BryanMLima commented on code in PR #9166: URL: https://github.com/apache/cloudstack/pull/9166#discussion_r1638760488 ## ui/src/components/view/ResourceLimitTab.vue: ## @@ -112,7 +112,7 @@ export default { this.formLoading = true this.dataResource = await this.l

Re: [PR] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9166: URL: https://github.com/apache/cloudstack/pull/9166#issuecomment-2166113122 UI build failed: :heavy_multiplication_x: (SL-JID-379) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] [Quota] Improve Quota balance calculation flow [cloudstack]

2024-06-13 Thread via GitHub
BryanMLima commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-2166555417 @sureshanaparti, do you think this PR can go to the `4.19.1.0` version? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] VMware: Improve error messaging / logs when starting non-user VMs, and secondary storage not available or doesn't have enough capacity [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9207: URL: https://github.com/apache/cloudstack/pull/9207#issuecomment-2166040164 [SF] Trillian Build Failed (tid-10437) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

Re: [I] Building DEB packages: "mvn -P deps" [cloudstack]

2024-06-13 Thread via GitHub
abh1sar commented on issue #8763: URL: https://github.com/apache/cloudstack/issues/8763#issuecomment-2166522777 I wasn't able to reproduce the issue. `mvn -P deps` is working for me fine. I tried with 4.19 and main on Ubunutu. ``` 00:01 cloudstack $ mvn -P deps

Re: [PR] Fix migration from local storage to NFS in KVM [cloudstack]

2024-06-13 Thread via GitHub
JoaoJandre commented on PR #8909: URL: https://github.com/apache/cloudstack/pull/8909#issuecomment-2166499037 > I apologise @JoaoJandre, I thought this was much deeper change, what you suggest is making sense now, and the issues I'm pointing out are not related to your PR. LGTM No pr

Re: [PR] Update extraconfig for platform param in xen/xcpng [cloudstack]

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

Re: [PR] Improvements to quota tariffs APIs and UI [cloudstack]

2024-06-13 Thread via GitHub
bigsys-it commented on PR #9225: URL: https://github.com/apache/cloudstack/pull/9225#issuecomment-2166054913 > ### Description > This PR proposes some improvements to quota tariffs APIs and UI. Two parameters have been added to the `quotaTariffList` API: `id` and `listonlyremoved`. The f

Re: [I] Race condition when automatically assigning IPs to VMs [cloudstack]

2024-06-13 Thread via GitHub
abh1sar commented on issue #7907: URL: https://github.com/apache/cloudstack/issues/7907#issuecomment-2166457578 +1 to @weizhouapache and @hsato03 . #9234 fixes public IP race by taking lock before making changes to the IPAddressVO table. For guest IPs the approach would be different #9240

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
dominar250 commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2166298196 @weizhouapache Thanks for the suggestion. Is it a good idea to run management, KVM, and NFS on a single machine? Does this setup cause routing-related issues? My plan is to use

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
dominar250 commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2165987676 @DaanHoogland root@s-55-VM:~# ip a 1: lo: mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000 link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00

Re: [I] BACKUP VR of VPC Enters MASTER state and then BACKUP state when add a new VPC tier [cloudstack]

2024-06-13 Thread via GitHub
vishesh92 commented on issue #8108: URL: https://github.com/apache/cloudstack/issues/8108#issuecomment-2166424928 Working on a fix in this PR: https://github.com/apache/cloudstack/pull/9251 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

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

Re: [PR] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2166417803 Packaging result [SF]: ✖️ el7 ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 9935 -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2166402923 @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] Prevent network disruption on adding a new VPC tier for redundant VRs [cloudstack]

2024-06-13 Thread via GitHub
vishesh92 commented on PR #9251: URL: https://github.com/apache/cloudstack/pull/9251#issuecomment-2166400397 @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: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
dominar250 commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2165998754 oot@v-54-VM:~# ip a 1: lo: mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000 link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00 inet 127.0.0.1/8

Re: [I] Host is stuck in Connecting state after update to 4.18.2.0 [cloudstack]

2024-06-13 Thread via GitHub
weizhouapache commented on issue #9250: URL: https://github.com/apache/cloudstack/issues/9250#issuecomment-2166267037 > We had a similar issue in our one of our smoke tests. I created this PR https://github.com/apache/cloudstack/pull/8502 to fix it. > > @weizhouapache do you think it

Re: [PR] allow users to add description to private gateway and static routes [cloudstack]

2024-06-13 Thread via GitHub
charlesweng commented on PR #9135: URL: https://github.com/apache/cloudstack/pull/9135#issuecomment-2166217020 @DaanHoogland will look into the lint errors in sql file and double check if description is needed in network, but VPCGateway description is definitely needed, thanks! -- This

Re: [I] Host is stuck in Connecting state after update to 4.18.2.0 [cloudstack]

2024-06-13 Thread via GitHub
vishesh92 commented on issue #9250: URL: https://github.com/apache/cloudstack/issues/9250#issuecomment-2166193374 We had a similar issue in our one of our smoke tests. I created this PR https://github.com/apache/cloudstack/pull/8502 to fix it. @weizhouapache do you think it could be t

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
dominar250 commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2166171881 @weizhouapache So cloudstack could'nt work with single ip range ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] VMware: Improve error messaging / logs when starting non-user VMs, and secondary storage not available or doesn't have enough capacity [cloudstack]

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

Re: [I] Host is stuck in Connecting state after update to 4.18.2.0 [cloudstack]

2024-06-13 Thread via GitHub
weizhouapache commented on issue #9250: URL: https://github.com/apache/cloudstack/issues/9250#issuecomment-2166164052 > @weizhouapache , the only known workaround is wait for 3600 seconds? Or is there something else we can do? restarting cloudstack-agent did not help (apparently).

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
weizhouapache commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2166161399 @dominar250 I think it is because your systemvm have two nics/ips in the same range typically the public and private should use different ip range and vlan/vni.

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
dominar250 commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2166009044 [system_vm-ip.txt](https://github.com/user-attachments/files/15823200/system_vm-ip.txt) -- This is an automated message from the Apache Git Service. To respond to the mes

Re: [PR] vmware: do not tear down vm disks if deploy-as-is vm has vm snapshots [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9243: URL: https://github.com/apache/cloudstack/pull/9243#issuecomment-2165951212 [SF] Trillian Build Failed (tid-10434) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

Re: [I] Host is stuck in Connecting state after update to 4.18.2.0 [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on issue #9250: URL: https://github.com/apache/cloudstack/issues/9250#issuecomment-2166075467 @weizhouapache , the only known workaround is wait for 3600 seconds? Or is there something else we can do? -- This is an automated message from the Apache Git Service. To r

Re: [PR] Improvements on information about disks size and offering in VM deployment wizard's InfoCard [cloudstack]

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

Re: [PR] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9166: URL: https://github.com/apache/cloudstack/pull/9166#issuecomment-2166070162 @DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] fix: domain limits tab ui is now able to present 0 to the user [cloudstack]

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

Re: [PR] VMware: Improve error messaging / logs when starting non-user VMs, and secondary storage not available or doesn't have enough capacity [cloudstack]

2024-06-13 Thread via GitHub
blueorangutan commented on PR #9207: URL: https://github.com/apache/cloudstack/pull/9207#issuecomment-2166056193 @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: [I] Race condition when automatically assigning IPs to VMs [cloudstack]

2024-06-13 Thread via GitHub
hsato03 commented on issue #7907: URL: https://github.com/apache/cloudstack/issues/7907#issuecomment-2165858307 I think Wei is right, they are different situations. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

Re: [PR] VMware: Improve error messaging / logs when starting non-user VMs, and secondary storage not available or doesn't have enough capacity [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #9207: URL: https://github.com/apache/cloudstack/pull/9207#issuecomment-2166047343 @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] Update extraconfig for platform param in xen/xcpng [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #9248: URL: https://github.com/apache/cloudstack/pull/9248#issuecomment-2166033429 @blueorangutan test alma9 xcpng82 -- 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 t

Re: [I] packet loss from KVM to SSVM public and private ip address [cloudstack]

2024-06-13 Thread via GitHub
dominar250 commented on issue #9246: URL: https://github.com/apache/cloudstack/issues/9246#issuecomment-2165973903 @DaanHoogland Thanks for addressing this issue. From SSVM/consoleproxy i can able to ping my kvm machine and management server which is stable . But I can't get constant ping f

Re: [PR] VMware: Improve error messaging / logs when starting non-user VMs, and secondary storage not available or doesn't have enough capacity [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on PR #9207: URL: https://github.com/apache/cloudstack/pull/9207#issuecomment-2166031386 @blueorangutan test alma9 vmware-70u2 -- 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] Fix snapshot deletion on template creation failure [cloudstack]

2024-06-13 Thread via GitHub
DaanHoogland commented on code in PR #9239: URL: https://github.com/apache/cloudstack/pull/9239#discussion_r1638427195 ## server/src/main/java/org/apache/cloudstack/snapshot/SnapshotHelper.java: ## @@ -101,15 +101,21 @@ public void expungeTemporarySnapshot(boolean kvmSnapshotOn

  1   2   3   4   >