testinfra (#44): mananged components now create simple file at /opt/a…

…lb/info/ to allow tests know what to check

handlers/main.yml

@@ -25,3 +25,62 @@
 
 - name: restart ufw
   service: name=ufw state=restarted
+
+## About /opt/alb/info/ files
+# This folder may contain some very basic information about enabled components
+# on each ALB node. The existence means a service was installed and (eventually)
+# it's contents (INI format) could be used to describe more about the service.
+#
+# One main initial reason is allow testinfra (and later other scripts) know
+# what state the node was expected to be. At this moment it just create empty
+# file.
+
+- name: "info alb component openresty"
+  file:
+    path: "/opt/alb/info/openresty"
+    state: touch
+    modification_time: preserve
+    access_time: preserve
+    owner: "{{ alb_internal_root_user }}"
+    group: "{{ alb_internal_root_group }}"
+    mode: "0640"
+
+- name: "info alb component haproxy"
+  file:
+    path: "/opt/alb/info/haproxy"
+    state: touch
+    modification_time: preserve
+    access_time: preserve
+    owner: "{{ alb_internal_root_user }}"
+    group: "{{ alb_internal_root_group }}"
+    mode: "0640"
+
+- name: "info alb component ufw"
+  file:
+    path: "/opt/alb/info/ufw"
+    state: touch
+    modification_time: preserve
+    access_time: preserve
+    owner: "{{ alb_internal_root_user }}"
+    group: "{{ alb_internal_root_group }}"
+    mode: "0640"
+
+- name: "info alb component apps"
+  file:
+    path: "/opt/alb/info/apps"
+    state: touch
+    modification_time: preserve
+    access_time: preserve
+    owner: "{{ alb_internal_root_user }}"
+    group: "{{ alb_internal_root_group }}"
+    mode: "0640"
+
+- name: "info alb component sysapps"
+  file:
+    path: "/opt/alb/info/sysapps"
+    state: touch
+    modification_time: preserve
+    access_time: preserve
+    owner: "{{ alb_internal_root_user }}"
+    group: "{{ alb_internal_root_group }}"
+    mode: "0640"

molecule/default/tests/test_default.py

@@ -7,45 +7,46 @@
 ).get_hosts('all')
 
 
-def test_hosts_file(host):
-    f = host.file('/etc/hosts')
+def test_required_haproxy_is_installed(host):
+    assert (not host.file("/opt/alb/info/haproxy").exists) \
+        or host.package("haproxy").is_installed
 
-    assert f.exists
-    assert f.user == 'root'
-    assert f.group == 'root' or f.group == 'wheel'
 
+def test_required_haproxy_is_enabled(host):
+    assert (not host.file("/opt/alb/info/haproxy").exists) \
+        or host.service("haproxy").is_enabled
 
-def test_haproxy_is_installed(host):
-    assert host.package("haproxy").is_installed
 
+def test_required_haproxy_is_running(host):
+    assert (not host.file("/opt/alb/info/haproxy").exists) \
+        or host.service("haproxy").is_running
 
-def test_haproxy_is_enabled(host):
-    assert host.service("haproxy").is_enabled
 
+def test_required_openresty_is_installed(host):
+    assert (not host.file("/opt/alb/info/openresty").exists) \
+        or host.package("openresty").is_installed
 
-def test_haproxy_is_running(host):
-    assert host.service("haproxy").is_running
 
+def test_required_openresty_is_enabled(host):
+    assert (not host.file("/opt/alb/info/openresty").exists) \
+        or host.service("openresty").is_enabled
 
-def test_openresty_is_installed(host):
-    assert host.package("openresty").is_installed
 
+def test_required_openresty_is_running(host):
+    assert (not host.file("/opt/alb/info/openresty").exists) \
+        or host.service("openresty").is_running
 
-def test_openresty_is_enabled(host):
-    assert host.service("openresty").is_enabled
 
+def test_required_ufw_is_installed(host):
+    assert (not host.file("/opt/alb/info/ufw").exists) \
+        or host.package("ufw").is_installed
 
-def test_openresty_is_running(host):
-    assert host.service("openresty").is_running
 
+def test_required_ufw_is_enabled(host):
+    assert (not host.file("/opt/alb/info/ufw").exists) \
+        or host.service("ufw").is_enabled
 
-#def test_ufw_is_installed(host):
-#    assert host.package("ufw").is_installed
-#
-#
-#def test_ufw_is_enabled(host):
-#    assert host.service("ufw").is_enabled
-#
-#
-#def test_ufw_is_running(host):
-#    assert host.service("ufw").is_running
+
+def test_required_ufw_is_running(host):
+    assert (not host.file("/opt/alb/info/ufw").exists) \
+        or host.service("ufw").is_running

tasks/apps/alb/openresty-conf.yml

@@ -24,4 +24,7 @@
     - "(alb_run_only_app_uid is not defined) or (alb_run_only_app_uid == item.app_uid)"
     - "(alb_apps_facts_deployed is changed) or (alb_forceredeploy|bool)"
   register: result_apps_rules
-  notify: reload openresty
+  notify:
+    - "reload openresty"
+    - "info alb component apps"
+

tasks/haproxy/management/bsd-init.yml

@@ -12,12 +12,13 @@
     mode: '0644'
     backup: yes
   notify:
-    - reload haproxy
+    - "reload haproxy"
+    - "info alb component haproxy"
 
 # Enabled/Started must be AFTER moving new /etc/haproxy/haproxy.cfg
 # configurations or Ansible will not be able to update one new valid
 # configuration if old one was already with error. By moving this step
-# after we avoid user being forced to solve manually on the server 
+# after we avoid user being forced to solve manually on the server
 - name: "haproxy | management | bsd-init.yml: enable haproxy.service"
   service:
     name: haproxy

tasks/haproxy/management/systemd.yml

@@ -13,6 +13,7 @@
     backup: yes
   notify:
     - reload haproxy
+    - info alb component haproxy
 
 # Enabled/Started must be AFTER moving new /etc/haproxy/haproxy.cfg
 # configurations or Ansible will not be able to update one new valid

tasks/openresty/install/default-files.yml

@@ -19,4 +19,5 @@
     mode: '0644'
     backup: yes
   notify:
-    - reload openresty
+    - "reload openresty"
+    - "info alb component openresty"

tasks/openresty/management/nginx-conf-global.yml

@@ -11,4 +11,5 @@
     mode: '0644'
     backup: yes
   notify:
-    - reload openresty
+    - "reload openresty"
+    - "info alb component openresty"

tasks/sysapps/sysapps.yml

@@ -69,7 +69,9 @@
   when:
     - alb_internal_sysapps is defined and alb_internal_sysapps[0] is defined
     - "(alb_sysapps_facts_deployed is changed) or (alb_forceredeploy|bool)"
-  notify: reload openresty
+  notify:
+    - "reload openresty"
+    - "info alb component sysapps"
 
 - name: "sysapps | mkdir /var/log/sysapp/[app_uid]"
   file:

tasks/ufw/install-ufw.yml

@@ -6,3 +6,5 @@
     state: present
   when:
     - "alb_manange_ufw_install is defined and alb_manange_ufw_install|bool"
+  notify:
+    - "info alb component ufw"

tasks/ufw/rules.yml

@@ -109,5 +109,7 @@
   loop: "{{ ufw_rules }}"
   loop_control:
     extended: yes
-  notify: reload ufw
+  notify:
+    - "reload ufw"
+    - "info alb component ufw"
   when: "ufw_rules is defined"

tests/test_alb-standard-node.py

@@ -1,36 +1,45 @@
 # FILE: /opt/alb/bin/tests/test_alb-standard-node.py
 
-def test_haproxy_is_installed(host):
-    assert host.package("haproxy").is_installed
+def test_required_haproxy_is_installed(host):
+    assert (not host.file("/opt/alb/info/haproxy").exists) \
+        or host.package("haproxy").is_installed
 
 
-def test_haproxy_is_enabled(host):
-    assert host.service("haproxy").is_enabled
+def test_required_haproxy_is_enabled(host):
+    assert (not host.file("/opt/alb/info/haproxy").exists) \
+        or host.service("haproxy").is_enabled
 
 
-def test_haproxy_is_running(host):
-    assert host.service("haproxy").is_running
+def test_required_haproxy_is_running(host):
+    assert (not host.file("/opt/alb/info/haproxy").exists) \
+        or host.service("haproxy").is_running
 
 
-def test_openresty_is_installed(host):
-    assert host.package("openresty").is_installed
+def test_required_openresty_is_installed(host):
+    assert (not host.file("/opt/alb/info/openresty").exists) \
+        or host.package("openresty").is_installed
 
 
-def test_openresty_is_enabled(host):
-    assert host.service("openresty").is_enabled
+def test_required_openresty_is_enabled(host):
+    assert (not host.file("/opt/alb/info/openresty").exists) \
+        or host.service("openresty").is_enabled
 
 
-def test_openresty_is_running(host):
-    assert host.service("openresty").is_running
+def test_required_openresty_is_running(host):
+    assert (not host.file("/opt/alb/info/openresty").exists) \
+        or host.service("openresty").is_running
 
 
-def test_ufw_is_installed(host):
-    assert host.package("ufw").is_installed
+def test_required_ufw_is_installed(host):
+    assert (not host.file("/opt/alb/info/ufw").exists) \
+        or host.package("ufw").is_installed
 
 
-def test_ufw_is_enabled(host):
-    assert host.service("ufw").is_enabled
+def test_required_ufw_is_enabled(host):
+    assert (not host.file("/opt/alb/info/ufw").exists) \
+        or host.service("ufw").is_enabled
 
 
-def test_ufw_is_running(host):
-    assert host.service("ufw").is_running
+def test_required_ufw_is_running(host):
+    assert (not host.file("/opt/alb/info/ufw").exists) \
+        or host.service("ufw").is_running