Code review comment for lp:~dimitern/juju-core/270-lp-1257649-ssh-timeout-bootstrap

Revision history for this message
Nate Finch (natefinch) wrote :

Please remove all user-facing mentions of SSH, since that's an internal
detail and something that would just confuse users. This is just a
bootstrap timeout to an end user, even if it does use SSH behind the
scenes.

https://codereview.appspot.com/58170045/diff/1/cmd/juju/bootstrap.go
File cmd/juju/bootstrap.go (right):

https://codereview.appspot.com/58170045/diff/1/cmd/juju/bootstrap.go#newcode48
cmd/juju/bootstrap.go:48: bootstrap-ssh-timeout: 600 # default: 10
minutes
ssh is an internal detail that will just be confusing to users. This is
just the "bootstrap-timeout" to an end user.
  Same goes for retry delay and addresses delay.

https://codereview.appspot.com/58170045/

« Back to merge proposal