Re: [I] [cloud_usage] vm_instance_id is null for volume usage (usage_type = 6) [cloudstack]
boring-cyborg[bot] commented on issue #10985: URL: https://github.com/apache/cloudstack/issues/10985#issuecomment-2950262936 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 message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[I] [cloud_usage] vm_instance_id is null for volume usage (usage_type = 6) [cloudstack]
Yuliia7-1 opened a new issue, #10985: URL: https://github.com/apache/cloudstack/issues/10985 ### problem When tracking volume usage in the `cloud_usage` table` (usage_type = 6)`, the `vm_instance_id` field is `NULL`, even if the volume is currently attached to a VM. This leads to potential inaccuracies in historical usage data, especially when volumes are detached and reattached to different VMs. Expected Behavior: ACS should record the correct `vm_instance_id` associated with a volume when usage occurs and store it in the `cloud_usage` table. ### versions The ACS version is 4.19.1.1 ### The steps to reproduce the bug Case 1: 1. Create and attach a volume to VM. 2. Use the VM for some time (allow usage data to be generated). 3. Observe that cloud_usage.vm_instance_id is still NULL for this VM. Case 2: 1. Create and attach a volume to VM A. 2. Use the VM for some time (allow usage data to be generated). 3. Detach the volume and attach it to VM B. 4. Trigger a reprocessing of usage data for the original time period. 5. Observe that cloud_usage.vm_instance_id is still NULL. ### What to do about it? Consider enhancing the ACS `UsageManagerImpl` or submitting a patch upstream to ensure `vm_instance_id` is consistently captured and stored when usage is recorded. My goal is to accurately track historical usage of volumes in relation to the VMs they were attached to. To achieve this and mitigate the issue of missing `vm_instance_id` values for` usage_type =` 6 in the `cloud_usage` table. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[I] Unable to create instance on PowerFlex storage (with small template) [cloudstack]
sureshanaparti opened a new issue, #10981: URL: https://github.com/apache/cloudstack/issues/10981 ### problem Failed to create instance on PowerFlex storage (with small template). Logs: ``` 2025-06-06 08:42:33,934 DEBUG [o.a.c.s.d.d.ScaleIOPrimaryDataStoreDriver] (Work-Job-Executor-15:[ctx-63de2f90, job-74/job-75, ctx-be8ecd3c]) (logid:9262f7e6) createAsync - creating volume 2025-06-06 08:42:33,934 DEBUG [o.a.c.s.d.d.ScaleIOPrimaryDataStoreDriver] (Work-Job-Executor-15:[ctx-63de2f90, job-74/job-75, ctx-be8ecd3c]) (logid:9262f7e6) Creating PowerFlex volume 2025-06-06 08:42:33,935 WARN [o.a.c.s.d.d.ScaleIOPrimaryDataStoreDriver] (Work-Job-Executor-15:[ctx-63de2f90, job-74/job-75, ctx-be8ecd3c]) (logid:9262f7e6) Unable to create PowerFlex Volume due to Size(GB) must be greater than 0 2025-06-06 08:42:33,935 ERROR [o.a.c.s.d.d.ScaleIOPrimaryDataStoreDriver] (Work-Job-Executor-15:[ctx-63de2f90, job-74/job-75, ctx-be8ecd3c]) (logid:9262f7e6) Unable to create PowerFlex Volume due to Size(GB) must be greater than 0 2025-06-06 08:42:33,941 DEBUG [o.a.c.s.v.VolumeServiceImpl] (Work-Job-Executor-15:[ctx-63de2f90, job-74/job-75, ctx-be8ecd3c]) (logid:9262f7e6) Failed to copy managed template on primary storage com.cloud.utils.exception.CloudRuntimeException: Creation of a volume failed: Unable to create PowerFlex Volume due to Size(GB) must be greater than 0 at org.apache.cloudstack.storage.volume.VolumeServiceImpl.createManagedVolumeCopyManagedTemplateAsync(VolumeServiceImpl.java:1178) at org.apache.cloudstack.storage.volume.VolumeServiceImpl.createManagedVolumeCopyManagedTemplateAsyncWithLock(VolumeServiceImpl.java:1562) at org.apache.cloudstack.storage.volume.VolumeServiceImpl.createManagedStorageVolumeFromTemplateAsync(VolumeServiceImpl.java:1533) at org.apache.cloudstack.engine.orchestration.VolumeOrchestrator.recreateVolume(VolumeOrchestrator.java:1780) at org.apache.cloudstack.engine.orchestration.VolumeOrchestrator.prepare(VolumeOrchestrator.java:1944) ``` Also, noticed Storpool Adaptor call during cleanup and resulting in in NPE, which needs fix. ``` 2025-06-06 08:45:49,621 DEBUG [c.c.a.t.Request] (AgentManager-Handler-19:[]) (logid:) Seq 1-4269412446747230759: Processing: { Ans: , MgmtId: 32986389020927, via: 1, Ver: v1, Flags: 10, [{"com.cloud.agent.api.Answer":{"result":"false","details":"java.lang.NullPointerException: Cannot invoke "String.startsWith(String)" because "volumeUuid" is null at com.cloud.hypervisor.kvm.storage.StorPoolStorageAdaptor.getVolumeNameFromPath(StorPoolStorageAdaptor.java:142) at com.cloud.hypervisor.kvm.storage.StorPoolStorageAdaptor.attachOrDetachVolume(StorPoolStorageAdaptor.java:152) at com.cloud.hypervisor.kvm.storage.StorPoolStorageAdaptor.disconnectPhysicalDisk(StorPoolStorageAdaptor.java:271) at com.cloud.hypervisor.kvm.storage.KVMStoragePoolManager.disconnectPhysicalDisk(KVMStoragePoolManager.java:221) at com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.cleanupDisk(LibvirtComputingResource.java:3452) at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtStopCommandWrapper.execute(LibvirtStopCommandWrapper.java:112) at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtStopCommandWrapper.execute(LibvirtStopCommandWrapper.java:47) at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtRequestWrapper.execute(LibvirtRequestWrapper.java:78) at com.cloud.hypervisor.kvm.resource.LibvirtComputingResource.executeRequest(LibvirtComputingResource.java:1958) at com.cloud.agent.Agent.processRequest(Agent.java:779) ``` ### versions KVM + Dell PowerFlex storage ### The steps to reproduce the bug 1. Add PowerFlex pool as primary storage using tag 'powerflex' 2. Register template with small size, eg. macchinina here: http://dl.openvm.eu/cloudstack/macchinina/x86_64/macchinina-kvm.qcow2.bz2 3. Create a Service/Compute Offering with storage tag 'powerflex' 4. Create instance using the above register template when it is ready and service offering ### What to do about it? Expected: Should be able to create instances using smaller templates on PowerFlex storage. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Unable to create instance on PowerFlex storage (with smaller templates) [cloudstack]
weizhouapache commented on issue #10981: URL: https://github.com/apache/cloudstack/issues/10981#issuecomment-2948648832 @sureshanaparti which ACS version ? it seems like a blocker -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Unable to create instance on PowerFlex storage (with smaller templates) [cloudstack]
sureshanaparti commented on issue #10981: URL: https://github.com/apache/cloudstack/issues/10981#issuecomment-2948656614 > [@sureshanaparti](https://github.com/sureshanaparti) which ACS version ? it seems like a blocker noticed in 4.20.1. RC3. I think, it exists before as well. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(cloudstack) branch 4.20 updated (823080cc9c3 -> c86b2f1da24)
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch 4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 823080cc9c3 systemvmtemplate: fix Debian 12.11.0 ISO url (#10927) add c61a5eb430a Updating pom.xml version numbers for release 4.20.1.0 add b5e2c181f9c Updating pom.xml version numbers for release 4.20.2.0-SNAPSHOT add c86b2f1da24 update 4.20.2.0-SNAPSHOT version No new revisions were added by this update. Summary of changes: agent/pom.xml| 2 +- api/pom.xml | 2 +- client/pom.xml | 2 +- core/pom.xml | 2 +- debian/changelog | 12 developer/pom.xml| 2 +- engine/api/pom.xml | 2 +- engine/components-api/pom.xml| 2 +- engine/orchestration/pom.xml | 2 +- engine/pom.xml | 2 +- engine/schema/pom.xml| 2 +- engine/service/pom.xml | 2 +- engine/storage/cache/pom.xml | 2 +- engine/storage/configdrive/pom.xml | 2 +- engine/storage/datamotion/pom.xml| 2 +- engine/storage/image/pom.xml | 2 +- engine/storage/integration-test/pom.xml | 2 +- engine/storage/object/pom.xml| 2 +- engine/storage/pom.xml | 2 +- engine/storage/snapshot/pom.xml | 2 +- engine/storage/volume/pom.xml| 2 +- engine/userdata/cloud-init/pom.xml | 2 +- engine/userdata/pom.xml | 2 +- framework/agent-lb/pom.xml | 2 +- framework/ca/pom.xml | 2 +- framework/cluster/pom.xml| 2 +- framework/config/pom.xml | 2 +- framework/db/pom.xml | 2 +- framework/direct-download/pom.xml| 2 +- framework/events/pom.xml | 2 +- framework/ipc/pom.xml| 2 +- framework/jobs/pom.xml | 2 +- framework/managed-context/pom.xml| 2 +- framework/pom.xml| 2 +- framework/quota/pom.xml | 2 +- framework/rest/pom.xml | 2 +- framework/security/pom.xml | 2 +- framework/spring/lifecycle/pom.xml | 2 +- framework/spring/module/pom.xml | 2 +- plugins/acl/dynamic-role-based/pom.xml | 2 +- plugins/acl/project-role-based/pom.xml | 2 +- plugins/acl/static-role-based/pom.xml| 2 +- .../affinity-group-processors/explicit-dedication/pom.xml| 2 +- plugins/affinity-group-processors/host-affinity/pom.xml | 2 +- plugins/affinity-group-processors/host-anti-affinity/pom.xml | 2 +- .../non-strict-host-affinity/pom.xml | 2 +- .../non-strict-host-anti-affinity/pom.xml| 2 +- plugins/alert-handlers/snmp-alerts/pom.xml | 2 +- plugins/alert-handlers/syslog-alerts/pom.xml | 2 +- plugins/api/discovery/pom.xml| 2 +- plugins/api/rate-limit/pom.xml | 2 +- plugins/api/solidfire-intg-test/pom.xml | 2 +- plugins/api/vmware-sioc/pom.xml | 2 +- plugins/backup/dummy/pom.xml | 2 +- plugins/backup/nas/pom.xml | 2 +- plugins/backup/networker/pom.xml | 2 +- plugins/backup/veeam/pom.xml | 2 +- plugins/ca/root-ca/pom.xml | 2 +- plugins/database/mysql-ha/pom.xml| 2 +- plugins/database/quota/pom.xml | 2 +- plugins/dedicated-resources/pom.xml | 2 +- plugins/deployment-planners/implicit-dedication/pom.xml | 2 +- plugins/deployment-planners/user-concentrated-pod/pom.xml| 2 +- plugins/deployment-planners/user-dispersing/p
Re: [PR] Support of snapshot copy to primary storage in different zones. [cloudstack]
slavkap commented on PR #9478: URL: https://github.com/apache/cloudstack/pull/9478#issuecomment-2949084368 Thank you @DaanHoogland, @sureshanaparti and @nvazquez for the reviews! I think I addressed all the requested changes -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(cloudstack) tag 4.19.3.0 created (now 0c7d47138d4)
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to tag 4.19.3.0 in repository https://gitbox.apache.org/repos/asf/cloudstack.git at 0c7d47138d4 (commit) No new revisions were added by this update.
[PR] Fixed and enhanced vlan field validation in the UI [cloudstack]
erikbocks opened a new pull request, #10983: URL: https://github.com/apache/cloudstack/pull/10983 ### Description This PR fixes and enhances the validation of the values inserted in the VLAN/VNI field when configuring a new zone. Currently, it only checks if any of the fields are empty and if the range start value is greater than the range end value. Thus, it is possible to insert invalid values, as negatives. With the PR changes, now the field validates if: any of the values is negative, any of the fields is empty or if the inserted values are inside the isolation method max and min values. The maximum and minimum values are now defined based on the selected isolation method (isolation method limits used can be find in the `NetworkServiceImpl` class). This PR also fixes a minor graphical issue, where the field displayed a red check icon, instead of a red error icon. ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] New feature (non-breaking change which adds functionality) - [ ] Bug fix (non-breaking change which fixes an issue) - [X] Enhancement (improves an existing feature and functionality) - [ ] Cleanup (Code refactoring and cleanup, that may add test cases) - [ ] build/CI - [ ] test (unit or integration test code) ### Feature/Enhancement Scale or Bug Severity Feature/Enhancement Scale - [ ] Major - [X] Minor ### Screenshots (if appropriate): Before the changes:  After the changes:  ### How Has This Been Tested? 1. Simulated the creation of a new zone, then validated the fix with the following tests: VLAN | Test| Start | End | Valid | |:---:|:--:|::|:--:| | Empty | Null | Null |False | | Negative value | -10 | 100 |False | | Equal | 100 | 100 |False | | Start > End | 100 | 10 |False | | End > Limit |1 | 4096 |False | | Start < Limit |0 | 100 |False | | Valid |1 | 100 | True | VXLAN | Test| Start |End | Valid | |:---:|:--:|::|:--:| | Empty | Null | Null |False | | Negative value | -4090 | 4100 |False | | Equal | 4096 | 4096 |False | | Start > End | 4100 | 4096 |False | | End > Limit | 4096 | 16777215 |False | | Start < Limit | 4095 | 4100 |False | | Valid | 4096 | 4100 | True | GRE | Test| Start | End| Valid | |:---:|:--:|:--:|:--:| | Empty | Null |Null|False | | Negative value | -10 | 100|False | | Equal | 100 | 100|False | | Start > End | 100 | 99 |False | | End > Limit |1 | 4294967296 |False | | Start < Limit | -1 | 100|False | | Valid |1 |4000| True | -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
miloserdoff commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2949345204 @weizhouapache I can't quite figure out what parameter to enable and what URLs to pass where to enable tags launchSecurity type='sev-snp' cbitpos>5160x0003
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
weizhouapache commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2949464428 > @weizhouapache I currently have these settings enabled. > >  > >  > >  > > > Please tell me if I'm doing something wrong? @miloserdoff extraconfig is different from userdata Currently it is unsupported on UI, you can set it via cmk -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Support of snapshot copy to primary storage in different zones. [cloudstack]
blueorangutan commented on PR #9478: URL: https://github.com/apache/cloudstack/pull/9478#issuecomment-2949413360 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13649 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] syslog [cloudstack]
DaanHoogland opened a new pull request, #10980: URL: https://github.com/apache/cloudstack/pull/10980 ### Description This PR updates the syslog configuration log4j 2 ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] New feature (non-breaking change which adds functionality) - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] Enhancement (improves an existing feature and functionality) - [ ] Cleanup (Code refactoring and cleanup, that may add test cases) - [ ] build/CI - [ ] test (unit or integration test code) ### Feature/Enhancement Scale or Bug Severity Feature/Enhancement Scale - [ ] Major - [ ] Minor Bug Severity - [ ] BLOCKER - [ ] Critical - [ ] Major - [ ] Minor - [ ] Trivial ### Screenshots (if appropriate): ### How Has This Been Tested? How did you try to break this feature and the system with this change? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] syslog [cloudstack]
weizhouapache commented on PR #10980: URL: https://github.com/apache/cloudstack/pull/10980#issuecomment-2948574082 @DaanHoogland can you give more details in the description and the title ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(cloudstack) annotated tag 4.20.1.0 updated (c61a5eb430a -> 73f1d556056)
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to annotated tag 4.20.1.0 in repository https://gitbox.apache.org/repos/asf/cloudstack.git *** WARNING: tag 4.20.1.0 was modified! *** from c61a5eb430a (commit) to 73f1d556056 (tag) tagging c61a5eb430a8ced2b10d7cec9948a8704c2c52ac (commit) replaces 4.19.2.0 by Pearl Dsilva on Fri May 30 12:43:13 2025 +0530 - Log - Tagging release 4.20.1.0 on branch 4.20. -BEGIN PGP SIGNATURE- iQGzBAABCgAdFiEE0VtE++Mz3EfncNJjZw6rE+9EPscFAmg5WokACgkQZw6rE+9E PsdZ1gv/Yu69mNWHOB8Ln85sePAj+scA/BKzFNRijo5hSsC/hIiSIBHabdNBxAV1 qb1BNoHXdMLmAs3S1/u7WH6j3a5SAzDPoOFfCgBWIboMeYlzzQDafIPc0qWmlQMg GJVy98mDN5/q8rqpgQJFeW16TmmFgHBcqFPOLlwI0RLMV4xNBOPFJ0ipfY3dqeaj vjYRohqdMgONi628Bieyr7jqclLDem4gdplUy3yXLJJD329N1q4ced0ls6gVaan/ 5tmvPmseaZtShGDR3VJps0xyQq1XxILyDpGoBnMEuUe+LzYqnQlawD33R04Xvh8w RAZmyWniwUDzNe+pEvx49ORCJ6eiTucQX+Jpjqe80wNPKq6/vxHKHKDxZG52p3kZ G7LlHZw6rlPmUslGdSwhJkhlKVGj8Nch6jBSrr2Bb8YmlMKv+j+7AfceEdj36tAV tpsvLshHhcWZiOkNVroSxMcz1pbq3yOak/5t6jywHbhIjHRXlz2ewGRCnDcTwNJ4 TK8kkblk =CxAL -END PGP SIGNATURE- --- No new revisions were added by this update. Summary of changes:
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
miloserdoff commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2949026635 > > [@weizhouapache](https://github.com/weizhouapache) When I tried to launch the guest OS via qemu-system-x86_64, everything was fine. > > does the guest OS (not host) support SEV-SNP ? if yes, can you share the dumpxml of the VM ? ` VM_NAME uid Debian GNU/Linux 12 (64-bit) 8392704 8392704 1 87 /machine Apache Software Foundation CloudStack KVM Hypervisor uid uid hvm /path/to/OVMF.fd /path/to/OVMF_VARS.fd destroy restart destroy /path/to/qemu-system-x86_64 138ce8b4b3d84e34bcbe 51 6 0x0003 ` This is the libvirt configuration that allowed me to start the VM and enable SEV-SNP on it. [ PASS ] - SEV: ENABLED [ PASS ] - SEV-ES: ENABLED [ PASS ] - SNP: ENABLED [ PASS ] - Optional Features statuses: [ PASS ] - VTOM: DISABLED [ PASS ] - ReflectVC: DISABLED [ PASS ] - Restricted Injection: DISABLED [ PASS ] - Alternate Injection: DISABLED [ PASS ] - Debug Swap: DISABLED [ PASS ] - Prevent Host IBS: DISABLED [ PASS ] - SNP BTB Isolation: DISABLED [ PASS ] - VMPL SSS: DISABLED [ PASS ] - Secure TSE: DISABLED [ PASS ] - VMG Exit Parameter: DISABLED [ PASS ] - IBS Virtualization: DISABLED [ PASS ] - VMSA Reg Prot: DISABLED [ PASS ] - SMT Protection: DISABLED -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] syslog [cloudstack]
DaanHoogland commented on PR #10980: URL: https://github.com/apache/cloudstack/pull/10980#issuecomment-2949243459 > @DaanHoogland can you give more details in the description and the title ? will do, work in prgress though (untested as of yet) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
miloserdoff commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2949360494 @weizhouapache I currently have these settings enabled.    Please tell me if I'm doing something wrong? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Support of snapshot copy to primary storage in different zones. [cloudstack]
nvazquez commented on PR #9478: URL: https://github.com/apache/cloudstack/pull/9478#issuecomment-2949226000 @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 comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(cloudstack) branch 4.20.1.0-RC20250530T1243 deleted (was c61a5eb430a)
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch 4.20.1.0-RC20250530T1243 in repository https://gitbox.apache.org/repos/asf/cloudstack.git was c61a5eb430a Updating pom.xml version numbers for release 4.20.1.0 The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
Re: [PR] Support of snapshot copy to primary storage in different zones. [cloudstack]
blueorangutan commented on PR #9478: URL: https://github.com/apache/cloudstack/pull/9478#issuecomment-2949228600 @nvazquez a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
miloserdoff commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2949263759 @weizhouapache Thanks for the answer. Changing the xml configuration is done manually. How to add all the necessary fields automatically when creating a virtual machine, so as not to go to the virsh settings and add settings manually? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] syslog [cloudstack]
codecov[bot] commented on PR #10980: URL: https://github.com/apache/cloudstack/pull/10980#issuecomment-2948569112 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10980?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report All modified and coverable lines are covered by tests :white_check_mark: > Project coverage is 4.00%. Comparing base [(`823080c`)](https://app.codecov.io/gh/apache/cloudstack/commit/823080cc9c379bfdbb47812bec0db1206c4ef566?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`ff1513c`)](https://app.codecov.io/gh/apache/cloudstack/commit/ff1513c19558047b117d40d7e07fb47d64dcdfd8?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). > :exclamation: There is a different number of reports uploaded between BASE (823080c) and HEAD (ff1513c). Click for more details. > > HEAD has 1 upload less than BASE > >| Flag | BASE (823080c) | HEAD (ff1513c) | >|--|--|--| >|unittests|1|0| > Additional details and impacted files ```diff @@ Coverage Diff @@ ## 4.20 #10980 +/- ## = - Coverage 16.14%4.00% -12.15% = Files 5656 401 -5255 Lines49789332598 -465295 Branches 60374 5788-54586 = - Hits 80405 1305-79100 + Misses 40852931144 -377385 + Partials 8959 149 -8810 ``` | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/10980/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [uitests](https://app.codecov.io/gh/apache/cloudstack/pull/10980/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `4.00% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/10980/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more. [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/cloudstack/pull/10980?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :rocket: New features to boost your workflow: - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(cloudstack) branch netris-integration-upstream updated (ae99f249ac8 -> 23f0ff1bab2)
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch netris-integration-upstream in repository https://gitbox.apache.org/repos/asf/cloudstack.git from ae99f249ac8 [VR] Fix IPv6 NIC IP on the VR add 96da94f9df8 Make reorder ACL items invoke Netris controller add 0cc961df0b1 VR: advertise SLAAC prefix only if VR is gateway add a97c5df63c6 VR: fix radvd misconfiguration for non-netris env add 89fa7008c45 [VR] Fix object comparisson to string comparisson on python add c657a9e0c9c Fix unit tests for ACL add 23f0ff1bab2 [UI] Zone wizard creation improvements - rename hostname to url and remove port for Netris Provider (#77) No new revisions were added by this update. Summary of changes: .../java/com/cloud/network/vpc/NetworkACLManager.java | 2 +- .../service/NetrisProviderServiceImplTest.java | 4 .../com/cloud/network/vpc/NetworkACLManagerImpl.java| 11 ++- .../com/cloud/network/vpc/NetworkACLServiceImpl.java| 11 +-- .../test/java/com/cloud/vpc/NetworkACLServiceTest.java | 6 ++ systemvm/debian/opt/cloud/bin/cs/CsVpcGuestNetwork.py | 17 - 6 files changed, 30 insertions(+), 21 deletions(-)
[I] [VMware to KVM Migration] Issue on migration when ovftool is not installed on the conversion KVM hosts [cloudstack]
nvazquez opened a new issue, #10982: URL: https://github.com/apache/cloudstack/issues/10982 ### problem There is a problem on VMware to KVM migration in case the converting host doesn't have ovftool installed: conversion process fails with: Cannot invoke org.apache.cloudstack.vm.UnmanagedInstanceTO.setCpuCores(java.lang.Integer)" because "convertedInstance is null Workaround: install ovftool on the KVM hosts used for conversion The reason of the failure is that the returned converted instance on line: https://github.com/apache/cloudstack/blob/4.20/server/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManagerImpl.java#L1985 is null. This logic is missing the import instance logic that is present on: https://github.com/apache/cloudstack/blob/4.20/server/src/main/java/org/apache/cloudstack/vm/UnmanagedVMsManagerImpl.java#L2036-L2056. ### versions CloudStack 4.20.1 RC3 KVM hosts installing virt-v2v ### The steps to reproduce the bug 1. Install only virt-v2v on KVM host used for conversion, but not ovftool 2. Migrate VMware VM to KVM through CloudStack ### What to do about it? Install ovftool on the KVM hosts used for conversion -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
miloserdoff commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2948907333 @sureshanaparti Hello, Does this mean that Cloudstack 4.21.0 will add support for SEV-SNP on VMs? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
(cloudstack) branch 4.19 updated (5be6b7980df -> 0d5a0ea6815)
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch 4.19 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from 5be6b7980df Skip verification of privileges on project accounts add 864327b8365 Skip verification of privileges on project accounts add dd066e0e0e0 Merge branch '4.19' of https://github.com/apache/cloudstack into 4.19 add 0c7d47138d4 Updating pom.xml version numbers for release 4.19.3.0 add c4ba601a636 Merge branch '4.19.3.0-RC20250530T0908' into 4.19 add 379ee07d88b Updating pom.xml version numbers for release 4.19.4.0-SNAPSHOT add 0d5a0ea6815 update 4.19.4.0-SNAPSHOT version No new revisions were added by this update. Summary of changes: agent/pom.xml| 2 +- api/pom.xml | 2 +- client/pom.xml | 2 +- core/pom.xml | 2 +- debian/changelog | 12 developer/pom.xml| 2 +- engine/api/pom.xml | 2 +- engine/components-api/pom.xml| 2 +- engine/orchestration/pom.xml | 2 +- engine/pom.xml | 2 +- engine/schema/pom.xml| 2 +- engine/service/pom.xml | 2 +- engine/storage/cache/pom.xml | 2 +- engine/storage/configdrive/pom.xml | 2 +- engine/storage/datamotion/pom.xml| 2 +- engine/storage/image/pom.xml | 2 +- engine/storage/integration-test/pom.xml | 2 +- engine/storage/object/pom.xml| 2 +- engine/storage/pom.xml | 2 +- engine/storage/snapshot/pom.xml | 2 +- engine/storage/volume/pom.xml| 2 +- engine/userdata/cloud-init/pom.xml | 2 +- engine/userdata/pom.xml | 2 +- framework/agent-lb/pom.xml | 2 +- framework/ca/pom.xml | 2 +- framework/cluster/pom.xml| 2 +- framework/config/pom.xml | 2 +- framework/db/pom.xml | 2 +- framework/direct-download/pom.xml| 2 +- framework/events/pom.xml | 2 +- framework/ipc/pom.xml| 2 +- framework/jobs/pom.xml | 2 +- framework/managed-context/pom.xml| 2 +- framework/pom.xml| 2 +- framework/quota/pom.xml | 2 +- framework/rest/pom.xml | 2 +- framework/security/pom.xml | 2 +- framework/spring/lifecycle/pom.xml | 2 +- framework/spring/module/pom.xml | 2 +- plugins/acl/dynamic-role-based/pom.xml | 2 +- plugins/acl/project-role-based/pom.xml | 2 +- plugins/acl/static-role-based/pom.xml| 2 +- .../affinity-group-processors/explicit-dedication/pom.xml| 2 +- plugins/affinity-group-processors/host-affinity/pom.xml | 2 +- plugins/affinity-group-processors/host-anti-affinity/pom.xml | 2 +- .../non-strict-host-affinity/pom.xml | 2 +- .../non-strict-host-anti-affinity/pom.xml| 2 +- plugins/alert-handlers/snmp-alerts/pom.xml | 2 +- plugins/alert-handlers/syslog-alerts/pom.xml | 2 +- plugins/api/discovery/pom.xml| 2 +- plugins/api/rate-limit/pom.xml | 2 +- plugins/api/solidfire-intg-test/pom.xml | 2 +- plugins/api/vmware-sioc/pom.xml | 2 +- plugins/backup/dummy/pom.xml | 2 +- plugins/backup/networker/pom.xml | 2 +- plugins/backup/veeam/pom.xml | 2 +- plugins/ca/root-ca/pom.xml | 2 +- plugins/database/mysql-ha/pom.xml| 2 +- plugins/database/quota/pom.xml | 2 +- plugins/dedicated-resources/pom.xml | 2 +- plugins/deployment-pl
(cloudstack) branch 4.20 updated (c86b2f1da24 -> 10fc1f3a1c5)
This is an automated email from the ASF dual-hosted git repository. pearl11594 pushed a change to branch 4.20 in repository https://gitbox.apache.org/repos/asf/cloudstack.git from c86b2f1da24 update 4.20.2.0-SNAPSHOT version add 10fc1f3a1c5 update version No new revisions were added by this update. Summary of changes: debian/changelog | 2 +- tools/marvin/setup.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
(cloudstack) branch 4.19.3.0-RC20250530T0908 deleted (was 0c7d47138d4)
This is an automated email from the ASF dual-hosted git repository. dahn pushed a change to branch 4.19.3.0-RC20250530T0908 in repository https://gitbox.apache.org/repos/asf/cloudstack.git was 0c7d47138d4 Updating pom.xml version numbers for release 4.19.3.0 The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
Re: [I] Starting a virtual machine with SEV-SNP parameters (qemu-system-x86-64) [cloudstack]
weizhouapache commented on issue #10009: URL: https://github.com/apache/cloudstack/issues/10009#issuecomment-2949208004 > > > [@weizhouapache](https://github.com/weizhouapache) When I tried to launch the guest OS via qemu-system-x86_64, everything was fine. > > > > > > does the guest OS (not host) support SEV-SNP ? if yes, can you share the dumpxml of the VM ? > > [sev-snp.txt](https://github.com/user-attachments/files/20627699/sev-snp.txt) > > This is the libvirt configuration that allowed me to start the VM and enable SEV-SNP on it. [ PASS ] - SEV: ENABLED [ PASS ] - SEV-ES: ENABLED [ PASS ] - SNP: ENABLED [ PASS ] - Optional Features statuses: [ PASS ] - VTOM: DISABLED [ PASS ] - ReflectVC: DISABLED [ PASS ] - Restricted Injection: DISABLED [ PASS ] - Alternate Injection: DISABLED [ PASS ] - Debug Swap: DISABLED [ PASS ] - Prevent Host IBS: DISABLED [ PASS ] - SNP BTB Isolation: DISABLED [ PASS ] - VMPL SSS: DISABLED [ PASS ] - Secure TSE: DISABLED [ PASS ] - VMG Exit Parameter: DISABLED [ PASS ] - IBS Virtualization: DISABLED [ PASS ] - VMSA Reg Prot: DISABLED [ PASS ] - SMT Protection: DISABLED @miloserdoff I commented on another issue some days ago: https://github.com/apache/cloudstack/discussions/9912#discussioncomment-12794076 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Add global setting to enable/disable service offering creation for end users (optionally zone-scoped) [cloudstack]
luganofer commented on issue #10886: URL: https://github.com/apache/cloudstack/issues/10886#issuecomment-2950667207 > > Hi [@DaanHoogland](https://github.com/DaanHoogland) , thanks for the feedback. > > np > > > If you have hundreds of accounts and thousands of users that belong to an account with the role ‘Domain Admin’ (role that is assigned to an end customer by default in most cases) you have to think about a migration process of creating not only the roles that deny those APIs (that wouldn't be much effort) , you also have to recreate the hundreds of accounts with the new role and also recreate the thousands of users in these new accounts to inherit this new role and force the thousands of users to re-register the 2FA if this feature is active. Unless you do something nasty to the database, of course. > > Why do you think the recreation of accounts is needed? A account can be assigned a new role, and this can be scripted with APIs. These are the ones you would need: https://cloudstack.apache.org/api/apidocs-4.20/apis/createRole.html https://cloudstack.apache.org/api/apidocs-4.20/apis/updateRolePermission.html https://cloudstack.apache.org/api/apidocs-4.20/apis/updateAccount.html only the last one would be called on all those users. The users/accounts will continue to exist with all their data. You are right 🥇 @DaanHoogland . From the APIs it is possible to assign a new role to the accounts and I wasn't aware of that ( thank you for the tip!). The good thing about this is we can make the changes we need without waiting for the new functionality 😄 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Add global setting to enable/disable service offering creation for end users (optionally zone-scoped) [cloudstack]
luganofer closed issue #10886: Add global setting to enable/disable service offering creation for end users (optionally zone-scoped) URL: https://github.com/apache/cloudstack/issues/10886 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] [CKS] Create Kubernetes ISO support for ARCH optional parameter [cloudstack]
nvazquez opened a new pull request, #10986: URL: https://github.com/apache/cloudstack/pull/10986 ### Description This PR introduces an optional parameter ARCH for the `create-kubernetes-binaries-iso.sh` script to include the corresponding arch binaries. ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] New feature (non-breaking change which adds functionality) - [ ] Bug fix (non-breaking change which fixes an issue) - [x] Enhancement (improves an existing feature and functionality) - [ ] Cleanup (Code refactoring and cleanup, that may add test cases) - [ ] build/CI - [ ] test (unit or integration test code) ### Feature/Enhancement Scale or Bug Severity Feature/Enhancement Scale - [ ] Major - [ ] Minor Bug Severity - [ ] BLOCKER - [ ] Critical - [ ] Major - [ ] Minor - [ ] Trivial ### Screenshots (if appropriate): ### How Has This Been Tested? Tested three scenarios: - Not passing the ARCH pameter -> observe the script downloads amd64 binaries on the ISO - Passing ARCH=arm64 -> observe the script downloads arm64 binaries on the ISO - Passing ARCH=aaarm -> ERROR How did you try to break this feature and the system with this change? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [CKS] Create Kubernetes ISO support for ARCH optional parameter [cloudstack]
codecov[bot] commented on PR #10986: URL: https://github.com/apache/cloudstack/pull/10986#issuecomment-2950713810 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10986?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report All modified and coverable lines are covered by tests :white_check_mark: > Project coverage is 3.96%. Comparing base [(`41b4f0a`)](https://app.codecov.io/gh/apache/cloudstack/commit/41b4f0afd5321e987973b615b566365e48228c6e?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`e2beb9b`)](https://app.codecov.io/gh/apache/cloudstack/commit/e2beb9b6f7cfa7b4321be4b824be91b5de5536d2?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). > :exclamation: There is a different number of reports uploaded between BASE (41b4f0a) and HEAD (e2beb9b). Click for more details. > > HEAD has 1 upload less than BASE > >| Flag | BASE (41b4f0a) | HEAD (e2beb9b) | >|--|--|--| >|unittests|1|0| > Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10986 +/- ## = - Coverage 16.57%3.96% -12.62% = Files 5719 405 -5314 Lines50720032959 -474241 Branches 61574 5869-55705 = - Hits 84093 1307-82786 + Misses 41368831502 -382186 + Partials 9419 150 -9269 ``` | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/10986/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [uitests](https://app.codecov.io/gh/apache/cloudstack/pull/10986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `3.96% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/10986/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more. [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/cloudstack/pull/10986?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :rocket: New features to boost your workflow: - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] Fix data being replicated on VM's metadata file in VR [cloudstack]
GutoVeronezi opened a new pull request, #10987: URL: https://github.com/apache/cloudstack/pull/10987 ### Description While creating VMs, ACS configures HTTPd on the VR to serve metadata files to be consumed by CloudInit. However, the file `/var/www/html/meta-data//metadata` is appended with the same content every time a VM is started through CloudStack, causing the file to have replicated lines; as VMs are created/started, this file grows _ad infinitum_, consuming the VR's storage, and making Apache HTTPd consume more memory than it should. This PR intends to fix this metadata replication on the VRs. ### Types of changes - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] New feature (non-breaking change which adds functionality) - [X] Bug fix (non-breaking change which fixes an issue) - [ ] Enhancement (improves an existing feature and functionality) - [ ] Cleanup (Code refactoring and cleanup, that may add test cases) - [ ] build/CI - [ ] test (unit or integration test code) ### Feature/Enhancement Scale or Bug Severity Bug Severity - [ ] BLOCKER - [ ] Critical - [ ] Major - [X] Minor - [ ] Trivial ### Screenshots (if appropriate): ### How Has This Been Tested? Without the changes: 1. I created a network; 2. I created a VM; 3. I checked the VM's metadata file in the VR, and observed that it had only a single entry for each attribute; 4. I checked the metadata in the VM; 5. I stopped the VM; 6. I started the VM again; 7. I checked that the VM's metadata file in the VR had duplicated entries for every attribute; 8. I checked the metadata in the VM; With the changes: 1. I created a network; 2. I created a VM; 3. I checked the VM's metadata file in the VR, and observed that it had only a single entry for each attribute; 4. I checked the metadata in the VM; 5. I stopped the VM; 6. I started the VM again; 7. I checked the VM's metadata file in the VR, and observed that it had only a single entry for each attribute; 8. I checked the metadata in the VM; -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Fix data being replicated on VM's metadata file in VR [cloudstack]
codecov[bot] commented on PR #10987: URL: https://github.com/apache/cloudstack/pull/10987#issuecomment-2950833816 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10987?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report All modified and coverable lines are covered by tests :white_check_mark: > Project coverage is 3.96%. Comparing base [(`41b4f0a`)](https://app.codecov.io/gh/apache/cloudstack/commit/41b4f0afd5321e987973b615b566365e48228c6e?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`2172c9e`)](https://app.codecov.io/gh/apache/cloudstack/commit/2172c9e5be26034fff345ef57dcf19dc8a019518?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). > :exclamation: There is a different number of reports uploaded between BASE (41b4f0a) and HEAD (2172c9e). Click for more details. > > HEAD has 1 upload less than BASE > >| Flag | BASE (41b4f0a) | HEAD (2172c9e) | >|--|--|--| >|unittests|1|0| > Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #10987 +/- ## = - Coverage 16.57%3.96% -12.62% = Files 5719 405 -5314 Lines50720032959 -474241 Branches 61574 5869-55705 = - Hits 84093 1307-82786 + Misses 41368831502 -382186 + Partials 9419 150 -9269 ``` | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/10987/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [uitests](https://app.codecov.io/gh/apache/cloudstack/pull/10987/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `3.96% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/10987/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more. [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/cloudstack/pull/10987?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :rocket: New features to boost your workflow: - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] [Vmware to KVM Migration] Preserve boot type and boot mode of instances to be migrated [cloudstack]
blueorangutan commented on PR #10975: URL: https://github.com/apache/cloudstack/pull/10975#issuecomment-2950916804 [SF] Trillian test result (tid-13475) Environment: vmware-70u3 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 63409 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10975-t13475-vmware-70u3.zip Smoke tests completed. 139 look OK, 2 have errors, 0 did not run Only failed and skipped tests results shown below: Test | Result | Time (s) | Test File --- | --- | --- | --- test_deploy_more_vms_than_limit_allows | `Error` | 150.11 | test_deploy_vms_in_parallel.py test_01_scale_vm | `Error` | 1.52 | test_scale_vm.py test_02_scale_vm_negative_offering_disable_scaling | `Error` | 1.40 | test_scale_vm.py test_03_scale_vm_negative_vm_disable_scaling | `Error` | 1.39 | test_scale_vm.py -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] Support firewall rules on Public IP in VPC [cloudstack]
CodeBleu commented on issue #8863: URL: https://github.com/apache/cloudstack/issues/8863#issuecomment-2950929370 Not sure if this is the correct place to ask or not, but with my current issue of locking down a Load Balancer in a VPC network. I've ran into issues where the VPC network ACL doesn't work for a Load Balancer and I'm not sure what the expected behavior should be? If I create or [update source CIDR](https://github.com/apache/cloudstack/pull/10968) of the LB rule it will restrict based on the source CIDR, however nothing in the VPC ACL will actually restrict access to my LB This is completely different behavior when it's a non-VPC. A non-VPC network will actually use the firewall correctly to restrict the access. I believe the Load Balancer source CIDR itself should be the first line of defense and if it is set to (0.0.0.0/0 - default), then the firewall ( non-vpc ) or ACL ( VPC ) should be the next line of defense. Any help in the direction this should go would be much appreciated. I'm needing this to work for the cloudstack-kubernetes-provider so it can set the LB source CIDR on creation, as the creation of the ACL's are not working and just leaves things wide open based on the default source CIDR of the LB being open by default. @Pearl1594 It appears you last worked on [this](https://github.com/apache/cloudstack-kubernetes-provider/pull/69), and hopefully you can be of some help here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
winterhazel commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2951023774 @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 comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
blueorangutan commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2951042914 @winterhazel 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
blueorangutan commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2951229318 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13653 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
apraba05 commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949562495 Yes the compute offering is and I'm still not able to see the ISO either -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
apraba05 commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949789538 @weizhouapache yes I actually did register it as it's present on the UI under Kubernetes ISOs. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
winterhazel commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2949900062 @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 comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
blueorangutan commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2949902657 @winterhazel 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
blueorangutan commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2949926015 Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 13650 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
shwstppr commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949759090 @apraba05 did you register any Kubernetes version ISO under section - Images > Kubernetes ISO? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Support of snapshot copy to primary storage in different zones. [cloudstack]
nvazquez commented on PR #9478: URL: https://github.com/apache/cloudstack/pull/9478#issuecomment-2949565181 @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 comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Support of snapshot copy to primary storage in different zones. [cloudstack]
blueorangutan commented on PR #9478: URL: https://github.com/apache/cloudstack/pull/9478#issuecomment-2949568808 @nvazquez a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
apraba05 commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949623408  -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
apraba05 commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949624081 I have no options prevalent here -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
apraba05 commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949769041 @shwstppr I actually did I have 2 ISOs running but it's still not present on the UI. I have 1.28.4 and 1.27.8 running but it isn't present. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
weizhouapache commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949784339 > [@shwstppr](https://github.com/shwstppr) I actually did I have 2 ISOs running but it's still not present on the UI. I have 1.28.4 and 1.27.8 running but it isn't present. @apraba05 did you register "ISO" or "Kubernetes version" ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
blueorangutan commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2949962659 @winterhazel 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
apraba05 commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949964858 Yes it is ready but I am still unable to see it.  -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
winterhazel commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2949958075 @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 comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
weizhouapache commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949970880 mine is Ready, not Allocated  you may go to ISOs page-> Zones tab -> Status to see what's wrong is SSVM running ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [I] K8s missing version and compute for cluster [cloudstack]
weizhouapache commented on issue #10972: URL: https://github.com/apache/cloudstack/issues/10972#issuecomment-2949959536 > [@weizhouapache](https://github.com/weizhouapache) yes I actually did register it as it's present on the UI under Kubernetes ISOs. What's the ISO state, is it Ready ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Refactor of Allocator classes [cloudstack]
blueorangutan commented on PR #9074: URL: https://github.com/apache/cloudstack/pull/9074#issuecomment-2949980648 Packaging result [SF]: ✖️ el8 ✖️ el9 ✖️ debian ✖️ suse15. SL-JID 13651 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org