he: add storage network for vms

To maintain uniformity across all suites using storage VMs, add storage network and NICs.
lveyde · May 25, 2022 · f5815d6 · f5815d6
1 parent 092cad7
commit f5815d6
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 19 deletions.
diff --git a/common/deploy-scripts/setup_storage.sh b/common/deploy-scripts/setup_storage.sh
@@ -52,13 +52,7 @@ set_selinux_on_nfs() {
 }
 
 setup_iscsi() {
-    # this is ugly, assumes that dedicated storage VMs (ost-[suite]-storage) use their primary network as storage network only on HE suite, and VMs with co-located engine have a dedicated storage network on eth1 (like basic-suite-master).
-    echo $SUITE
-    if [[ $(hostname) == *"-storage"* ]] && [[ $(hostname) == *"he-"*  ]]; then
-        NIC=eth0
-    else
-        NIC=eth1
-    fi
+    NIC=eth1
     IP=$(/sbin/ip -o addr show dev $NIC scope global | tac | awk '{split($4,a,"."); print a[1] "." a[2] "." a[3] "." a[4]}'| awk -F/ '{print $1; exit}')
 
     pvcreate --zero n /dev/${ISCSI_DEV}

diff --git a/common/deploy-scripts/setup_storage_he_extra.sh b/common/deploy-scripts/setup_storage_he_extra.sh
@@ -34,13 +34,7 @@ setup_he_lun() {
 
     # The rest of this function is copied from setup_storage.sh. Keep in sync.
     # TODO: Unite or rewrite some day
-
-    # this is ugly, assumes that dedicated storage VMs (ost-[suite]-storage) use their primary network as storage network, and VMs with co-located engine have a dedicated storage network on eth1 (like basic-suite-master).
-    if [[ $(hostname) == *"-storage"* ]]; then
-        NIC=eth0
-    else
-        NIC=eth1
-    fi
+    NIC=eth1
     IP=$(/sbin/ip -o addr show dev $NIC scope global | tac | awk '{split($4,a,"."); print a[1] "." a[2] "." a[3] "." a[4]}'| awk -F/ '{print $1; exit}')
 
     iscsiadm -m discovery -t sendtargets -p $IP

diff --git a/common/init-configs/1_hosted_engine_2_hosts.json b/common/init-configs/1_hosted_engine_2_hosts.json
@@ -8,6 +8,14 @@
       "host-0",
       "host-1"
     ]
+  },
+    "storage": {
+    "template": "common/libvirt-templates/net_template",
+    "nics": [
+      "host-0-storage",
+      "host-1-storage",
+      "storage-storage"
+    ]
   }
 },
 "vms" : {
@@ -19,7 +27,8 @@
       "common/deploy-scripts/setup_storage_he_extra.sh"
     ],
     "nics": {
-      "storage": { "template": "common/libvirt-templates/nic_template" }
+      "storage": { "template": "common/libvirt-templates/nic_template" },
+      "storage-storage": { "template": "common/libvirt-templates/nic_template" }
     },
     "root_disk_var": "OST_IMAGES_BASE",
     "disks": {
@@ -47,7 +56,8 @@
       "common/deploy-scripts/setup_host.sh"
     ],
     "nics": {
-      "host-0": { "template": "common/libvirt-templates/nic_template" }
+      "host-0": { "template": "common/libvirt-templates/nic_template" },
+      "host-0-storage": { "template": "common/libvirt-templates/nic_template" }
     },
     "root_disk_var": "OST_IMAGES_HE_INSTALLED",
     "disks": {}
@@ -59,7 +69,8 @@
       "common/deploy-scripts/setup_host.sh"
     ],
     "nics": {
-      "host-1": { "template": "common/libvirt-templates/nic_template" }
+      "host-1": { "template": "common/libvirt-templates/nic_template" },
+      "host-1-storage": { "template": "common/libvirt-templates/nic_template" }
     },
     "root_disk_var": "OST_IMAGES_NODE",
     "disks": {}

diff --git a/he-basic-suite-master/test-scenarios/conftest.py b/he-basic-suite-master/test-scenarios/conftest.py
@@ -37,9 +37,10 @@
 
 @pytest.fixture(scope="session")
 def sd_iscsi_host_ip(
-    storage_management_ips,
+    storage_ips_for_network,
+    storage_network_name,
 ):  # pylint: disable=function-redefined
-    return storage_management_ips[0]
+    return storage_ips_for_network(storage_network_name)[0]
 
 
 @pytest.fixture(scope="session")