zuul/nodepool
Code Issues Proposed changes

Revert "Create mandatory /etc/nodepool for later use"

Browse Source 
At first glance, this may be causing a hang on the remote side that is
causing the nodepool test to hang. We didn't notice it because the test
was otherwise broken.

This reverts commit 44ee08c2d71937816d0029bb60c05a9d8b802655.

Change-Id: Iaef2a140e33fc48f8bfa8ff4769eded37ce152c6
This commit is contained in:
Monty Taylor 2017-01-31 16:18:08 -06:00
parent 7c217682a4
commit 491fae7b28
No known key found for this signature in database
GPG Key ID: 7BAE94BC7141A594
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
nodepool/nodepool.py
View File

@ -625,9 +625,7 @@ class NodeLauncher(threading.Thread):
if not host: if not host:
raise Exception("Unable to log in via SSH") raise Exception("Unable to log in via SSH")
host.ssh("Create config dir", "sudo mkdir -p /etc/nodepool") host.ssh("test for config dir", "ls /etc/nodepool")
host.ssh("Change config dir owner",
"sudo chown %s -R /etc/nodepool" % self.image.username)
ftp = host.client.open_sftp() ftp = host.client.open_sftp()