[node-patches] Change in ovirt-node[master]: services: Adjust service dependencies

fabiand at fedoraproject.org fabiand at fedoraproject.org
Mon Jul 15 11:50:54 UTC 2013


Fabian Deutsch has uploaded a new change for review.

Change subject: services: Adjust service dependencies
......................................................................

services: Adjust service dependencies

Previously firstboot (and others) depended on systemd services or
targets which could go down during the original's service runtime.
Because some target service would depend on a service it would bring
down, systemd would also kill the target service (because a dependency
wasn't met anymore).

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=983547
Change-Id: I0596c054cd996d10032f737a8fcc010d60254bf0
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M services/ovirt-awake.service
M services/ovirt-cim.service
M services/ovirt-early.service
M services/ovirt-firstboot.service
M services/ovirt-kdump.service
M services/ovirt-node-igor-slave.service
M services/ovirt-post.service
M services/ovirt.service
8 files changed, 6 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/55/16855/1

diff --git a/services/ovirt-awake.service b/services/ovirt-awake.service
index e71903d..26da10e 100644
--- a/services/ovirt-awake.service
+++ b/services/ovirt-awake.service
@@ -1,6 +1,6 @@
 [Unit]
 Description=oVirt Node management server alert service
-After=ovirt-early.service
+After=ovirt-early.service ovirt-firstboot.service
 Wants=ovirt-early.service
 
 [Service]
diff --git a/services/ovirt-cim.service b/services/ovirt-cim.service
index 2a1eba0..9889fe1 100644
--- a/services/ovirt-cim.service
+++ b/services/ovirt-cim.service
@@ -1,6 +1,6 @@
 [Unit]
 Description=oVirt Node CIM configuration service
-After=ovirt-post.service
+After=ovirt-post.service ovirt-firstboot.service
 Wants=ovirt-post.service
 
 [Service]
diff --git a/services/ovirt-early.service b/services/ovirt-early.service
index 1fafc17..fb3c20c 100644
--- a/services/ovirt-early.service
+++ b/services/ovirt-early.service
@@ -1,7 +1,7 @@
 [Unit]
 Description=oVirt Node early configuration service
 After=fedora-readonly.target local-fs.target network.target
-Wants=fedora-readonly.target local-fs.target network.target
+Wants=fedora-readonly.target local-fs.target
 Before=sshd.service rsyslog.service
 
 [Service]
diff --git a/services/ovirt-firstboot.service b/services/ovirt-firstboot.service
index aae8f54..51d6787 100644
--- a/services/ovirt-firstboot.service
+++ b/services/ovirt-firstboot.service
@@ -1,7 +1,6 @@
 [Unit]
 Description=oVirt Node installation and configuration service
 After=ovirt-early.service
-Wants=ovirt-early.service
 Before=getty at tty1.service
 
 [Service]
diff --git a/services/ovirt-kdump.service b/services/ovirt-kdump.service
index 68d3142..e467593 100644
--- a/services/ovirt-kdump.service
+++ b/services/ovirt-kdump.service
@@ -1,6 +1,6 @@
 [Unit]
 Description=oVirt Node kdump service
-After=network.target
+After=network.target ovirt-early.service
 
 [Service]
 Type=oneshot
diff --git a/services/ovirt-node-igor-slave.service b/services/ovirt-node-igor-slave.service
index cef0bed..0a47a1e 100644
--- a/services/ovirt-node-igor-slave.service
+++ b/services/ovirt-node-igor-slave.service
@@ -1,6 +1,5 @@
 [Unit]
 Description=Communication with Igor (for test automation)
-Wants=basic.target network.target ovirt-early.service
 After=basic.target network.target
 Before=ovirt-early.service
 DefaultDependencies=no
diff --git a/services/ovirt-post.service b/services/ovirt-post.service
index 25c5581..5f64b8c 100644
--- a/services/ovirt-post.service
+++ b/services/ovirt-post.service
@@ -1,6 +1,6 @@
 [Unit]
 Description=oVirt Node post configuration service
-After=ovirt.service
+After=ovirt.service ovirt-firstboot.service
 Wants=ovirt.service
 Before=getty at tty1.service
 
diff --git a/services/ovirt.service b/services/ovirt.service
index a5a4ae2..ec2fa2d 100644
--- a/services/ovirt.service
+++ b/services/ovirt.service
@@ -1,6 +1,6 @@
 [Unit]
 Description=oVirt Node managed/standalone configuration service
-After=ovirt-awake.service
+After=ovirt-awake.service ovirt-firstboot.service
 Wants=ovirt-awake.service
 
 [Service]


-- 
To view, visit http://gerrit.ovirt.org/16855
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0596c054cd996d10032f737a8fcc010d60254bf0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>



More information about the node-patches mailing list