Restore fails with no instances found
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical Juju |
Fix Released
|
Critical
|
Cheryl Jennings | ||
juju-core |
Fix Released
|
High
|
Anastasia | ||
1.25 |
Fix Released
|
High
|
Anastasia |
Bug Description
From a recent change on master, the restore command now fails with "no instances found; perhaps the model was not bootstrapped".
<http://
<http://
Instrumenting node failure for member 0: i-45c013de at 54.165.0.218
Deleting i-45c013de.
INSTANCE i-45c013de
Waiting for port to close on 54.165.0.218
Closed.
Starting restore.
Call of juju restore exited with an error
Restore failed:
2016-03-15 04:50:50 INFO juju.cmd supercommand.go:60 running juju [2.0-beta3 gc go1.2.1]
2016-03-15 04:50:50 INFO juju.environs.
2016-03-15 04:50:50 INFO juju.provider.ec2 provider.go:48 opening model "functional-
2016-03-15 04:50:51 ERROR cmd supercommand.go:448 no instances found; perhaps the model was not bootstrapped
tags: | added: blocker |
Changed in juju-core: | |
assignee: | nobody → Cheryl Jennings (cherylj) |
Changed in juju-core: | |
status: | Triaged → In Progress |
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |
tags: | added: 2.0-count |
affects: | juju-core → juju |
Changed in juju: | |
milestone: | 2.0-beta3 → none |
milestone: | none → 2.0-beta3 |
Changed in juju-core: | |
assignee: | nobody → Anastasia (anastasia-macmood) |
importance: | Undecided → High |
status: | New → Fix Released |
Injected by PR: https:/ /github. com/juju/ juju/pull/ 4467
The above PR changes how we keep track of our controllers. Previously, it was a list inside juju that was added to when new controllers were created, and pruned when removing controllers. After this PR, we simply tag ec2 instances with juju-is- controller: true, and run a query on ec2 instances with that tag to get the list of controllers.
This means that for this CI test, since we terminate the controller instance before attempting to restore, we no longer return any controllers for this model. This should be fine, since it means it should be eligible for restore -b, but the logic in restore was not updated to handle this behavior.
This should be a simple matter of updating restore to treat a controller instance list length of 0 the same as not being able to find a controller instance by ID.