Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1701)

Issue 81340043: Introduce InstanceDistributor policy

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 1 month ago by axw
Modified:
10 years, 1 month ago
Reviewers:
mp+212994
Visibility:
Public.

Description

Introduce InstanceDistributor policy This is a policy that is invoked whenever we attempt to assign a unit to an empty machine. The policy will be invoked with the associated instances (i.e. provisioned, empty machines) and the distribution group instances (i.e. all other instances which have assigned units of the same type.) Azure will return nil, whereas ec2 will want to check which availability zone each machine is in and distribute for maximum distance. https://code.launchpad.net/~axwalk/juju-core/instancedistributor-policy/+merge/212994 Requires: https://code.launchpad.net/~axwalk/juju-core/startinstance-principalunit/+merge/210746 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+174 lines, -32 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M environs/statepolicy.go View 1 chunk +11 lines, -0 lines 0 comments Download
M state/apiserver/provisioner/provisioner.go View 1 chunk +3 lines, -20 lines 0 comments Download
A state/distribution.go View 1 chunk +77 lines, -0 lines 0 comments Download
M state/policy.go View 3 chunks +24 lines, -0 lines 0 comments Download
M state/unit.go View 2 chunks +57 lines, -12 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b