add tests 13 and 14 to github CI

Author: sjpb

.github/workflows/ci.yml

@@ -43,6 +43,8 @@ jobs:
           - test10
           - test11
           - test12
+          - test13
+          - test14
 
         exclude:
           - image: 'centos:7'
@@ -61,6 +63,10 @@ jobs:
             scenario: test11
           - image: 'centos:7'
             scenario: test12
+          - image: 'centos:7'
+            scenario: test13
+          - image: 'centos:7'
+            scenario: test14
 
     steps:
       - name: Check out the codebase.

molecule/README.md

@@ -21,6 +21,7 @@ test10 | 1            | N                       | As for #5 but then tries to ad
 test11 | 1            | N                       | As for #5 but then deletes a node (actually changes the partition due to molecule/ansible limitations)
 test12 | 1            | N                       | As for #5 but enabling job completion and testing `sacct -c`
 test13 | 1            | N                       | As for #5 but tests `openhpc_config` variable.
+test14 | 1            |                         | As for #5 but also tests `extra_nodes` via State=DOWN nodes.
 
 # Local Installation & Running
 

molecule/test14/converge.yml

@@ -0,0 +1,29 @@
+---
+- name: Converge
+  hosts: all
+  tasks:
+    - name: "Include ansible-role-openhpc"
+      include_role:
+        name: "{{ lookup('env', 'MOLECULE_PROJECT_DIRECTORY') | basename }}"
+      vars:
+        openhpc_enable:
+          control: "{{ inventory_hostname in groups['testohpc_login'] }}"
+          batch: "{{ inventory_hostname in groups['testohpc_compute'] }}"
+          runtime: true
+        openhpc_slurm_control_host: "{{ groups['testohpc_login'] | first }}"
+        openhpc_slurm_partitions:
+          - name: "compute"
+            extra_nodes:
+              # Need to specify IPs as even with State=DOWN, if slurmctld can't lookup a host it just excludes it from the config entirely
+              # Can't add to /etc/hosts via ansible due to Docker limitations on modifying /etc/hosts
+              - NodeName: fake-x,fake-y
+                NodeAddr: 0.42.42.0,0.42.42.1
+                State: DOWN
+                CPUs: 1
+              - NodeName: fake-2cpu-[3,7-9]
+                NodeAddr: 0.42.42.3,0.42.42.7,0.42.42.8,0.42.42.9
+                State: DOWN
+                CPUs: 2
+        openhpc_cluster_name: testohpc
+        openhpc_slurm_configless: true
+

molecule/test14/molecule.yml

@@ -0,0 +1,60 @@
+---
+name: single partition, group is partition
+driver:
+  name: docker
+platforms:
+  - name: testohpc-login-0
+    image: ${MOLECULE_IMAGE}
+    pre_build_image: true
+    groups:
+      - testohpc_login
+    command: /sbin/init
+    tmpfs:
+      - /run
+      - /tmp
+    volumes:
+      - /sys/fs/cgroup:/sys/fs/cgroup:ro
+    networks:
+      - name: net1
+    docker_networks:
+      - name: net1
+        driver_options:
+          com.docker.network.driver.mtu: ${DOCKER_MTU:-1500} # 1500 is docker default
+  - name: testohpc-compute-0
+    image: ${MOLECULE_IMAGE}
+    pre_build_image: true
+    groups:
+      - testohpc_compute
+    command: /sbin/init
+    tmpfs:
+      - /run
+      - /tmp
+    volumes:
+      - /sys/fs/cgroup:/sys/fs/cgroup:ro
+    networks:
+      - name: net1
+    docker_networks:
+      - name: net1
+        driver_options:
+          com.docker.network.driver.mtu: ${DOCKER_MTU:-1500} # 1500 is docker default
+  - name: testohpc-compute-1
+    image: ${MOLECULE_IMAGE}
+    pre_build_image: true
+    groups:
+      - testohpc_compute
+    command: /sbin/init
+    tmpfs:
+      - /run
+      - /tmp
+    volumes:
+      - /sys/fs/cgroup:/sys/fs/cgroup:ro
+    networks:
+      - name: net1
+    docker_networks:
+      - name: net1
+        driver_options:
+          com.docker.network.driver.mtu: ${DOCKER_MTU:-1500} # 1500 is docker default
+provisioner:
+  name: ansible
+verifier:
+  name: ansible

molecule/test14/verify.yml

@@ -0,0 +1,12 @@
+---
+
+- name: Check slurm hostlist
+  hosts: testohpc_login
+  tasks:
+  - name: Get slurm partition info
+    command: sinfo --noheader --format="%P,%a,%l,%D,%t,%N" # using --format ensures we control whitespace
+    register: sinfo
+  - name: 
+    assert:                        # PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST
+      that: "sinfo.stdout_lines == ['compute*,up,60-00:00:00,6,down*,fake-2cpu-[3,7-9],fake-x,fake-y', 'compute*,up,60-00:00:00,2,idle,testohpc-compute-[0-1]']"
+      fail_msg: "FAILED - actual value: {{ sinfo.stdout_lines }}"