cloud-init-dev team mailing list archive

[Robert Schweikert <rjschwei@xxxxxxxx>]

Robert Schweikert has proposed merging ~rjschwei/cloud-init:serviceOrder into cloud-init:master.

Commit message:
Network configuration file gets written when cloud-init.service
runs. Therefore this needs to run prior to the network tools (wicked)

Reference: lp#1799709

Requested reviews:
  cloud-init commiters (cloud-init-dev)
Related bugs:
  Bug #1799709 in cloud-init: "service order is incorrect for SUSE distros"
  https://bugs.launchpad.net/cloud-init/+bug/1799709

For more details, see:
https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/357752
-- 
Your team cloud-init commiters is requested to review the proposed merge of ~rjschwei/cloud-init:serviceOrder into cloud-init:master.

diff --git a/systemd/cloud-init.service.tmpl b/systemd/cloud-init.service.tmpl
index b92e8ab..5cb0037 100644
--- a/systemd/cloud-init.service.tmpl
+++ b/systemd/cloud-init.service.tmpl
@@ -14,8 +14,7 @@ After=networking.service
 After=network.service
 {% endif %}
 {% if variant in ["suse"] %}
-Requires=wicked.service
-After=wicked.service
+Before=wicked.service
 # setting hostname via hostnamectl depends on dbus, which otherwise
 # would not be guaranteed at this point.
 After=dbus.service