[I] Resetting a VM password doesn't work. (Password created while first VM setup does work) [cloudstack]

2024-01-31 Thread via GitHub
dataCobra opened a new issue, #8584: URL: https://github.com/apache/cloudstack/issues/8584 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ Cloudstack UI ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1.0 (Release version) ~~~ # CONFIGURATION Usi

Re: [I] Resetting a VM password doesn't work. (Password created while first VM setup does work) [cloudstack]

2024-01-31 Thread via GitHub
boring-cyborg[bot] commented on issue #8584: URL: https://github.com/apache/cloudstack/issues/8584#issuecomment-1920705721 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 mes

Re: [PR] New API "checkVolume" to check and repair any leaks or issues reported by qemu-img check [cloudstack]

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

Re: [PR] New API "checkVolume" to check and repair any leaks or issues reported by qemu-img check [cloudstack]

2024-01-31 Thread via GitHub
harikrishna-patnala commented on PR #8577: URL: https://github.com/apache/cloudstack/pull/8577#issuecomment-1920633245 @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 s

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
weizhouapache commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1920587867 We can backport to 4.18, then merge forward to 4.19 and main @shwstppr @nvazquez @DaanHoogland @JoaoJandre -- This is an automated message from the Apache Git Service. To re

Re: [I] UI struggles with ICMP entries in VPC ACLs [cloudstack]

2024-01-31 Thread via GitHub
rajujith commented on issue #8583: URL: https://github.com/apache/cloudstack/issues/8583#issuecomment-1920536570 @NuxRo This appears to be an issue across the UI, not all input values are displayed later on. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [WIP] System VMs/VRs upgrade to Python3 and Debian 12 [cloudstack]

2024-01-31 Thread via GitHub
blueorangutan commented on PR #8497: URL: https://github.com/apache/cloudstack/pull/8497#issuecomment-1920514968 [SF] Trillian test result (tid-9022) Environment: xcpng81 (x2), Advanced Networking with Mgmt server r8 Total time taken: 72709 seconds Marvin logs: https://github.com/bl

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
shwstppr commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1920457497 Good catch @nvazquez. If we want it in 4.19.1 I guess either we need to revert and merge again in 4.19 branch once 4.19 release is done or we would need to backport the commit. Any ot

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
nvazquez commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1920408519 Hi guys, I think in case 4.19.0 RC4 makes it to be the final release this fix won't be on 4.19.1 as the milestone states, but will be present only on the future 4.20. @BryanMLima @shw

Re: [PR] [WIP] System VMs/VRs upgrade to Python3 and Debian 12 [cloudstack]

2024-01-31 Thread via GitHub
blueorangutan commented on PR #8497: URL: https://github.com/apache/cloudstack/pull/8497#issuecomment-1920215722 [SF] Trillian test result (tid-9021) Environment: vmware-80 (x2), Advanced Networking with Mgmt server r8 Total time taken: 54592 seconds Marvin logs: https://github.com/

[I] UI struggles with ICMP entries in VPC ACLs [cloudstack]

2024-01-31 Thread via GitHub
NuxRo opened a new issue, #8583: URL: https://github.com/apache/cloudstack/issues/8583 Cloudstack 4.19rc4 It seems like the UI is having problems displaying the ICMP types and codes. It will accept the ICMP type and code, however it will not list them properly for some entries and

[I] There is no secondary storage VM for secondary storage after create 2nd zone [cloudstack]

2024-01-31 Thread via GitHub
levindecaro opened a new issue, #8582: URL: https://github.com/apache/cloudstack/issues/8582 # ISSUE TYPE * Bug Report # COMPONENT NAME ~~~ System VM ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1 ~~~ # CONFIGURATION

Re: [PR] Fix libvirt domain event listener by properly processing events [cloudstack]

2024-01-31 Thread via GitHub
mlsorensen commented on PR #8437: URL: https://github.com/apache/cloudstack/pull/8437#issuecomment-1919990649 Sorry everyone, didn't mean to disappear on this and ghost any feedback. I initially had some time to look at the problem, created the PR, and promptly switched context to other iss

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

2024-01-31 Thread via GitHub
BryanMLima commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-1919774028 @JoaoJandre, do you think this PR can go to the `4.18.2.0` release? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

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

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

Re: [PR] Display dates in Quota and Usage messages according to the timezone configurations [cloudstack]

2024-01-31 Thread via GitHub
winterhazel commented on PR #8230: URL: https://github.com/apache/cloudstack/pull/8230#issuecomment-1919661947 @DaanHoogland I have modified this PR, could you take another look at it? Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please l

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

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

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

2024-01-31 Thread via GitHub
blueorangutan commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-1919570275 @BryanMLima 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] [Quota] Improve Quota balance calculation flow [cloudstack]

2024-01-31 Thread via GitHub
BryanMLima commented on PR #8581: URL: https://github.com/apache/cloudstack/pull/8581#issuecomment-1919543628 @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

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

2024-01-31 Thread via GitHub
BryanMLima opened a new pull request, #8581: URL: https://github.com/apache/cloudstack/pull/8581 ### Description This PR aims to improve the overall Quota balance calculation by breaking into separate methods and calculating based on the period dates rather than the Quota usage recor

Re: [PR] Add logs to `LibvirtComputingResource`'s metrics collecting process [cloudstack]

2024-01-31 Thread via GitHub
winterhazel commented on code in PR #8511: URL: https://github.com/apache/cloudstack/pull/8511#discussion_r1473133257 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java: ## @@ -4401,127 +4403,193 @@ protected String getDiskPa

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919272620 > I don´t know, I didn't test it @GutoVeronezi sorry, I didn't read very well and answered hastely. Yes @GutoVeronezi, clgtm -- This is an automated message from the Apach

Re: [PR] removing the usage of volumeFreeze StorPool API call [cloudstack]

2024-01-31 Thread via GitHub
sureshanaparti commented on code in PR #8575: URL: https://github.com/apache/cloudstack/pull/8575#discussion_r1472923385 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -632,22 +590,17 @@

Re: [PR] removing the usage of volumeFreeze StorPool API call [cloudstack]

2024-01-31 Thread via GitHub
sureshanaparti commented on code in PR #8575: URL: https://github.com/apache/cloudstack/pull/8575#discussion_r1472919065 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -485,55 +484,14 @@

Re: [PR] removing the usage of volumeFreeze StorPool API call [cloudstack]

2024-01-31 Thread via GitHub
sureshanaparti commented on code in PR #8575: URL: https://github.com/apache/cloudstack/pull/8575#discussion_r1472907496 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -846,6 +791,42 @@ p

Re: [PR] removing the usage of volumeFreeze StorPool API call [cloudstack]

2024-01-31 Thread via GitHub
sureshanaparti commented on code in PR #8575: URL: https://github.com/apache/cloudstack/pull/8575#discussion_r1472904889 ## plugins/storage/volume/storpool/src/main/java/org/apache/cloudstack/storage/datastore/driver/StorPoolPrimaryDataStoreDriver.java: ## @@ -846,6 +791,42 @@ p

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
weizhouapache commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919143444 > Aside from the third-party testing, do we still need anything else on this one? > > cc: @DaanHoogland @weizhouapache @GutoVeronezi code looks ok to me.

Re: [I] register templates or ISO Failed [cloudstack]

2024-01-31 Thread via GitHub
weizhouapache commented on issue #8580: URL: https://github.com/apache/cloudstack/issues/8580#issuecomment-1919125736 > Get。So it‘s a simulator , not run well in test enviroment right. You can try with mbx to create testing environments in homelab https://github.com/shapeblue/mbx

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
weizhouapache commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1919123029 > @DaanHoogland @shwstppr, my mistake, I can revert the merge commit if it is required. No need to revert, I think @BryanMLima -- This is an automated message from

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
BryanMLima commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1919115376 @DaanHoogland @shwstppr, my mistake, I can revert the merge commit if it is required. -- This is an automated message from the Apache Git Service. To respond to the message, pleas

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919107302 I don´t know, I didn't test it @GutoVeronezi -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1919102863 Hm @BryanMLima , we were still in freeze. don't think it will be a problem though. cc @shwstppr . -- This is an automated message from the Apache Git Service. To respond to the

(cloudstack) branch main updated: Change Cryptsetup validation (#8482)

2024-01-31 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 39e0a8e8d43 Change Cryptsetup validation (#8482)

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
boring-cyborg[bot] commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1919097357 Awesome work, congrats on your first merged pull request! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
BryanMLima merged PR #8482: URL: https://github.com/apache/cloudstack/pull/8482 -- 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] Change the check for the existence of `cryptsetup` command [cloudstack]

2024-01-31 Thread via GitHub
BryanMLima commented on PR #8482: URL: https://github.com/apache/cloudstack/pull/8482#issuecomment-1919097043 Merging this based on four approvals and manual test. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] Enable over provisioning for SharedMountPoint primary storages [cloudstack]

2024-01-31 Thread via GitHub
GutoVeronezi commented on PR #8481: URL: https://github.com/apache/cloudstack/pull/8481#issuecomment-1919084553 Aside from the third-party testing, do we still need anything else on this one? cc: @DaanHoogland @weizhouapache -- This is an automated message from the Apache Git Serv

Re: [I] register templates or ISO Failed [cloudstack]

2024-01-31 Thread via GitHub
SuperXiaoxiong commented on issue #8580: URL: https://github.com/apache/cloudstack/issues/8580#issuecomment-1918994401 Get。So it‘s a simulator , not run well in test enviroment -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

Re: [I] register templates or ISO Failed [cloudstack]

2024-01-31 Thread via GitHub
SuperXiaoxiong closed issue #8580: register templates or ISO Failed URL: https://github.com/apache/cloudstack/issues/8580 -- 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

Re: [I] System vm's volumes should be expunged when a primary storage is set in Maintenance Mode [cloudstack]

2024-01-31 Thread via GitHub
JoaoJandre commented on issue #8450: URL: https://github.com/apache/cloudstack/issues/8450#issuecomment-1918967734 > @kiranchavala can I interpret this as a bug actually? volumes on a storage in maintenance are marked as destroyed. I would expect them to be either migrated or expunged indee

Re: [PR] Add logs to `LibvirtComputingResource`'s metrics collecting process [cloudstack]

2024-01-31 Thread via GitHub
JoaoJandre commented on code in PR #8511: URL: https://github.com/apache/cloudstack/pull/8511#discussion_r1472716226 ## plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java: ## @@ -4401,127 +4403,193 @@ protected String getDiskPat

Re: [PR] NSX integration [cloudstack]

2024-01-31 Thread via GitHub
blueorangutan commented on PR #7919: URL: https://github.com/apache/cloudstack/pull/7919#issuecomment-1918918569 [SF] Trillian test result (tid-9012) Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server r8 Total time taken: 54799 seconds Marvin logs: https://github.co

Re: [PR] [WIP] System VMs/VRs upgrade to Python3 and Debian 12 [cloudstack]

2024-01-31 Thread via GitHub
blueorangutan commented on PR #8497: URL: https://github.com/apache/cloudstack/pull/8497#issuecomment-1918781520 [SF] Trillian test result (tid-8998) Environment: xcpng82 (x2), Advanced Networking with Mgmt server r8 Total time taken: 72929 seconds Marvin logs: https://github.com/bl

Re: [PR] Fix typo in alert name [cloudstack]

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

Re: [PR] Fix typo in alert name [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on PR #8579: URL: https://github.com/apache/cloudstack/pull/8579#issuecomment-1918764069 @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 typo in alert name [cloudstack]

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

Re: [I] register templates or ISO Failed [cloudstack]

2024-01-31 Thread via GitHub
weizhouapache commented on issue #8580: URL: https://github.com/apache/cloudstack/issues/8580#issuecomment-1918710010 @SuperXiaoxiong the simulator environment does not have a "real" host, and system vms either. some operations will fail.. -- This is an automated message fro

Re: [I] register templates or ISO Failed [cloudstack]

2024-01-31 Thread via GitHub
boring-cyborg[bot] commented on issue #8580: URL: https://github.com/apache/cloudstack/issues/8580#issuecomment-1918683233 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 mes

[I] register templates or ISO Failed [cloudstack]

2024-01-31 Thread via GitHub
SuperXiaoxiong opened a new issue, #8580: URL: https://github.com/apache/cloudstack/issues/8580 # ISSUE TYPE ~~~ Bug Report ~~~ # COMPONENT NAME ~~~ API ~~~ # CLOUDSTACK VERSION ~~~ 4.18.1 ~~~ # CONFIGURATION

Re: [PR] New API "checkVolume" to check and repair any leaks or issues reported by qemu-img check [cloudstack]

2024-01-31 Thread via GitHub
DaanHoogland commented on code in PR #8577: URL: https://github.com/apache/cloudstack/pull/8577#discussion_r1472480827 ## engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java: ## @@ -1915,6 +1915,13 @@ public void prepare(VirtualM