https://codereview.appspot.com/81700043/diff/1/cmd/juju/ensureha.go#newcode50
cmd/juju/ensureha.go:50: Name: "ensure-ha",
I'm not sure about "ha" in lower case - it looks too much like a word.
I'm thinking that either we make it into a proper word ("availability")
or we capitalise the ha - ensure-HA, thus making it clearer that it's an
acronym.
https://codereview.appspot.com/81700043/diff/1/cmd/juju/ensureha.go#newcode58
cmd/juju/ensureha.go:58: f.IntVar(&c.NumStateServers, "n", 1, "number of
state servers to make available")
Thought for the future - it would be good to be able to run "juju
ensure-HA" without any arguments at all, for when we just want to
maintain the current availability. Perhaps rather than defaulting to 1,
we should default to -1 and complain if it's not specified. That way we
leave that path open.
LGTM modulo the below, thanks!
https:/ /codereview. appspot. com/81700043/ diff/1/ cmd/juju/ ensureha. go ensureha. go (right):
File cmd/juju/
https:/ /codereview. appspot. com/81700043/ diff/1/ cmd/juju/ ensureha. go#newcode19 ensureha. go:19: // If specified, use this series, else use the
cmd/juju/
environment default-series
use this series for newly started instances ?
https:/ /codereview. appspot. com/81700043/ diff/1/ cmd/juju/ ensureha. go#newcode50 ensureha. go:50: Name: "ensure-ha",
cmd/juju/
I'm not sure about "ha" in lower case - it looks too much like a word.
I'm thinking that either we make it into a proper word ("availability")
or we capitalise the ha - ensure-HA, thus making it clearer that it's an
acronym.
https:/ /codereview. appspot. com/81700043/ diff/1/ cmd/juju/ ensureha. go#newcode58 ensureha. go:58: f.IntVar( &c.NumStateServ ers, "n", 1, "number of
cmd/juju/
state servers to make available")
Thought for the future - it would be good to be able to run "juju
ensure-HA" without any arguments at all, for when we just want to
maintain the current availability. Perhaps rather than defaulting to 1,
we should default to -1 and complain if it's not specified. That way we
leave that path open.
https:/ /codereview. appspot. com/81700043/ diff/1/ state/api/ client. go
File state/api/client.go (right):
https:/ /codereview. appspot. com/81700043/ diff/1/ state/api/ client. go#newcode680 client. go:680: } else if err := result.Error; err != nil {
state/api/
There's no point in having an Error in the result as well as an error
return.
https:/ /codereview. appspot. com/81700043/ diff/1/ state/api/ params/ params. go params/ params. go (right):
File state/api/
https:/ /codereview. appspot. com/81700043/ diff/1/ state/api/ params/ params. go#newcode649 params/ params. go:649: Error *Error
state/api/
d
https:/ /codereview. appspot. com/81700043/ diff/1/ state/apiserver /client/ client. go /client/ client. go (right):
File state/apiserver
https:/ /codereview. appspot. com/81700043/ diff/1/ state/apiserver /client/ client. go#newcode990 /client/ client. go:990: if err := EnsureAvailabil ity(args. NumStateServers , args.Constraints, EnsureAvailabil ity(args. NumStateServers , EnsureAvailabil ityResult{ }, err
state/apiserver
c.api.state.
series); err != nil {
err := c.api.state.
args.Constraints, series)
return params.
https:/ /codereview. appspot. com/81700043/