Skip to content

Commit 741825c

Browse files
carmstrongMatthew Fisher
authored andcommitted
bug(systemd): fix dependencies
This commit does several things: * Refactor After and Requires to list all units on one line * Make seed-registry job dependent upon deis-registry * Make controller dependent on (and wait for) Builder * Make all services only start after etcd closes #706, #708
1 parent 110f093 commit 741825c

9 files changed

Lines changed: 26 additions & 40 deletions

File tree

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-builder
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-builder || /usr/bin/docker run --name deis-builder -p 2222:22 -e PUBLISH=22 -e HOST=172.17.8.100 -e PORT=2222 -e ETCD=172.17.8.100:4001 --privileged deis/builder'
108
ExecStop=/usr/bin/docker stop deis-builder
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target

cache/systemd/deis-cache.service

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-cache
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-cache || /usr/bin/docker run --name deis-cache -p 6379:6379 -e PUBLISH=6379 -e HOST=172.17.8.100 -e ETCD=172.17.8.100:4001 deis/cache'
108
ExecStop=/usr/bin/docker stop deis-cache
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target

contrib/coreos/user-data

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,8 @@ coreos:
8787
content: |
8888
[Unit]
8989
Description=Seed Deis Registry with Docker Images
90-
Requires=seed-docker-images.service
90+
Requires=seed-docker-images.service deis-registry.service
91+
After=deis-registry.service
9192

9293
[Service]
9394
Type=oneshot
Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-controller
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service deis-builder.service
4+
After=docker.service etcd.service deis-builder.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-controller || /usr/bin/docker run --name deis-controller -p 8000:8000 -e PUBLISH=8000 -e HOST=172.17.8.100 -e ETCD=172.17.8.100:4001 deis/controller'
108
ExecStop=/usr/bin/docker stop deis-controller
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target
Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-database
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-database || /usr/bin/docker run --name deis-database -p 5432:5432 -e PUBLISH=5432 -e HOST=172.17.8.100 -e ETCD=172.17.8.100:4001 deis/database'
108
ExecStop=/usr/bin/docker stop deis-database
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target
Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
[Unit]
22
Description=deis-discovery
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
65
StandardOutput=null
76

87
[Service]
98
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-discovery || /usr/bin/docker run --name deis-discovery -p 5001:4001 -e PUBLIC_IP=10.1.42.1 deis/discovery'
109
ExecStop=/usr/bin/docker stop deis-discovery
1110

1211
[Install]
13-
WantedBy=multi-user.target
12+
WantedBy=multi-user.target

logger/systemd/deis-logger.service

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-logger
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-logger || /usr/bin/docker run --name deis-logger -p 514:514 -e PUBLISH=514 -e HOST=172.17.8.100 -e ETCD=172.17.8.100:4001 deis/logger'
108
ExecStop=/usr/bin/docker stop deis-logger
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target
Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-registry
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-registry || /usr/bin/docker run --name deis-registry -p 5000:5000 -e PUBLISH=5000 -e HOST=172.17.8.100 -e ETCD=172.17.8.100:4001 deis/registry'
108
ExecStop=/usr/bin/docker stop deis-registry
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target

router/systemd/deis-router.service

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,11 @@
11
[Unit]
22
Description=deis-router
3-
After=docker.service
4-
Requires=docker.service
5-
Requires=docker-tcp.socket
6-
Requires=etcd.service
3+
Requires=docker.service docker-tcp.socket etcd.service
4+
After=docker.service etcd.service
75

86
[Service]
97
ExecStart=/bin/bash -c '/usr/bin/docker start -a deis-router || /usr/bin/docker run --name deis-router -p 80:80 -e PUBLISH=80 -e HOST=172.17.8.100 -e ETCD=172.17.8.100:4001 deis/router'
108
ExecStop=/usr/bin/docker stop deis-router
119

1210
[Install]
13-
WantedBy=multi-user.target
11+
WantedBy=multi-user.target

0 commit comments

Comments
 (0)