api/uniter/settings: Machine addressability affects how unit settings are updated
Bug #1221798 reported by
Dimiter Naydenov
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
Low
|
Ian Booth |
Bug Description
Once machine addressability changes land, we need to fix a few places in Uniter API:
* (Set)(Private|
* The Settings type must eliminate race conditions (*not* specific to private-address, which must *not* ever be updated automatically).
* ProviderType() is currently only needed to get the unit addresses at uniter startup,
it won't be needed anymore.
Related branches
lp:~wallyworld/juju-core/remove-provider-addr-methods
- Juju Engineering: Pending requested
-
Diff: 1037 lines (+105/-556)21 files modifiedcmd/juju/status_test.go (+10/-4)
environs/interface.go (+0/-6)
provider/all/all.go (+1/-1)
provider/azure/environprovider.go (+0/-105)
provider/azure/environprovider_test.go (+0/-112)
provider/dummy/environs.go (+0/-8)
provider/ec2/ec2.go (+0/-42)
provider/ec2/export_test.go (+0/-12)
provider/ec2/local_test.go (+0/-20)
provider/joyent/provider.go (+0/-17)
provider/local/environprovider.go (+0/-16)
provider/maas/environprovider.go (+0/-26)
provider/maas/environprovider_test.go (+0/-26)
provider/manual/provider.go (+0/-13)
provider/openstack/export_test.go (+0/-26)
provider/openstack/local_test.go (+0/-38)
provider/openstack/provider.go (+0/-31)
state/unit.go (+6/-4)
state/unit_test.go (+60/-12)
worker/uniter/modes.go (+0/-26)
worker/uniter/uniter_test.go (+28/-11)
description: | updated |
tags: | added: tech-debt |
Changed in juju-core: | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in juju-core: | |
importance: | Medium → Low |
tags: | added: addressability |
Changed in juju-core: | |
assignee: | nobody → Ian Booth (wallyworld) |
Changed in juju-core: | |
milestone: | none → 1.19.0 |
status: | Triaged → Fix Committed |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.