Merge lp://staging/~wallyworld/juju-core/watcher-address-issues into lp://staging/~go-bot/juju-core/trunk
Proposed by
Ian Booth
| Status: | Superseded |
|---|---|
| Proposed branch: | lp://staging/~wallyworld/juju-core/watcher-address-issues |
| Merge into: | lp://staging/~go-bot/juju-core/trunk |
| Diff against target: |
164 lines (+82/-19) 3 files modified
state/machine.go (+19/-11) state/megawatcher.go (+25/-8) state/megawatcher_internal_test.go (+38/-0) |
| To merge this branch: | bzr merge lp://staging/~wallyworld/juju-core/watcher-address-issues |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Juju Engineering | Pending | ||
|
Review via email:
|
|||
Description of the change
Ensure unit addresses populated
Change the internals of the megawatcher to ensure
that machine addresses are correct and that the
public and private addresses for untis come from
the assigned machine.
To post a comment you must log in.
Revision history for this message
| Ian Booth (wallyworld) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Reviewers: mp+213967_
Message:
Please take a look.
Description:
Ensure unit addresses populated
Change the internals of the megawatcher to ensure
that machine addresses are correct and that the
public and private addresses for untis come from
the assigned machine.
https:/
(do not edit description out of merge proposal)
Please review this at https:/
Affected files (+84, -19 lines):
A [revision details]
M state/machine.go
M state/megawatch
M state/megawatch