machinefile with port numbers broken #7618
Closed
wavexx opened this Issue on Jul 15, 2014 · 3 comments
Projects
None yet
Labels
bug
parallel
regression
Milestone
 
0.3
Assignees
No one assigned
2 participants
@wavexx
@JeffBezanson
Notifications

You’re not receiving notifications from this thread.
@wavexx
wavexx commented on Jul 15, 2014

I noticed that with the following machinefile:

first:1234
anotherhost

fails to connect. This is due launch_ssh_workers appending the current port to config[:sshflags]. If the subsequent hosts do not specify a port number, the previous one is used.

I could fix this, but I'm waiting for #7599
@wavexx wavexx referenced this issue on Jul 15, 2014
Merged
Add `-T` -a` to the default ssh command/s. #7599
@JeffBezanson JeffBezanson added bug parallel regression labels on Jul 20, 2014
@JeffBezanson
The Julia Language member
JeffBezanson commented on Jul 23, 2014

@wavexx do you have time to write up a PR to fix this?
@wavexx
wavexx commented on Jul 24, 2014
On 07/23/2014 11:44 PM, Jeff Bezanson wrote:
 @wavexx do you have time to write up a PR to fix this?
In the next days, yes.
@JeffBezanson JeffBezanson added this to the 0.3 milestone on Jul 26, 2014
@JeffBezanson JeffBezanson added a commit that closed this issue on Jul 29, 2014
	@JeffBezanson 	fix #7618, handling of port numbers starting ssh workers. 			345f3bd
@JeffBezanson JeffBezanson closed this in 345f3bd on Jul 29, 2014
@JeffBezanson
The Julia Language member
JeffBezanson commented on Jul 29, 2014

I think this should work but let me know if I missed anything.