MAINT: Remove volume facts from play

We cannot rely on the OpenStack API to provide accurate information about where the device is mounted. This is due to a discrepancy between OpenStack and the OS of the VM. To work around this we will try the two most common devices, sdb and vdb. If both of these fail intervention will be required. A bonus to this is it removes the convoluted ansible fact solution previously used.

Author: khalford

chatops_deployment/ansible/deploy.yml

@@ -20,3 +20,9 @@
   roles:
     - role: ssh_known_hosts
       tags: known_hosts
+
+- name: Create certificates
+  hosts: localhost
+  roles:
+    - role: ssl_certificates
+      tags: ssl_certificates

chatops_deployment/ansible/roles/terraform/tasks/deploy.yml

@@ -83,11 +83,3 @@
     force_init: true
     workspace: "{{ env }}"
   register: terraform_output
-
-- name: Write ansible inventory file from Terraform outputs
-  ansible.builtin.template:
-    src: hosts.ini.j2
-    dest: "{{ playbook_dir }}/hosts.ini"
-    owner: "{{ ansible_env.USER }}"
-    group: "{{ ansible_env.USER }}"
-    mode: "0774"

chatops_deployment/ansible/roles/volume/tasks/main.yml

@@ -1,12 +1,23 @@
 ---
 - name: Attach volume to stack host
-  become: true
-  ansible.posix.mount:
-    boot: true
-    path: /var/stack
-    src: "{{ ansible_local.terraform.vars.stack_device }}"
-    state: mounted
-    fstype: ext4
+  block:
+    - name: Try sdb
+      become: true
+      ansible.posix.mount:
+        boot: true
+        path: /var/stack
+        src: /dev/sdb
+        state: mounted
+        fstype: ext4
+  rescue:
+    - name: Try vdb
+      become: true
+      ansible.posix.mount:
+        boot: true
+        path: /var/stack
+        src: /dev/vdb
+        state: mounted
+        fstype: ext4
 
 - name: Set permissions
   become: true

chatops_deployment/terraform/modules/compute/outputs.tf

@@ -1,3 +0,0 @@
-output "stack_volume_device" {
-  value = openstack_compute_volume_attach_v2.stack_volume.device
-}

chatops_deployment/terraform/outputs.tf

@@ -1,3 +0,0 @@
-output "stack_volume_device" {
-  value = module.compute.stack_volume_device
-}