Merge lp://staging/~sinzui/juju-ci-tools/ha-recovery into lp://staging/juju-ci-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 1609
Proposed branch: lp://staging/~sinzui/juju-ci-tools/ha-recovery
Merge into: lp://staging/juju-ci-tools
Diff against target: 90 lines (+41/-3)
2 files modified
assess_recovery.py (+8/-3)
tests/test_assess_recovery.py (+33/-0)
To merge this branch: bzr merge lp://staging/~sinzui/juju-ci-tools/ha-recovery
Reviewer Review Type Date Requested Status
Christopher Lee (community) Approve
Review via email: mp+306297@code.staging.launchpad.net

Description of the change

Do not wait for dead controller machines to be resurrected.

The recent change to check the status of agents and workloads does not work for the ha-backup scenario where we expect the controller to report 2 lost machines.

In the future we want to change the the test to delete the dead machines and enable-ha, but
https://launchpad.net/bugs/1449633 means we cannot.

To post a comment you must log in.
1610. By Curtis Hovey

Put the charm in a non-ready state after setup for the test to check on new ready.

Revision history for this message
Christopher Lee (veebers) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches