cloud-init-dev team mailing list archive
-
cloud-init-dev team
-
Mailing list archive
-
Message #03464
Re: [Merge] ~rjschwei/cloud-init:tmplExpand into cloud-init:master
Diff comments:
> diff --git a/systemd/cloud-init-local.service.tmpl b/systemd/cloud-init-local.service.tmpl
> index ff9c644..870ccf1 100644
> --- a/systemd/cloud-init-local.service.tmpl
> +++ b/systemd/cloud-init-local.service.tmpl
> @@ -13,6 +13,10 @@ Before=shutdown.target
> Before=sysinit.target
> Conflicts=shutdown.target
> {% endif %}
> +{% if variant in ["suse"] %}
> +Before=basic.target
> +Conflicts=shutdown.target
trial and error, with me not documenting why one over the other. AT this point I'd rather keep it this way as I have no time ATM for re-testing
> +{% endif %}
> RequiresMountsFor=/var/lib/cloud
>
> [Service]
> diff --git a/systemd/cloud-init.service.tmpl b/systemd/cloud-init.service.tmpl
> index 2c71889..0dd45e5 100644
> --- a/systemd/cloud-init.service.tmpl
> +++ b/systemd/cloud-init.service.tmpl
> @@ -13,6 +13,11 @@ After=networking.service
> {% if variant in ["centos", "fedora", "redhat"] %}
> After=network.service
> {% endif %}
> +{% if variant in ["suse"] %}
> +Requires=wicked.service
> +After=wicked.service
> +After=dbus.service
Well that's the hostname setting bug, hostnamectl needs dbus, I know it's an ugly work around as discussed at the summit. In SUSE this does not cause any loops, thus until we get kdbus when this type of stuff goes away I think it is fair to use this ugly work around.
> +{% endif %}
> Before=network-online.target
> Before=sshd-keygen.service
> Before=sshd.service
--
https://code.launchpad.net/~rjschwei/cloud-init/+git/cloud-init/+merge/331140
Your team cloud-init commiters is requested to review the proposed merge of ~rjschwei/cloud-init:tmplExpand into cloud-init:master.
References