Merge lp://staging/~abentley/juju-ci-tools/assess-cloud-2 into lp://staging/juju-ci-tools
Status: | Merged |
---|---|
Merged at revision: | 1752 |
Proposed branch: | lp://staging/~abentley/juju-ci-tools/assess-cloud-2 |
Merge into: | lp://staging/juju-ci-tools |
Prerequisite: | lp://staging/~abentley/juju-ci-tools/assess-cloud |
Diff against target: | 395 lines (+199/-57) |
To merge this branch: | bzr merge lp://staging/~abentley/juju-ci-tools/assess-cloud-2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Seman (community) | Approve | ||
Review via email: mp+311439@code.staging.launchpad.net |
Commit message
Replace wait_for with condition-based version.
Description of the change
Replace wait_for with condition-based implementation.
EnvJujuClient.
This branch replaces the implementation with a more flexible one to be used assess-cloud, and intended to replace (or be used to implement) our existing family of wait_for methods.
This implementation accepts an iterable of conditions and then tests status against each of the conditions. If all the conditions are satisfied, it returns status.
Two examples are provided: WaitForSearch, which implements the original wait_for's test, and WaitMachineNotP
Status errors are handled via the new Status.
Planned but not implemented yet:
- GroupReporter output
- Logging status text on exception.
Thank you.