You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by "nvazquez (via GitHub)" <gi...@apache.org> on 2025/09/13 19:59:21 UTC

[PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

nvazquez opened a new pull request, #11633:
URL: https://github.com/apache/cloudstack/pull/11633

   ### Description
   
   This PR addresses 2 bugs on the SolidFire 1:1 plugin using zone-wide storage:
   - When trying to migrate VMs on different clusters within the same zone, CloudStack showed storage migration was required. This PR fixes that, not involving storage migration for inter-cluster migration within the same zone (#11297)
   - Inter-cluster VM migration fails when the VM LUN was not mounted on the destination host. This PR addresses that by mounting the LUN on the destination host and unmounting it from the origin host after the migration succeeds.
   
   Fixes: #11297 
   
   <!--- ******************************************************************************* -->
   <!--- NOTE: AUTOMATION USES THE DESCRIPTIONS TO SET LABELS AND PRODUCE DOCUMENTATION. -->
   <!--- PLEASE PUT AN 'X' in only **ONE** box -->
   <!--- ******************************************************************************* -->
   
   ### 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
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [ ] Minor
   
   #### Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [ ] Minor
   - [ ] Trivial
   
   ### Screenshots (if appropriate):
   
   ### How Has This Been Tested?
   
   <!-- Please describe in detail how you tested your changes. -->
   <!-- Include details of your testing environment, and the tests you ran to -->
   
   #### How did you try to break this feature and the system with this change?
   
   <!-- see how your change affects other areas of the code, etc. -->
   
   <!-- Please read the [CONTRIBUTING](https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md) document -->
   


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3803062384

   @nvazquez can you address the outstanding comments and re-target this to 4.20 if relevant. thanks.


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3292280960

   Packaging result [SF]: ✖️ el8 ✖️ el9  ✔️ debian ✖️ suse15. SL-JID 15011


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3291740664

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3799300114

   Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 16529


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3799092506

   @borisstoyanov 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3293374316

   @blueorangutan test ol8 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 to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3292316466

   @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 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3288787883

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3289694883

   <b>[SF] Trillian test result (tid-14325)</b>
   Environment: xcpng82 (x2), zone: Advanced Networking with Mgmt server ol9
   Total time taken: 68278 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14325-xcpng82.zip
   Smoke tests completed. 132 look OK, 1 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_primary_storage_iscsi | `Error` | 0.84 | test_primary_storage.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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez closed pull request #11633: [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools
URL: https://github.com/apache/cloudstack/pull/11633


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3646958467

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3294223525

   <b>[SF] Trillian Build Failed (tid-14349)<b/>


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3289483176

   <b>[SF] Trillian test result (tid-14323)</b>
   Environment: kvm-ubuntu22 (x2), zone: Advanced Networking with Mgmt server u22
   Total time taken: 49683 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14323-kvm-ubuntu22.zip
   Smoke tests completed. 133 look OK, 0 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on code in PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#discussion_r2348056183


##########
engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java:
##########
@@ -1560,6 +1561,17 @@ private Map<String, String> getDetails(VolumeInfo volumeInfo, DataStore dataStor
             details.put(DiskTO.CHAP_TARGET_SECRET, chapInfo.getTargetSecret());
         }
 
+        // Zone wide storage Solidfire inter-cluster VM migrations needs the destination host mount the LUN before migrating
+        if (storagePool.isManaged() && storagePool.getScope() != null && ScopeType.ZONE == storagePool.getScope()) {

Review Comment:
   ```suggestion
           if (storagePool.isManaged() && ScopeType.ZONE.equals(storagePool.getScope())) {
   ```a



-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "sureshanaparti (via GitHub)" <gi...@apache.org>.
sureshanaparti commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3292308122

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3306576713

   @weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + vmware-80u3) 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on code in PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#discussion_r2348113235


##########
plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java:
##########
@@ -4645,6 +4655,36 @@ protected Answer execute(PrepareForMigrationCommand cmd) {
         }
     }
 
+    private void prepareDatastoreForZoneWideManagedStorageInterClusterMigration(DiskTO disk, VmwareHypervisorHost hyperHost) throws Exception {

Review Comment:
   this method will be called if storage is not SolidFire, right ?



-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3289468083

   <b>[SF] Trillian test result (tid-14322)</b>
   Environment: kvm-ol8 (x2), zone: Advanced Networking with Mgmt server ol8
   Total time taken: 47460 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14322-kvm-ol8.zip
   Smoke tests completed. 132 look OK, 1 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_secure_vm_migration | `Error` | 135.69 | test_vm_life_cycle.py
   test_01_secure_vm_migration | `Error` | 135.70 | test_vm_life_cycle.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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3654686800

   Hi @DaanHoogland @weizhouapache this PR is not ready yet, marked it as draft for now


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3293178500

   Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15021


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "borisstoyanov (via GitHub)" <gi...@apache.org>.
borisstoyanov commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3799089568

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on code in PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#discussion_r2348504001


##########
plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java:
##########
@@ -4645,6 +4655,36 @@ protected Answer execute(PrepareForMigrationCommand cmd) {
         }
     }
 
+    private void prepareDatastoreForZoneWideManagedStorageInterClusterMigration(DiskTO disk, VmwareHypervisorHost hyperHost) throws Exception {

Review Comment:
   Hi @weizhouapache - yes. It would actually reach non Solidfire storages but will exit fast in those cases as it has a precondition for specific detail keys which are set only in case of Solidfire zone-wide pools after this commit: https://github.com/apache/cloudstack/pull/11633/commits/56241d3b147b6992bcdc195255c72f8559d36a17



-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3288865821

   @nvazquez a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3290772281

   @harikrishna-patnala @sureshanaparti @vishesh92 @shwstppr 
   can you please review ?
   thanks


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3334581442

   
   @blueorangutan test ol8 vmware-80u3
   
   


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3646963024

   @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.


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3288787231

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3289487431

   <b>[SF] Trillian test result (tid-14324)</b>
   Environment: vmware-70u3 (x2), zone: Advanced Networking with Mgmt server ol8
   Total time taken: 49815 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14324-vmware-70u3.zip
   Smoke tests completed. 132 look OK, 1 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_05_ping_in_cpvm_success | `Failure` | 15.66 | test_diagnostics.py
   test_13_retrieve_vr_default_files | `Failure` | 1.18 | test_diagnostics.py
   test_14_retrieve_vr_one_file | `Failure` | 1.15 | test_diagnostics.py
   test_15_retrieve_ssvm_default_files | `Failure` | 1.14 | test_diagnostics.py
   test_16_retrieve_ssvm_single_file | `Failure` | 1.16 | test_diagnostics.py
   test_17_retrieve_cpvm_default_files | `Failure` | 1.16 | test_diagnostics.py
   test_18_retrieve_cpvm_single_file | `Failure` | 1.16 | test_diagnostics.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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3646962684

   @weizhouapache @nvazquez , anything we can do to move this forwards? (do we still plan to merge this on 4.19?)


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3301852885

   <b>[SF] Trillian test result (tid-14359)</b>
   Environment: vmware-70u3 (x2), zone: Advanced Networking with Mgmt server ol8
   Total time taken: 65659 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14359-vmware-70u3.zip
   Smoke tests completed. 131 look OK, 2 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_restore_vm | `Error` | 308.18 | test_restore_vm.py
   test_02_restore_vm_allocated_root | `Error` | 1201.70 | test_restore_vm.py
   ContextSuite context=TestRestoreVM>:teardown | `Error` | 1210.24 | test_restore_vm.py
   test_01_deploy_vm_on_specific_host | `Error` | 3605.26 | test_vm_deployment_planner.py
   test_02_deploy_vm_on_specific_cluster | `Error` | 4.51 | test_vm_deployment_planner.py
   test_03_deploy_vm_on_specific_pod | `Error` | 4.58 | test_vm_deployment_planner.py
   test_04_deploy_vm_on_host_override_pod_and_cluster | `Error` | 4.64 | test_vm_deployment_planner.py
   test_05_deploy_vm_on_cluster_override_pod | `Error` | 15.84 | test_vm_deployment_planner.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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3291745137

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3293376134

   @weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + vmware-70u3) 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3298725873

   @blueorangutan test ol8 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 to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3305239930

   Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15068


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on code in PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#discussion_r2348500257


##########
engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java:
##########
@@ -1560,6 +1561,17 @@ private Map<String, String> getDetails(VolumeInfo volumeInfo, DataStore dataStor
             details.put(DiskTO.CHAP_TARGET_SECRET, chapInfo.getTargetSecret());
         }
 
+        // Zone wide storage Solidfire inter-cluster VM migrations needs the destination host mount the LUN before migrating
+        if (storagePool.isManaged() && storagePool.getScope() != null && ScopeType.ZONE == storagePool.getScope()) {

Review Comment:
   Thanks, fixed



-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3288790590

   ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/11633?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   :white_check_mark: All modified and coverable lines are covered by tests.
   :white_check_mark: Project coverage is 4.28%. Comparing base ([`2e113e5`](https://app.codecov.io/gh/apache/cloudstack/commit/2e113e5ed704ea6d24295ff765c30f66a1d0601b?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`8b0b163`](https://app.codecov.io/gh/apache/cloudstack/commit/8b0b163a261817063020179ba3ea4950d6cb7c54?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 (2e113e5) and HEAD (8b0b163). Click for more details.
   > 
   > <details><summary>HEAD has 1 upload less than BASE</summary>
   >
   >| Flag | BASE (2e113e5) | HEAD (8b0b163) |
   >|------|------|------|
   >|unittests|1|0|
   ></details>
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@              Coverage Diff              @@
   ##               4.19   #11633       +/-   ##
   =============================================
   - Coverage     15.18%    4.28%   -10.90%     
   =============================================
     Files          5415      372     -5043     
     Lines        476082    29745   -446337     
     Branches      58129     5229    -52900     
   =============================================
   - Hits          72288     1275    -71013     
   + Misses       395702    28324   -367378     
   + Partials       8092      146     -7946     
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/11633/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/11633/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `4.28% <ø> (ø)` | |
   | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/11633/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.
   </details>
   
   [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/cloudstack/pull/11633?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).
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :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.
   </details>


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3334590201

   @weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + vmware-80u3) 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3654371891

   @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + vmware-80u3) 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "DaanHoogland (via GitHub)" <gi...@apache.org>.
DaanHoogland commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3654366179

   @blueorangutan test ol8 vmware-80u3


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#discussion_r2727273111


##########
engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java:
##########
@@ -2893,6 +2894,20 @@ protected void migrate(final VMInstanceVO vm, final long srcHostId, final Deploy
         }
     }
 
+    private void postMigrationRelease(VirtualMachineProfile profile, long vmId, long srcHostId) {
+        List<String> poolsToReleaseOnOrigin = volumeMgr.postMigrationReleaseDatastoresOnOriginHost(profile, vmId);
+        if (CollectionUtils.isEmpty(poolsToReleaseOnOrigin)) {
+            return;
+        }
+        // Unmount the datastores from this host only (useful for SolidFire 1:1 plugin zone-wide storage migrations)
+        UnmountDatastoresCommand cmd = new UnmountDatastoresCommand(poolsToReleaseOnOrigin);
+        try {
+            _agentMgr.send(srcHostId, cmd);
+        } catch (AgentUnavailableException | OperationTimedoutException e) {
+            throw new RuntimeException(e);

Review Comment:
   Throwing a RuntimeException wrapping AgentUnavailableException or OperationTimedoutException may cause the entire migration to fail unexpectedly. Consider logging the error and handling it gracefully instead of propagating it as a RuntimeException, since this is a cleanup operation that happens after the VM has successfully migrated. The migration has already succeeded at this point (line 2887), so failing here could leave the system in an inconsistent state.
   ```suggestion
               s_logger.warn("Failed to unmount datastores on origin host " + srcHostId +
                       " after migration of VM id=" + vmId + ". Datastores to unmount: " + poolsToReleaseOnOrigin, e);
   ```



##########
plugins/storage/volume/solidfire/src/main/java/org/apache/cloudstack/storage/datastore/driver/SolidFirePrimaryDataStoreDriver.java:
##########
@@ -118,6 +119,7 @@ public class SolidFirePrimaryDataStoreDriver implements PrimaryDataStoreDriver {
     @Inject private VolumeDao volumeDao;
     @Inject private VolumeDetailsDao volumeDetailsDao;
     @Inject private VolumeDataFactory volumeFactory;
+    @Inject private AgentManager agentManager;

Review Comment:
   The AgentManager is injected but never used in this class. This appears to be an unnecessary import and injection. Consider removing this if it's not needed for the functionality being implemented.



##########
plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java:
##########
@@ -4645,6 +4655,29 @@ protected Answer execute(PrepareForMigrationCommand cmd) {
         }
     }
 
+    private void prepareDatastoreForZoneWideManagedStorageInterClusterMigration(DiskTO disk, VmwareHypervisorHost hyperHost) throws Exception {
+        Map<String, String> details = disk.getDetails();
+        if (MapUtils.isEmpty(details) || !details.containsKey(DiskTO.SCOPE) ||
+                !details.containsKey(DiskTO.MANAGED) && !details.containsKey(DiskTO.INTER_CLUSTER_MIGRATION)) {

Review Comment:
   The boolean logic in this condition is incorrect. The expression `!details.containsKey(DiskTO.MANAGED) && !details.containsKey(DiskTO.INTER_CLUSTER_MIGRATION)` should be `(!details.containsKey(DiskTO.MANAGED) || !details.containsKey(DiskTO.INTER_CLUSTER_MIGRATION))` with OR instead of AND. 
   
   The current logic means the method returns only when BOTH MANAGED and INTER_CLUSTER_MIGRATION are missing, but the subsequent checks on lines 4666-4668 explicitly check for the presence of these keys. This means if either key is present (but not both), the method won't return early but will later evaluate false in the checks below, making the early return ineffective.
   
   The intended logic should be: return early if SCOPE is missing OR if either MANAGED or INTER_CLUSTER_MIGRATION is missing.
   ```suggestion
                   (!details.containsKey(DiskTO.MANAGED) || !details.containsKey(DiskTO.INTER_CLUSTER_MIGRATION))) {
   ```



##########
plugins/storage/volume/solidfire/src/main/java/org/apache/cloudstack/storage/datastore/driver/SolidFirePrimaryDataStoreDriver.java:
##########
@@ -25,6 +25,7 @@
 
 import javax.inject.Inject;
 
+import com.cloud.agent.AgentManager;

Review Comment:
   The import for AgentManager is unused. This import was added but the AgentManager field is never actually used in the class. Consider removing this import along with the unused field injection.
   ```suggestion
   
   ```



##########
plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java:
##########
@@ -4645,6 +4655,29 @@ protected Answer execute(PrepareForMigrationCommand cmd) {
         }
     }
 
+    private void prepareDatastoreForZoneWideManagedStorageInterClusterMigration(DiskTO disk, VmwareHypervisorHost hyperHost) throws Exception {
+        Map<String, String> details = disk.getDetails();
+        if (MapUtils.isEmpty(details) || !details.containsKey(DiskTO.SCOPE) ||
+                !details.containsKey(DiskTO.MANAGED) && !details.containsKey(DiskTO.INTER_CLUSTER_MIGRATION)) {
+            return;
+        }
+
+        VmwareContext context = hyperHost.getContext();
+        boolean isManaged = details.containsKey(DiskTO.MANAGED) && BooleanUtils.toBoolean(details.get(DiskTO.MANAGED));
+        boolean isZoneWideStorage = details.containsKey(DiskTO.SCOPE) && details.get(DiskTO.SCOPE).equalsIgnoreCase(ScopeType.ZONE.name());
+        boolean isInterClusterMigration = details.containsKey(DiskTO.INTER_CLUSTER_MIGRATION) && BooleanUtils.toBoolean(details.get(DiskTO.INTER_CLUSTER_MIGRATION));
+
+        if (isManaged && isZoneWideStorage && isInterClusterMigration) {
+            s_logger.debug(String.format("Preparing storage on destination cluster for host %s", hyperHost.getHyperHostName()));
+            String iScsiName = details.get(DiskTO.IQN); // details should not be null for managed storage (it may or may not be null for non-managed storage)
+            String datastoreName = VmwareResource.getDatastoreName(iScsiName);
+            s_logger.debug(String.format("Ensuring datastore %s is mounted on destination cluster", datastoreName));
+            _storageProcessor.prepareManagedDatastore(context, hyperHost, datastoreName,
+                    details.get(DiskTO.IQN), details.get(DiskTO.STORAGE_HOST),
+                    Integer.parseInt(details.get(DiskTO.STORAGE_PORT)));

Review Comment:
   There's a potential NumberFormatException if details.get(DiskTO.STORAGE_PORT) returns null or a non-numeric value. While the comment on line 4672 suggests details should not be null for managed storage, there's no explicit validation that STORAGE_PORT is present or valid before parsing. Consider adding a null check and validation before calling Integer.parseInt().
   ```suggestion
               String storagePortString = details.get(DiskTO.STORAGE_PORT);
               if (StringUtils.isBlank(storagePortString) || !NumberUtils.isDigits(storagePortString)) {
                   s_logger.warn(String.format("Skipping preparation of managed datastore %s due to invalid storage port value: %s",
                           datastoreName, storagePortString));
                   return;
               }
               int storagePort = Integer.parseInt(storagePortString);
               _storageProcessor.prepareManagedDatastore(context, hyperHost, datastoreName,
                       details.get(DiskTO.IQN), details.get(DiskTO.STORAGE_HOST), storagePort);
   ```



-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3803167982

   @sureshanaparti closing the PR for now, will reopen when 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on code in PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#discussion_r2348504001


##########
plugins/hypervisors/vmware/src/main/java/com/cloud/hypervisor/vmware/resource/VmwareResource.java:
##########
@@ -4645,6 +4655,36 @@ protected Answer execute(PrepareForMigrationCommand cmd) {
         }
     }
 
+    private void prepareDatastoreForZoneWideManagedStorageInterClusterMigration(DiskTO disk, VmwareHypervisorHost hyperHost) throws Exception {

Review Comment:
   Hi @weizhouapache - yes. However it has a precondition for specific detail keys which are set only in case of Solidfire zone-wide pools after this commit: https://github.com/apache/cloudstack/pull/11633/commits/56241d3b147b6992bcdc195255c72f8559d36a17



-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3349127503

   <b>[SF] Trillian test result (tid-14440)</b>
   Environment: vmware-80u3 (x2), zone: Advanced Networking with Mgmt server ol8
   Total time taken: 365815 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14440-vmware-80u3.zip
   Smoke tests completed. 97 look OK, 36 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_CRUD_operations_guest_OS | `Error` | 0.06 | test_guest_os.py
   test_CRUD_operations_guest_OS_mapping | `Error` | 0.05 | test_guest_os.py
   test_guest_OS_mapping_check_with_hypervisor | `Error` | 0.05 | test_guest_os.py
   ContextSuite context=TestClusterDRS>:setup | `Error` | 0.00 | test_cluster_drs.py
   test_01_invalid_upgrade_kubernetes_cluster | `Failure` | 3681.18 | test_kubernetes_clusters.py
   test_02_upgrade_kubernetes_cluster | `Failure` | 3677.34 | test_kubernetes_clusters.py
   test_03_deploy_and_scale_kubernetes_cluster | `Failure` | 3687.89 | test_kubernetes_clusters.py
   test_04_autoscale_kubernetes_cluster | `Failure` | 3678.39 | test_kubernetes_clusters.py
   test_05_basic_lifecycle_kubernetes_cluster | `Failure` | 3699.16 | test_kubernetes_clusters.py
   test_06_delete_kubernetes_cluster | `Failure` | 3669.32 | test_kubernetes_clusters.py
   test_08_upgrade_kubernetes_ha_cluster | `Failure` | 3784.19 | test_kubernetes_clusters.py
   test_10_vpc_tier_kubernetes_cluster | `Failure` | 4789.17 | test_kubernetes_clusters.py
   ContextSuite context=TestListVolumes>:setup | `Error` | 0.00 | test_list_volumes.py
   test_list_vms_metrics_admin | `Error` | 3624.49 | test_metrics_api.py
   test_list_vms_metrics_history | `Error` | 6.02 | test_metrics_api.py
   test_list_volumes_metrics_history | `Error` | 3627.88 | test_metrics_api.py
   test_01_add_primary_storage_disabled_host | `Error` | 0.28 | test_primary_storage.py
   ContextSuite context=TestStorageTags>:setup | `Error` | 119.08 | test_primary_storage.py
   test_01_primary_storage_scope_change | `Error` | 0.17 | test_primary_storage_scope.py
   test_01_vpc_privategw_acl | `Failure` | 1021.71 | test_privategw_acl.py
   test_02_vpc_privategw_static_routes | `Failure` | 1021.48 | test_privategw_acl.py
   test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 1002.30 | test_privategw_acl.py
   test_04_rvpc_privategw_static_routes | `Failure` | 1023.10 | test_privategw_acl.py
   test_09_project_suspend | `Error` | 1149.58 | test_projects.py
   test_deploy_vm_with_registered_userdata | `Error` | 1121.51 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata | `Error` | 1121.59 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata_with_override_policy_allow | `Error` | 1145.57 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata_with_override_policy_allow | `Error` | 1145.65 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata_with_override_policy_append | `Error` | 1154.11 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata_with_override_policy_append | `Error` | 1154.19 | test_register_userdata.py
   ContextSuite context=TestResetVmOnReboot>:setup | `Error` | 0.00 | test_reset_vm_on_reboot.py
   test_01_so_removal_resource_update | `Error` | 1132.23 | test_resource_accounting.py
   test_01_restore_vm | `Error` | 135.94 | test_restore_vm.py
   test_02_restore_vm_allocated_root | `Error` | 69.84 | test_restore_vm.py
   ContextSuite context=TestRouterDHCPHosts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
   ContextSuite context=TestRouterDHCPOpts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
   ContextSuite context=TestRouterDns>:setup | `Error` | 0.00 | test_router_dns.py
   ContextSuite context=TestRouterDnsService>:setup | `Error` | 0.00 | test_router_dnsservice.py
   test_02_routervm_iptables_policies | `Error` | 1217.56 | test_routers_iptables_default_policy.py
   test_01_single_VPC_iptables_policies | `Error` | 1005.99 | test_routers_iptables_default_policy.py
   test_01_isolate_network_FW_PF_default_routes_egress_true | `Error` | 1188.23 | test_routers_network_ops.py
   test_02_isolate_network_FW_PF_default_routes_egress_false | `Failure` | 1281.23 | test_routers_network_ops.py
   test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | `Error` | 1028.58 | test_routers_network_ops.py
   test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | `Error` | 1007.16 | test_routers_network_ops.py
   test_03_RVR_Network_check_router_state | `Error` | 1046.84 | test_routers_network_ops.py
   test_03_restart_network_cleanup | `Error` | 1043.67 | test_routers.py
   test_04_restart_network_wo_cleanup | `Failure` | 148.14 | test_routers.py
   ContextSuite context=TestServiceOfferings>:setup | `Error` | 1157.97 | test_service_offerings.py
   test_03_create_vpc_with_specified_source_nat_ip_address | `Error` | 1006.10 | test_set_sourcenat.py
   test_04_change_source_nat_ip_address_for_vpc | `Error` | 1019.38 | test_set_sourcenat.py
   ContextSuite context=TestSnapshotRootDisk>:setup | `Error` | 0.00 | test_snapshots.py
   ContextSuite context=TestSnapshotStandaloneBackup>:setup | `Error` | 0.00 | test_snapshots.py
   test_02_list_cpvm_vm | `Failure` | 0.05 | test_ssvm.py
   test_04_cpvm_internals | `Failure` | 0.06 | test_ssvm.py
   test_06_stop_cpvm | `Failure` | 0.07 | test_ssvm.py
   test_11_destroy_ssvm | `Failure` | 924.96 | test_ssvm.py
   test_12_destroy_cpvm | `Failure` | 929.82 | test_ssvm.py
   test_01_import_storage_policies | `Error` | 67.16 | test_storage_policy.py
   ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 67.21 | test_storage_policy.py
   ContextSuite context=TestTemplates>:setup | `Error` | 1222.69 | test_templates.py
   ContextSuite context=TestVmUsage>:setup | `Error` | 1265.13 | test_usage.py
   test_01_volume_usage | `Error` | 104.75 | test_usage.py
   ContextSuite context=TestVpnUsage>:setup | `Error` | 1328.10 | test_usage.py
   test_01_scale_up_verify | `Failure` | 576.73 | test_vm_autoscaling.py
   test_02_update_vmprofile_and_vmgroup | `Error` | 2.31 | test_vm_autoscaling.py
   test_03_scale_down_verify | `Error` | 1.13 | test_vm_autoscaling.py
   test_04_stop_remove_vm_in_vmgroup | `Error` | 3.41 | test_vm_autoscaling.py
   test_06_autoscaling_vmgroup_on_project_network | `Failure` | 587.75 | test_vm_autoscaling.py
   test_06_autoscaling_vmgroup_on_project_network | `Error` | 587.76 | test_vm_autoscaling.py
   test_07_autoscaling_vmgroup_on_vpc_network | `Error` | 1006.36 | test_vm_autoscaling.py
   ContextSuite context=TestVmAutoScaling>:teardown | `Error` | 1016.61 | test_vm_autoscaling.py
   test_01_deploy_vm_on_specific_host | `Error` | 0.12 | test_vm_deployment_planner.py
   test_04_deploy_vm_on_host_override_pod_and_cluster | `Error` | 0.17 | test_vm_deployment_planner.py
   test_01_offline_migrate_VM_and_root_volume | `Error` | 1203.45 | test_vm_life_cycle.py
   test_02_offline_migrate_VM_with_two_data_disks | `Error` | 111.74 | test_vm_life_cycle.py
   test_03_live_migrate_VM_with_two_data_disks | `Error` | 126.36 | test_vm_life_cycle.py
   test_01_vapps_vm_cycle | `Error` | 42.50 | test_vm_life_cycle.py
   test_08_migrate_vm | `Error` | 0.07 | test_vm_life_cycle.py
   test_01_unmanage_vm_cycle | `Error` | 1007.17 | test_vm_lifecycle_unmanage_import.py
   ContextSuite context=TestUnmanageVM>:teardown | `Error` | 1007.21 | test_vm_lifecycle_unmanage_import.py
   test_04_deploy_vnf_appliance | `Error` | 1376.66 | test_vnf_templates.py
   test_04_deploy_vnf_appliance | `Error` | 1376.78 | test_vnf_templates.py
   test_05_delete_vnf_template | `Error` | 1.14 | test_vnf_templates.py
   ContextSuite context=TestVnfTemplates>:teardown | `Error` | 2.33 | test_vnf_templates.py
   ContextSuite context=TestVolumes>:setup | `Error` | 1254.13 | test_volumes.py
   test_01_verify_ipv6_vpc | `Error` | 1020.50 | test_vpc_ipv6.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Error` | 1035.43 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | `Error` | 1012.37 | test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | `Error` | 31.18 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | `Error` | 1011.89 | test_vpc_redundant.py
   test_05_rvpc_multi_tiers | `Error` | 1026.15 | test_vpc_redundant.py
   test_01_VPC_nics_after_destroy | `Error` | 1009.81 | test_vpc_router_nics.py
   test_02_VPC_default_routes | `Error` | 1009.32 | test_vpc_router_nics.py
   test_01_redundant_vpc_site2site_vpn | `Failure` | 1011.51 | test_vpc_vpn.py
   test_01_vpc_site2site_vpn_multiple_options | `Failure` | 1005.35 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | `Failure` | 1005.60 | test_vpc_vpn.py
   test_01_vpc_site2site_vpn | `Failure` | 1011.71 | test_vpc_vpn.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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3305112456

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3305115072

   @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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3322590025

   @weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + vmware-80u3) 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3288848617

   Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 14992


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3319095548

   <b>[SF] Trillian test result (tid-14378)</b>
   Environment: vmware-80u3 (x2), zone: Advanced Networking with Mgmt server ol8
   Total time taken: 355599 seconds
   Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr11633-t14378-vmware-80u3.zip
   Smoke tests completed. 100 look OK, 33 have errors, 0 did not run
   Only failed and skipped tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_CRUD_operations_guest_OS | `Error` | 0.06 | test_guest_os.py
   test_CRUD_operations_guest_OS_mapping | `Error` | 0.05 | test_guest_os.py
   test_guest_OS_mapping_check_with_hypervisor | `Error` | 0.04 | test_guest_os.py
   ContextSuite context=TestClusterDRS>:setup | `Error` | 0.00 | test_cluster_drs.py
   test_01_invalid_upgrade_kubernetes_cluster | `Failure` | 3682.44 | test_kubernetes_clusters.py
   test_02_upgrade_kubernetes_cluster | `Failure` | 3674.05 | test_kubernetes_clusters.py
   test_03_deploy_and_scale_kubernetes_cluster | `Failure` | 3679.81 | test_kubernetes_clusters.py
   test_04_autoscale_kubernetes_cluster | `Failure` | 3695.88 | test_kubernetes_clusters.py
   test_05_basic_lifecycle_kubernetes_cluster | `Failure` | 3683.48 | test_kubernetes_clusters.py
   test_06_delete_kubernetes_cluster | `Failure` | 3694.23 | test_kubernetes_clusters.py
   test_08_upgrade_kubernetes_ha_cluster | `Failure` | 3754.68 | test_kubernetes_clusters.py
   test_10_vpc_tier_kubernetes_cluster | `Failure` | 4806.05 | test_kubernetes_clusters.py
   test_01_add_primary_storage_disabled_host | `Error` | 0.28 | test_primary_storage.py
   test_03_migration_options_storage_tags | `Error` | 0.02 | test_primary_storage.py
   test_01_vpc_privategw_acl | `Failure` | 998.66 | test_privategw_acl.py
   test_02_vpc_privategw_static_routes | `Failure` | 999.95 | test_privategw_acl.py
   test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 1016.26 | test_privategw_acl.py
   test_04_rvpc_privategw_static_routes | `Failure` | 1000.70 | test_privategw_acl.py
   test_09_project_suspend | `Error` | 1136.27 | test_projects.py
   test_10_project_activation | `Error` | 56.17 | test_projects.py
   test_deploy_vm_with_registered_userdata | `Error` | 1112.43 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata | `Error` | 1112.51 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata_with_override_policy_allow | `Error` | 1100.78 | test_register_userdata.py
   test_deploy_vm_with_registered_userdata_with_override_policy_allow | `Error` | 1100.85 | test_register_userdata.py
   ContextSuite context=TestResetVmOnReboot>:setup | `Error` | 0.00 | test_reset_vm_on_reboot.py
   test_01_so_removal_resource_update | `Error` | 1127.42 | test_resource_accounting.py
   test_01_restore_vm | `Error` | 1139.72 | test_restore_vm.py
   test_02_restore_vm_allocated_root | `Error` | 46.19 | test_restore_vm.py
   ContextSuite context=TestRestoreVM>:teardown | `Error` | 57.76 | test_restore_vm.py
   ContextSuite context=TestRouterDHCPHosts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
   ContextSuite context=TestRouterDHCPOpts>:setup | `Error` | 0.00 | test_router_dhcphosts.py
   ContextSuite context=TestRouterDns>:setup | `Error` | 0.00 | test_router_dns.py
   ContextSuite context=TestRouterDnsService>:setup | `Error` | 0.00 | test_router_dnsservice.py
   test_02_routervm_iptables_policies | `Error` | 1141.47 | test_routers_iptables_default_policy.py
   test_01_single_VPC_iptables_policies | `Error` | 997.57 | test_routers_iptables_default_policy.py
   test_01_isolate_network_FW_PF_default_routes_egress_true | `Error` | 1158.43 | test_routers_network_ops.py
   test_02_isolate_network_FW_PF_default_routes_egress_false | `Failure` | 1163.49 | test_routers_network_ops.py
   test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | `Error` | 1013.41 | test_routers_network_ops.py
   test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | `Error` | 1014.09 | test_routers_network_ops.py
   test_03_RVR_Network_check_router_state | `Error` | 1001.92 | test_routers_network_ops.py
   ContextSuite context=TestRouterServices>:setup | `Error` | 0.00 | test_routers.py
   test_05_scale_vm_dont_allow_disk_offering_change | `Error` | 3671.72 | test_scale_vm.py
   ContextSuite context=TestServiceOfferings>:setup | `Error` | 1263.34 | test_service_offerings.py
   test_03_create_vpc_with_specified_source_nat_ip_address | `Error` | 1031.79 | test_set_sourcenat.py
   test_04_change_source_nat_ip_address_for_vpc | `Error` | 999.73 | test_set_sourcenat.py
   ContextSuite context=TestSnapshotRootDisk>:setup | `Error` | 0.00 | test_snapshots.py
   ContextSuite context=TestSnapshotStandaloneBackup>:setup | `Error` | 0.00 | test_snapshots.py
   test_02_list_cpvm_vm | `Failure` | 0.07 | test_ssvm.py
   test_04_cpvm_internals | `Failure` | 0.05 | test_ssvm.py
   test_06_stop_cpvm | `Failure` | 0.05 | test_ssvm.py
   test_11_destroy_ssvm | `Failure` | 927.87 | test_ssvm.py
   test_12_destroy_cpvm | `Failure` | 931.70 | test_ssvm.py
   test_01_import_storage_policies | `Error` | 80.01 | test_storage_policy.py
   ContextSuite context=TestVMWareStoragePolicies>:teardown | `Error` | 80.05 | test_storage_policy.py
   ContextSuite context=TestNatRuleUsage>:setup | `Error` | 1340.22 | test_usage.py
   ContextSuite context=TestVolumeUsage>:setup | `Error` | 1280.41 | test_usage.py
   test_01_scale_up_verify | `Failure` | 576.69 | test_vm_autoscaling.py
   test_02_update_vmprofile_and_vmgroup | `Error` | 2.27 | test_vm_autoscaling.py
   test_03_scale_down_verify | `Error` | 1.09 | test_vm_autoscaling.py
   test_04_stop_remove_vm_in_vmgroup | `Error` | 3.35 | test_vm_autoscaling.py
   test_06_autoscaling_vmgroup_on_project_network | `Failure` | 587.40 | test_vm_autoscaling.py
   test_06_autoscaling_vmgroup_on_project_network | `Error` | 587.40 | test_vm_autoscaling.py
   test_07_autoscaling_vmgroup_on_vpc_network | `Error` | 1046.69 | test_vm_autoscaling.py
   ContextSuite context=TestVmAutoScaling>:teardown | `Error` | 1061.03 | test_vm_autoscaling.py
   test_01_deploy_vm_on_specific_host | `Error` | 0.12 | test_vm_deployment_planner.py
   test_04_deploy_vm_on_host_override_pod_and_cluster | `Error` | 0.14 | test_vm_deployment_planner.py
   test_01_offline_migrate_VM_and_root_volume | `Error` | 1323.96 | test_vm_life_cycle.py
   test_02_offline_migrate_VM_with_two_data_disks | `Error` | 136.30 | test_vm_life_cycle.py
   test_03_live_migrate_VM_with_two_data_disks | `Error` | 136.27 | test_vm_life_cycle.py
   test_01_vapps_vm_cycle | `Error` | 59.70 | test_vm_life_cycle.py
   ContextSuite context=TestVMLifeCycle>:setup | `Error` | 1149.08 | test_vm_life_cycle.py
   test_01_unmanage_vm_cycle | `Error` | 1030.90 | test_vm_lifecycle_unmanage_import.py
   ContextSuite context=TestUnmanageVM>:teardown | `Error` | 1030.95 | test_vm_lifecycle_unmanage_import.py
   test_04_deploy_vnf_appliance | `Error` | 1340.70 | test_vnf_templates.py
   test_04_deploy_vnf_appliance | `Error` | 1340.78 | test_vnf_templates.py
   test_05_delete_vnf_template | `Error` | 1.12 | test_vnf_templates.py
   ContextSuite context=TestVnfTemplates>:teardown | `Error` | 2.27 | test_vnf_templates.py
   ContextSuite context=TestVolumes>:setup | `Error` | 1276.80 | test_volumes.py
   test_01_verify_ipv6_vpc | `Error` | 1004.37 | test_vpc_ipv6.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Error` | 1049.37 | test_vpc_redundant.py
   test_02_redundant_VPC_default_routes | `Error` | 1051.41 | test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | `Error` | 1036.83 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | `Error` | 1051.60 | test_vpc_redundant.py
   test_05_rvpc_multi_tiers | `Error` | 1037.07 | test_vpc_redundant.py
   test_01_VPC_nics_after_destroy | `Error` | 1037.17 | test_vpc_router_nics.py
   test_02_VPC_default_routes | `Error` | 1036.58 | test_vpc_router_nics.py
   test_01_redundant_vpc_site2site_vpn | `Failure` | 1038.33 | test_vpc_vpn.py
   test_01_vpc_site2site_vpn_multiple_options | `Failure` | 1034.05 | test_vpc_vpn.py
   test_01_vpc_remote_access_vpn | `Failure` | 1032.36 | test_vpc_vpn.py
   test_01_vpc_site2site_vpn | `Failure` | 1028.06 | test_vpc_vpn.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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3322588047

   
   @blueorangutan test ol8 vmware-80u3
   
   


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3654447533

   <b>[SF] Trillian Build Failed (tid-14981)<b/>


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3647370376

   Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 16031


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "nvazquez (via GitHub)" <gi...@apache.org>.
nvazquez commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3288864942

   @blueorangutan test matrix


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "weizhouapache (via GitHub)" <gi...@apache.org>.
weizhouapache commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3306572904

   @blueorangutan test ol8 vmware-80u3


-- 
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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [SolidFire] Fix inter-cluster VM migrations on zone-wide SolidFire pools [cloudstack]

Posted by "blueorangutan (via GitHub)" <gi...@apache.org>.
blueorangutan commented on PR #11633:
URL: https://github.com/apache/cloudstack/pull/11633#issuecomment-3298735141

   @DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + vmware-70u3) 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-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org