Skip to content

Commit ecf167d

Browse files
author
Matthew Fisher
committed
fix(builder): reside beside the controller
The client makes the assumption that the builder is on the same machine as the controller. We should eventually get this in a distributed way, but placing it on the same machine as the controller for now will work.
1 parent 1dcf378 commit ecf167d

2 files changed

Lines changed: 4 additions & 0 deletions

File tree

builder/systemd/deis-builder.service

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,3 +10,6 @@ ExecStop=/usr/bin/docker rm -f deis-builder
1010

1111
[Install]
1212
WantedBy=multi-user.target
13+
14+
[X-Fleet]
15+
X-ConditionMachineOf=deis-controller.service

client/deis.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -456,6 +456,7 @@ def apps_create(self, args):
456456
app_id = data['id']
457457
print("done, created {}".format(app_id))
458458
# add a git remote
459+
# TODO: retrieve the hostname from service discovery
459460
hostname = urlparse.urlparse(self._settings['controller']).netloc.split(':')[0]
460461
git_remote = "ssh://git@{hostname}:2222/{app_id}.git".format(**locals())
461462
try:

0 commit comments

Comments
 (0)