remove rebuild group from everything layout - configure in generated inventory instead

Author: sjpb

README.md

@@ -121,7 +121,7 @@ Although most of the inventory uses the group convention described above there a
     - An inventory group `<cluster_name>_<partition_name>` defining the hosts it contains - these must be homogenous w.r.t CPU and memory.
     - An entry in the `openhpc_slurm_partitions` mapping in `environments/<environment>/inventory/group_vars/openhpc/overrides.yml`.
     See the [openhpc role documentation](https://github.com/stackhpc/ansible-role-openhpc#slurmconf) for more options.
-
+- On an OpenStack cloud, rebuilding/reimaging compute nodes from Slurm can be enabled by defining a `rebuild` group containing the relevant compute hosts (e.g. in the generated `hosts` file).
 
 ## Adding new functionality
 TODO: this is just rough notes:

dev/vagrant-example-configure.sh

@@ -18,16 +18,6 @@ echo "Running generate-passwords.yml"
 
 ansible-playbook ansible/adhoc/generate-passwords.yml
 
-echo "Mocking openstack command for rebuild functionality"
-# we don't need to mock the cloud file as it only needs to exist
-mkdir /tmp/vagrant-example
-cat > /tmp/vagrant-example/openstack <<EOL
-#!/usr/bin/bash
-echo "fake openstack command"
-EOL
-chmod u+x /tmp/vagrant-example/openstack
-export PATH=/tmp/vagrant-example:$PATH
-
 echo "Running site.yml"
 
 ansible-playbook -vvvv ansible/site.yml -e "openhpc_rebuild_clouds=/tmp/vagrant-example/openstack"

environments/common/layouts/everything

@@ -36,6 +36,3 @@ control
 
 [filebeat:children]
 slurm_stats
-
-[rebuild:children]
-compute