lp:~pjdc/mojo/juju-2-no-fail-when-retrying-errors

Branch merges

Propose for merging

No branches dependent on this one.

On hold for merging into lp:mojo

Mojo Maintainers: Pending requested 2017-03-22

Diff: 395 lines (+228/-36)

4 files modified

mojo/juju/status.py (+53/-15)
mojo/tests/test_juju2.py (+64/-15)
mojo/tests/test_phase.py (+21/-6)
mojo/tests/testdata/juju2-model-config.yaml (+90/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

414. By Paul Collins on 2017-04-12

Juju2StatusTestCaseNoAutomaticallyRetryHooks: inherit from Juju2StatusTestCase so that all tests are run for both situations

413. By Paul Collins on 2017-04-12

merge trunk

412. By Paul Collins on 2017-04-12

Juju2Status.__init__: automatically_retry_hooks_default=None so we can tell when it's been explicitly set

411. By Paul Collins on 2017-03-29

update tests to consider "error" a busy state, and add Juju2StatusTestCaseNoAutomaticallyRetryHooks

410. By Paul Collins on 2017-03-29

Juju2Status.__init__: when status_yaml is set, consult automatically_retry_hooks_default instead of self.get_automatically_retry_hooks()

409. By Paul Collins on 2017-03-22

Juju2Status: use a per-instance copy of the juju status, since we modify it; this appears to mainly affect the test suite

408. By Paul Collins on 2017-03-22

get_automatically_retry_hooks: default the first .get() to an empty dict for the next .get()

407. By Paul Collins on 2017-03-22

Juju2Status: consult automatically-retry-hooks when deciding whether "error" is a busy state or an error state

406. By Paul Collins on 2017-03-22

Juju2Status: Juju 2 defaults to retrying errors, so treat it as a busy state

We should really check to see if automatically-retry-hooks is enabled,
but for now we assume it is still at the default value of "true".

405. By Laurent Sesquès on 2017-03-13

[mthaddon,r=sajoupa] Add info to jenkins bzr job that includes versions of binaries and variables for clarity