Retry failed ssh commands in the ec2 python script. #555

Merged
merged 2 commits into from Apr 10, 2013

Conversation

Projects
None yet
3 participants
Contributor

holdenk commented Apr 6, 2013

This is especially useful when starting up when the hosts may not have yet come on-line but can be useful at other points for people with flakey connections

@holdenk holdenk Retry failed ssh commands. This is especially useful during system st…
…artup when the hosts may not have yet come on-line but can be useful at other points for people with flakey connections
ff2130a

Can one of the admins verify this patch?

Owner

mateiz commented Apr 8, 2013

This looks like a good thing to fix, but I'd prefer doing it just inside the "ssh" method instead of using a decorator, since we won't use this in a lot of other places, and it will make the code easier to follow. Just put a loop in there.

Contributor

holdenk commented Apr 10, 2013

Sounds good, I'll do that.

Contributor

holdenk commented Apr 10, 2013

Cool I've switched it to just be a simple loop in the script and removed the retry decorator code.

mateiz merged commit 271a4f3 into mesos:master Apr 10, 2013

Owner

mateiz commented Apr 10, 2013

Thanks, that looks great.

@nkronenfeld nkronenfeld pushed a commit to nkronenfeld/spark that referenced this pull request Jul 15, 2014

@kalpit @pwendell kalpit + pwendell sbt 0.13.X should be using sbt-assembly 0.11.X
https://github.com/sbt/sbt-assembly/blob/master/README.md

Author: Kalpit Shah <shahkalpit84@gmail.com>

Closes #555 from kalpit/upgrade/sbtassembly and squashes the following commits:

1fa7324 [Kalpit Shah] sbt 0.13.X should be using sbt-assembly 0.11.X
5473aa7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment