From aca63b66e5133e38f2899fb7e79ca00072694150 Mon Sep 17 00:00:00 2001
From: Sean Mooney <work@seanmooney.info>
Date: Wed, 20 Mar 2024 09:44:21 +0000
Subject: [PATCH] Test live migration between hosts with differnet
 cpu_shared_sets

This change modifes the nova-live-migration job to define
[compute]cpu_shared_set to different ranges on the controller
and compute node. in ci we have 8 cpus so the controller is set
to 0-5 and the compute uses 2-7

6 cores with our default allocation ratio should be sufficent
for our current level of concurrancy. This change also adds
the remaining cores as cpu_dedicated_set, they will currently
be unused but are thre if we want to enable pinned cpu testing
in the future.

Change-Id: I0d8463cf584d6ac141a5e1cb492d17996db458e0
---
 .zuul.yaml | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/.zuul.yaml b/.zuul.yaml
index 298cd17d89d2..cac9bb743ca4 100644
--- a/.zuul.yaml
+++ b/.zuul.yaml
@@ -130,12 +130,35 @@
         neutron-trunk: true
         openstack-cli-server: true
       devstack_local_conf:
+        post-config:
+          $NOVA_CPU_CONF:
+            compute:
+              # we want to test live migration of floating instances between hosts with different
+              # cpu_shared_set to ensure the xml is updated properly. This is important when migrating
+              # between hosts with different number of cpu cores even when cpu_shared_set is not defined.
+              # In the ci, we have 8 cpu cores, so we set the cpu_shared_set to 0-5 for the first compute
+              # node and 2-7 for the second compute node.
+              cpu_shared_set: "0-5"
+              cpu_dedicated_set: "6-7"
         test-config:
           $TEMPEST_CONFIG:
             compute-feature-enabled:
               volume_backed_live_migration: true
               block_migration_for_live_migration: true
               block_migrate_cinder_iscsi: true
+    group-vars:
+      subnode:
+        devstack_local_conf:
+          post-config:
+            $NOVA_CPU_CONF:
+              compute:
+                # we want to test live migration of floating instances between hosts with different
+                # cpu_shared_set to ensure the xml is updated properly. This is important when migrating
+                # between hosts with different number of cpu cores even when cpu_shared_set is not defined.
+                # In the ci, we have 8 cpu cores, so we set the cpu_shared_set to 0-5 for the first compute
+                # node and 2-7 for the second compute node.
+                cpu_shared_set: "2-7"
+                cpu_dedicated_set: "0-1"
     post-run: playbooks/nova-live-migration/post-run.yaml
 
 - job: