ensure-availability can add more state servers while ones it just started haven't come up yet
Bug #1307736 reported by
Roger Peppe
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
High
|
Wayne Witzel III |
Bug Description
I was just playing with HA, and saw the following behaviour.
Before running the commands, I'd had an environment with
3 environment managers (0, 3 and 4), of which I had destroyed the instance
of 0. I was expecting it to remove the EnvironManager status
of that one (machine 0) but definitely did not expect
to see it add two more machines... (6 and 7)
http://
(Note that I'd tricked out juju status with the ability to show voting status)
Related branches
lp:~wwitzel3/juju-core/010-bootstrap-ensure
Ready for review
for merging
into
lp:~go-bot/juju-core/trunk
- Juju Engineering: Pending requested
-
Diff: 487 lines (+80/-70)12 files modifiedcmd/juju/ensureavailability_test.go (+2/-2)
cmd/juju/status_test.go (+10/-10)
state/addmachine.go (+1/-1)
state/apiserver/admin.go (+12/-3)
state/apiserver/client/api_test.go (+1/-1)
state/apiserver/client/client_test.go (+14/-14)
state/apiserver/client/status.go (+2/-2)
state/apiserver/server_test.go (+2/-2)
state/machine.go (+8/-7)
state/machine_test.go (+11/-11)
state/unit.go (+6/-6)
state/unit_test.go (+11/-11)
tags: | added: ha |
Changed in juju-core: | |
status: | New → Triaged |
importance: | Undecided → High |
milestone: | none → 1.20.0 |
Changed in juju-core: | |
milestone: | 1.20.0 → next-stable |
Changed in juju-core: | |
assignee: | nobody → Wayne Witzel III (wwitzel3) |
status: | Triaged → In Progress |
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
milestone: | next-stable → 1.19.4 |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I don't think there's a bug here; note that machine agents 3 and 4 are down after the ensure-availability call, and that their wants-vote=false.
Unless there really is a bug, those machine agents were down when you called ensure- availability, and it did the right thing and took away their voting rights and created two replacement machines.
Having said that, this isn't ideal. Machines 3 and 4 were down temporarily; it would be good to (a) have ensure-availability say what it has done (and why), and (b) improve the availability heuristic. Perhaps we need an uptime for agents as well as current availability.