Code review comment for lp://staging/~fnordahl/openstack-charm-testing/kv3-update
Revision history for this message
| Billy Olsen (billy-olsen) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
4b6b663
demo site
(Get the code!)
@Alex - I never took it as snarky :-)
My comment was mostly to suggest that multiple units should be handled differently since a different address is important here. A subsequent merge proposal can be provided for the HA/multiple unit case. I see no reason to hold this change up based on the multiple unit scenario as the previous cases using juju-deployer do not properly handle that case either.