juju does not retry provisioning against transient provider errors
Bug #1227450 reported by
Kapil Thangavelu
This bug affects 10 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
Medium
|
Unassigned |
Bug Description
Say i've exceeded some limit against my provider. juju will fail at provisioning. i then go to increase my limit at the provider, or remove unused resources. The condition was effectively just a transient error. Juju does not attempt to provision the resource again, the resources is a in a permanent error state in juju's perspective, even though a retry would succeed.
Related branches
lp:~wallyworld/juju-core/provisioner-retry
- Juju Engineering: Pending requested
-
Diff: 648 lines (+311/-43)12 files modifiedstate/api/provisioner/provisioner.go (+13/-0)
state/api/provisioner/provisioner_test.go (+1/-11)
state/apiserver/provisioner/machineerror.go (+80/-0)
state/apiserver/provisioner/provisioner.go (+32/-4)
state/apiserver/provisioner/provisioner_test.go (+43/-2)
state/machine.go (+5/-1)
state/machine_test.go (+10/-2)
state/status.go (+6/-2)
state/unit.go (+1/-1)
worker/provisioner/provisioner.go (+10/-4)
worker/provisioner/provisioner_task.go (+35/-11)
worker/provisioner/provisioner_test.go (+75/-5)
lp:~wallyworld/juju-core/machineswithtransienterrors-api
- Juju Engineering: Pending requested
-
Diff: 707 lines (+361/-34)17 files modifiedstate/api/client.go (+14/-0)
state/api/machiner/machine.go (+1/-1)
state/api/params/internal.go (+7/-4)
state/api/provisioner/machine.go (+3/-3)
state/api/provisioner/provisioner.go (+23/-0)
state/api/provisioner/provisioner_test.go (+50/-2)
state/api/uniter/unit.go (+1/-1)
state/apiserver/client/client.go (+18/-4)
state/apiserver/client/client_test.go (+15/-0)
state/apiserver/common/setstatus.go (+50/-0)
state/apiserver/common/setstatus_test.go (+66/-2)
state/apiserver/machine/machiner_test.go (+1/-1)
state/apiserver/provisioner/provisioner.go (+36/-1)
state/apiserver/provisioner/provisioner_test.go (+68/-13)
state/apiserver/uniter/uniter_test.go (+1/-1)
state/interface.go (+6/-0)
worker/provisioner/provisioner_task.go (+1/-1)
lp:~wallyworld/juju-core/retryprovisioning-command
- Juju Engineering: Pending requested
-
Diff: 314 lines (+168/-18)9 files modifiedcmd/juju/main.go (+1/-0)
cmd/juju/main_test.go (+1/-0)
cmd/juju/retryprovisioning.go (+59/-0)
cmd/juju/retryprovisioning_test.go (+82/-0)
state/api/client.go (+8/-9)
state/apiserver/client/client.go (+2/-2)
state/apiserver/client/client_test.go (+2/-2)
state/apiserver/common/setstatus.go (+5/-0)
state/apiserver/common/setstatus_test.go (+8/-5)
Changed in juju-core: | |
importance: | Medium → Low |
tags: | added: landscape |
Changed in juju-core: | |
milestone: | none → 1.18.0 |
tags: | added: charmers |
Changed in juju-core: | |
milestone: | 1.18.0 → 1.17.5 |
Changed in juju-core: | |
assignee: | nobody → Ian Booth (wallyworld) |
status: | Triaged → In Progress |
Changed in juju-core: | |
milestone: | 1.18.0 → 1.17.7 |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
assignee: | Ian Booth (wallyworld) → nobody |
Changed in juju-core: | |
milestone: | none → next-stable |
tags: | added: reliability |
Changed in juju-core: | |
milestone: | 1.21 → 1.22 |
Changed in juju-core: | |
milestone: | 1.22 → 1.23 |
Changed in juju-core: | |
milestone: | 1.23 → none |
importance: | High → Medium |
tags: | added: adoption |
To post a comment you must log in.
I believe the intent was that we could add a "juju resolved" command that would let you indicate to Juju that the error has been resolved and Juju should try again.