← Back to team overview

cloud-init-dev team mailing list archive

[Merge] ~prometheanfire/cloud-init:fix-gentoo-init into cloud-init:master

 

Matthew Thode has proposed merging ~prometheanfire/cloud-init:fix-gentoo-init into cloud-init:master.

Requested reviews:
  cloud init development team (cloud-init-dev)

For more details, see:
https://code.launchpad.net/~prometheanfire/cloud-init/+git/cloud-init/+merge/307969

    update Gentoo initscripts to run in the correct order
    
    cloud-init-local needs to be run before net (and in the boot runlevel)
    as it sets up networking config and adds it to the default runlevel.
    
    cloud-init-local needs to be run in the boot runlevel because it
    modifies services in the default runlevel.  When a runlevel is started
    it is cached, so modifications that happen to the current runlevel while
    you are in it are not acted upon.
    
    cloud-init needs to run after net, it does not need net because we still
    want it to set up other things if it can.
-- 
Your team cloud init development team is requested to review the proposed merge of ~prometheanfire/cloud-init:fix-gentoo-init into cloud-init:master.
diff --git a/sysvinit/gentoo/cloud-init b/sysvinit/gentoo/cloud-init
index 5afc0f2..531a715 100644
--- a/sysvinit/gentoo/cloud-init
+++ b/sysvinit/gentoo/cloud-init
@@ -2,6 +2,7 @@
 # add depends for network, dns, fs etc
 depend() {
   after cloud-init-local
+  after net
   before cloud-config
   provide cloud-init
 }
diff --git a/sysvinit/gentoo/cloud-init-local b/sysvinit/gentoo/cloud-init-local
index 9bd0b56..0f8cf65 100644
--- a/sysvinit/gentoo/cloud-init-local
+++ b/sysvinit/gentoo/cloud-init-local
@@ -2,7 +2,7 @@
 
 depend() {
   after localmount
-  after netmount
+  before net
   before cloud-init
   provide cloud-init-local
 }

Follow ups