Merge "Rearrange set-hostnames and cloud-init removal"

playbooks/roles/base-server/tasks/Debian.yaml

@@ -41,8 +41,12 @@
       - popularity-contest
       - lxd
       - lxd-client
+      - cloud-init
     state: absent
 
+- name: Get rid of extra depends
+  command: apt-get autoremove -y
+
 - name: Configure file limits
   copy:
     mode: 0644

playbooks/roles/set-hostname/README.rst

@@ -1,7 +1,6 @@
 Set hostname
 
-Remove ``cloud-init`` and statically set the hostname, hosts and
-mailname
+Statically set the hostname, hosts and mailname
 
 **Role Variables**
 

playbooks/roles/set-hostname/tasks/main.yml

@@ -1,11 +1,11 @@
----
-- name: Remove cloud-init
-  package:
-    name: cloud-init
-    state: absent
-
-- name: Get rid of cloud-init depends
-  command: apt-get autoremove -y
+# Setting hostname with systemd apparently
+# requires dbus. We have this on our cloud-provided
+# nodes, but not on the minimal ones we get from
+# nodepool.
+- name: ensure dbus for working hostnamectl
+  apt:
+    name: dbus
+    state: present
 
 # Set hostname and /etc/hosts
 # Inspired by:

playbooks/set-hostnames.yaml

@@ -1,4 +1,4 @@
-- hosts: "{{ target }}"
+- hosts: "!disabled"
   gather_facts: false
   user: root
   roles:

playbooks/zuul/run-base.yaml

@@ -91,6 +91,8 @@
         dest: /home/zuul/src/opendev.org/opendev/system-config/playbooks/host_vars/bridge.openstack.org.yaml
       become: true
 
+    - name: Set hostname on host
+      command: ansible-playbook -v /home/zuul/src/opendev.org/opendev/system-config/playbooks/set-hostnames.yaml
     - name: Run base.yaml
       command: ansible-playbook -v /home/zuul/src/opendev.org/opendev/system-config/playbooks/base.yaml
     - name: Run bridge service playbook