zookeeper connection needs to detect errors differently
Bug #825307 reported by
William Reade
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| pyjuju |
Fix Released
|
Undecided
|
William Reade | ||
Bug Description
We no longer get EnvironmentPending errors, so we will often try to connect to ZK machines that aren't yet up.
In addition to handling launch_time -- if available -- we should also detect the case where the ProviderMachine has no dns_name and supply a custom error message.
This needs to work for both ec2 and orchestra.
Related branches
lp://staging/~fwereade/pyjuju/cobbler-zk-connect
- Jim Baker (community): Approve
- Gustavo Niemeyer: Approve
-
Diff: 1884 lines (+551/-353)50 files modifiedensemble/agents/provision.py (+4/-4)
ensemble/agents/tests/test_provision.py (+3/-3)
ensemble/agents/tests/test_unit.py (+4/-4)
ensemble/control/status.py (+3/-3)
ensemble/control/tests/test_status.py (+0/-2)
ensemble/errors.py (+7/-19)
ensemble/formula/bundle.py (+3/-3)
ensemble/formula/provider.py (+1/-1)
ensemble/formula/tests/test_bundle.py (+8/-4)
ensemble/formula/tests/test_config.py (+6/-4)
ensemble/formula/tests/test_repository.py (+4/-4)
ensemble/hooks/tests/test_arguments.py (+2/-2)
ensemble/hooks/tests/test_cli.py (+5/-2)
ensemble/hooks/tests/test_executor.py (+1/-1)
ensemble/hooks/tests/test_invoker.py (+1/-1)
ensemble/lib/tests/test_schema.py (+1/-1)
ensemble/machine/__init__.py (+1/-3)
ensemble/machine/tests/test_unit_deployment.py (+6/-4)
ensemble/machine/unit.py (+1/-1)
ensemble/providers/common/base.py (+25/-21)
ensemble/providers/common/connect.py (+35/-26)
ensemble/providers/common/findzookeepers.py (+1/-1)
ensemble/providers/common/tests/test_connect.py (+133/-0)
ensemble/providers/common/tests/test_findzookeepers.py (+27/-2)
ensemble/providers/common/tests/test_utils.py (+2/-5)
ensemble/providers/common/utils.py (+3/-1)
ensemble/providers/ec2/__init__.py (+7/-25)
ensemble/providers/ec2/launch.py (+2/-3)
ensemble/providers/ec2/machine.py (+1/-6)
ensemble/providers/ec2/securitygroup.py (+6/-7)
ensemble/providers/ec2/tests/common.py (+0/-10)
ensemble/providers/ec2/tests/test_connect.py (+48/-90)
ensemble/providers/ec2/tests/test_getmachines.py (+2/-3)
ensemble/providers/ec2/tests/test_launch.py (+3/-4)
ensemble/providers/ec2/tests/test_machine.py (+7/-26)
ensemble/providers/ec2/tests/test_provider.py (+4/-1)
ensemble/providers/ec2/tests/test_securitygroup.py (+6/-6)
ensemble/providers/orchestra/__init__.py (+4/-4)
ensemble/providers/orchestra/cobbler.py (+5/-5)
ensemble/providers/orchestra/machine.py (+4/-0)
ensemble/providers/orchestra/tests/test_cobbler.py (+14/-9)
ensemble/providers/orchestra/tests/test_connect.py (+111/-0)
ensemble/providers/orchestra/tests/test_launch.py (+14/-6)
ensemble/providers/orchestra/tests/test_provider.py (+4/-2)
ensemble/state/service.py (+2/-1)
ensemble/state/sshclient.py (+2/-0)
ensemble/state/tests/test_security.py (+1/-1)
ensemble/state/tests/test_service.py (+9/-5)
ensemble/tests/test_errors.py (+6/-16)
ensemble/unit/tests/test_formula.py (+2/-1)
| Changed in ensemble: | |
| status: | New → In Progress |
| assignee: | nobody → William Reade (fwereade) |
| milestone: | none → dublin |
| Changed in ensemble: | |
| milestone: | dublin → eureka |
| Changed in ensemble: | |
| status: | In Progress → Fix Released |
To post a comment you must log in.
